Skip to content

Upgrade django_health_checks#3602

Open
rhysyngsun wants to merge 1 commit into
mainfrom
nl/upgrade-health-checks
Open

Upgrade django_health_checks#3602
rhysyngsun wants to merge 1 commit into
mainfrom
nl/upgrade-health-checks

Conversation

@rhysyngsun
Copy link
Copy Markdown
Collaborator

What are the relevant tickets?

N/A

Description (What does it do?)

Upgrade django-health-checks to the latest version. This included breaking changes to the setup so that's addressed here. The migration check did not carry over as they dropped that because the django check framework is supposed to be used to check that.

How can this be tested?

Run the app, you should then be able to access the health check urls:

http://mitxonline.odl.local/health/startup/?format=json
http://mitxonline.odl.local/health/liveness/?format=json
http://mitxonline.odl.local/health/readiness/?format=json
http://mitxonline.odl.local/health/full/?format=json

@github-actions
Copy link
Copy Markdown

OpenAPI Changes

Show/hide ## Changes for v0.yaml:
## Changes for v0.yaml:
No changes detected

## Changes for v1.yaml:
No changes detected

## Changes for v2.yaml:
No changes detected

Unexpected changes? Ensure your branch is up-to-date with main (consider rebasing).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant