Skip to content

add disable postgres check flag on healthcheck#700

Merged
tsuzukayama merged 1 commit intomainfrom
fix/healthcheck
Dec 15, 2025
Merged

add disable postgres check flag on healthcheck#700
tsuzukayama merged 1 commit intomainfrom
fix/healthcheck

Conversation

@tsuzukayama
Copy link
Contributor

@tsuzukayama tsuzukayama commented Dec 12, 2025

Add internalApi.healthcheck.skipPostgresCheck flag to disable Postgres checking on healthcheck endpoint. Doing this to enable Postgres maintenance. Maestro core functionality should still work even though the DB is offline (Scheduler Management will be down).


Note

Adds a config flag to optionally skip the Postgres check in /readyz and updates the handler/logging and responses accordingly.

  • Internal API:
    • Readiness (/readyz):
      • Add config-gated Postgres check via internalApi.healthcheck.skipPostgresCheck (when true, skip DB connectivity check).
      • Update handler to accept config, apply skip logic, and set Content-Type consistently.
      • Return 503 Service Unavailable with body when unhealthy; log a warning if Postgres check is disabled.
  • Config:
    • Add internalApi.healthcheck.skipPostgresCheck: false default in config/config.yaml.

Written by Cursor Bugbot for commit 640a13a. This will update automatically on new commits. Configure here.

@tsuzukayama tsuzukayama merged commit 196d0f1 into main Dec 15, 2025
7 checks passed
@tsuzukayama tsuzukayama deleted the fix/healthcheck branch December 15, 2025 11:56
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