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

   ## Summary
   
   The parallel test timeout exists so that a hanging test group is stopped 
while the job can still dump the container logs and upload them as artifacts. 
It has never managed that: the alarm is armed in the job's last step, while the 
65 minute budget starts at the first. Restoring the CI image takes six to eight 
minutes and the migration tests another four to nineteen, so a fixed 60 minute 
test timeout could only fire in a job GitHub had already cancelled — every hang 
surfaced as a bare job timeout, with no logs and no sign of which test group 
was stuck.
   
   Deriving the timeout from what is left of the budget, rather than raising 
the budget, restores the guarantee `additional-ci-image-checks.yml` gets from 
`breeze shell --max-time`; that one holds only because nothing of variable 
length runs before it. The five minutes held back cover stopping the containers 
and uploading their logs.
   
   `timeout-minutes` becomes a `workflow_call` input so the tests read the 
budget GitHub actually enforces instead of a second copy that can drift. No 
caller has to pass it.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 5)
   
   Generated-by: Claude Code (Opus 5) 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