chore: switch health check to lightweight /health endpoint #119

Open
opened 2026-04-09 16:40:32 +00:00 by cal · 1 comment
Owner

Context

The Dockerfile HEALTHCHECK currently hits /api/v3/current, which makes a DB query on every check. If the database is temporarily slow, Docker may restart a healthy container.

Task

Switch the HEALTHCHECK to use a lightweight /health endpoint (verify it exists or create one). The endpoint should return 200 without hitting the database.

HEALTHCHECK --interval=30s --timeout=10s --start-period=10s --retries=3 \
    CMD curl -f http://localhost:80/health || exit 1
## Context The Dockerfile HEALTHCHECK currently hits `/api/v3/current`, which makes a DB query on every check. If the database is temporarily slow, Docker may restart a healthy container. ## Task Switch the HEALTHCHECK to use a lightweight `/health` endpoint (verify it exists or create one). The endpoint should return 200 without hitting the database. ```dockerfile HEALTHCHECK --interval=30s --timeout=10s --start-period=10s --retries=3 \ CMD curl -f http://localhost:80/health || exit 1 ```
Claude added the
ai-working
label 2026-04-09 19:01:17 +00:00
Claude added
ai-pr-opened
and removed
ai-working
labels 2026-04-09 19:02:35 +00:00
Collaborator

PR #125 opened: #125

Added GET /health to app/main.py (returns {"status": "ok"}, no DB queries) and updated the Dockerfile HEALTHCHECK to use /health instead of /api/v3/current.

PR #125 opened: https://git.manticorum.com/cal/major-domo-database/pulls/125 Added `GET /health` to `app/main.py` (returns `{"status": "ok"}`, no DB queries) and updated the Dockerfile `HEALTHCHECK` to use `/health` instead of `/api/v3/current`.
Sign in to join this conversation.
No Milestone
No project
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: cal/major-domo-database#119
No description provided.