Skip to content
This repository has been archived by the owner on Feb 22, 2023. It is now read-only.

Add database connectivity to healthcheck endpoint #1079

Closed
sarayourfriend opened this issue Jan 12, 2023 · 0 comments · Fixed by #1083
Closed

Add database connectivity to healthcheck endpoint #1079

sarayourfriend opened this issue Jan 12, 2023 · 0 comments · Fixed by #1083
Labels
🕹 aspect: interface Concerns end-users' experience with the software ✨ goal: improvement Improvement to an existing user-facing feature good first issue New-contributor friendly help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon
Projects
Openverse

Comments

@sarayourfriend
Copy link
Contributor

sarayourfriend commented Jan 12, 2023 •
edited by zackkrida

Problem

The healtcheck endpoint should check that the database is accessible. If the db is inaccessible, the service is definitively not healthy.

Description

Add another check (in addition to the ES check) for the database connectivity. Calling django.db.connection.ensure_connection() should be sufficient. It raises an error when the database connection is unavailable.

Alternatives

Additional context
@sarayourfriend sarayourfriend added good first issue New-contributor friendly help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon ✨ goal: improvement Improvement to an existing user-facing feature 🕹 aspect: interface Concerns end-users' experience with the software labels Jan 12, 2023
@openverse-bot openverse-bot added this to Backlog in Openverse Jan 12, 2023
@krysal krysal mentioned this issue Jan 13, 2023
Merged
7 tasks
@openverse-bot openverse-bot moved this from Backlog to In progress in Openverse Jan 13, 2023
Openverse automation moved this from In progress to Done! Feb 1, 2023
@sarayourfriend sarayourfriend mentioned this issue May 3, 2023
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
🕹 aspect: interface Concerns end-users' experience with the software ✨ goal: improvement Improvement to an existing user-facing feature good first issue New-contributor friendly help wanted Open to participation from the community 🟨 priority: medium Not blocking but should be addressed soon
Projects
No open projects
Openverse
  
Done!
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add database connectivity to healthcheck endpoint WordPress/openverse-api
1 participant
@sarayourfriend