Skip to content

Conversation

@plural-copilot
Copy link

Summary:
The current /ping endpoint raises an intentional exception approximately one-third of the time, causing 500 Internal Server Errors and triggering alerts for high error rates. This PR removes the intentional exception to make /ping reliably respond with success.

Changes Made:

  • Removed conditional exception raise in /ping endpoint in app/main.py.

Rationale:
This change ensures the /ping endpoint, typically used for Kubernetes health checks, always returns a successful response. This will prevent spurious 500 errors, keep the alert from firing, and improve service reliability.

After merging, the updated service should be redeployed to clear the alert.

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.

2 participants