Skip to content

ci: use Compose healthchecks for database startup - #67

Merged
lovasoa merged 2 commits into
mainfrom
codex/bounded-database-readiness
Sep 28, 2026
Merged

lovasoa merged 2 commits into
mainfrom
codex/bounded-database-readiness

Conversation

@lovasoa

@lovasoa lovasoa commented Sep 27, 2026 •

Copy link
Copy Markdown
Collaborator

Define native database healthchecks in Compose and replace CI's fixed sleeps and unbounded polling with docker compose up --wait --wait-timeout 120. Each database job prints Compose logs on failure. CI selects its existing fixed ports through environment variables; local runs retain random loopback ports.

Only the workflow and Compose file change. No custom polling scripts, Python mocks, or additional CI jobs.

Validation: Compose configuration and workflow shell syntax checks pass. Native Compose waits passed with PostgreSQL 16 and MySQL 9.5 containers; fixtures were copied into those containers because local bind mounts were denied. The full database version matrix remains for CI.

@lovasoa
lovasoa added this pull request to stack #71 September 27, 2026 23:16
@lovasoa
lovasoa force-pushed the codex/bounded-database-readiness branch from 71cd9b3 to 01aed7f Compare September 27, 2026 23:16
@lovasoa lovasoa changed the title ci: wait for database test schemas with bounded probes ci: use Compose healthchecks for database startup Sep 27, 2026
@lovasoa
lovasoa added this pull request to the merge queue Sep 28, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to no response for status checks Sep 28, 2026
@lovasoa
lovasoa added this pull request to the merge queue Sep 28, 2026
@lovasoa
lovasoa removed this pull request from the merge queue due to a manual request Sep 28, 2026
@lovasoa
lovasoa added this pull request to the merge queue Sep 28, 2026
Merged via the queue into main with commit e81c1d1 Sep 28, 2026
27 checks passed
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.

1 participant