Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

API: Add DB connection check to health check? #622

Closed
QuantumDancer opened this issue Jan 11, 2024 · 0 comments · Fixed by #734
Closed

API: Add DB connection check to health check? #622

QuantumDancer opened this issue Jan 11, 2024 · 0 comments · Fixed by #734
Assignees

Comments

@QuantumDancer
Copy link
Contributor

Right now the health check (/health_check) just returns a 200 OK. This shows that Auditor is running but does not test if the DB connection works. As Auditor itself is stateless we store all the state in the DB. Therefore, we should think about if incorporating a DB connection check to the health check, because without it all requests will fail. The downside of this is that the health check becomes more expensive to perform and that more load is put on the DB.

@QuantumDancer QuantumDancer changed the title Add DB connection check to health check? API: Add DB connection check to health check? Jan 11, 2024
@QuantumDancer QuantumDancer added this to the Release v0.5.0 milestone Feb 19, 2024
@QuantumDancer QuantumDancer self-assigned this Feb 19, 2024
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 a pull request may close this issue.

1 participant