Andrushika opened a new pull request, #73320:
URL: https://github.com/apache/airflow/pull/73320

   ## Why
   
   Follow-up of #73167.
   
   `breeze` starts an integration container (redis, cassandra, trino, ...) and 
waits until Docker marks it healthy before the airflow container may start. 
Docker probes the healthcheck every `start_interval` during `start_period`, and 
the default is 5 seconds. Most of these services are ready in 1 to 3 seconds, 
so the first probe at 5 seconds is what everyone waits for. #73167 fixed this 
for the postgres and mysql backends. The review there pointed out the same gap 
in the integration files, so this PR covers the rest.
   
   ## What
   
   Add `start_interval: 1s` to the 13 healthchecks under 
`scripts/ci/docker-compose/integration-*.yml` that have a `start_period`. Same 
key, same value as #73167. The Docker Engine 25 / Compose 2.20.2 floor in 
`global_constants.py` already covers it.
   
   Measured from `compose up` to healthy:
   
   | | before | after |
   |---|---|---|
   | redis | 5.4s | 1.3s |
   | cassandra | 5.9s | 1.8s |
   
   related: #73167
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Fable 5.1)
   
   Generated-by: Claude Code (Fable 5.1) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to