Skip to content

fix(flashlight): add MariaDB sidecar + bump readiness timeout to 240s#4

Merged
PrestaEdit merged 1 commit into
mainfrom
fix/flashlight-mysql-sidecar
Jul 11, 2026
Merged

fix(flashlight): add MariaDB sidecar + bump readiness timeout to 240s#4
PrestaEdit merged 1 commit into
mainfrom
fix/flashlight-mysql-sidecar

Conversation

@PrestaEdit

Copy link
Copy Markdown
Contributor

Root cause of Flashlight not ready after 120000ms

PrestaShop Flashlight does not embed MySQL. It depends on an external MariaDB service via depends_on: condition: service_healthy. Without that sidecar, PrestaShop waits forever for a DB it never gets and never binds to :80.

Confirmed against the reference basic-example docker-compose.yml.

Changes

  • compose-template: adds mysql service (mariadb:lts) with the healthcheck the Flashlight docs prescribe (healthcheck.sh --connect, 10s interval, 5 retries). PrestaShop now waits for MySQL to be healthy before starting via depends_on.
  • docker.ts: waitFor timeout raised 120s → 240s. Cold-boot on GH Actions includes MySQL image pull (~10s) + healthcheck loop (~30–60s) + PS install (~60–90s). 120s was systematically hitting the cliff.

Test plan

  • Existing 41 unit tests + 1 new (sidecar assertion) = 42/42.
  • Bundle rebuilt.
  • Re-run github-action-smoke after merge — expect Flashlight to become ready and the composer step to run against http://localhost:8000.

🤖 Generated with Claude Code

Root cause of 'Flashlight not ready after 120000ms': Flashlight does
NOT embed MySQL — it depends on an external MariaDB service via
depends_on: condition: service_healthy. Without it, PrestaShop waits
forever for a DB it never gets.

Changes:
- compose-template: add mysql service (mariadb:lts) with the exact
  healthcheck the Flashlight docs prescribe (healthcheck.sh --connect,
  10s interval, 5 retries). PrestaShop now waits for MySQL health
  before starting.
- docker: waitFor timeout raised from 120s to 240s. Cold-boot on GH
  Actions runners includes: MySQL image pull + healthcheck loop
  (~30-60s) + PS install (~60-90s). 120s was systematically hitting
  the cliff.

Ref: https://github.com/PrestaShop/prestashop-flashlight/blob/main/examples/basic-example/docker-compose.yml

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@PrestaEdit
PrestaEdit merged commit e96a716 into main Jul 11, 2026
1 check 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