This is an automated email from the ASF dual-hosted git repository.
potiuk pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 65d4e7dd6a5 Stop core DB test jobs timing out when migration tests run
(#73231)
65d4e7dd6a5 is described below
commit 65d4e7dd6a5df985232aa0f4a6267801c2d7dade
Author: rjgoyln <[email protected]>
AuthorDate: Tue Sep 22 07:47:06 2026 +0800
Stop core DB test jobs timing out when migration tests run (#73231)
The migration tests run inside the core unit test job, and run_unit_tests.sh
gives the unit tests whatever is left of the job budget once the steps
before
them have taken their share. On MySQL that regularly leaves around half an
hour
of the 65 minutes for tests that need close to it, so the job fails
whenever the
setup has a slow day.
The 65 was sized as the tests' own timeout plus five minutes, with nothing
in it
for the steps that run before them.
---
.github/workflows/ci-amd.yml | 1 +
.github/workflows/ci-arm.yml | 1 +
.github/workflows/run-unit-tests.yml | 4 +++-
3 files changed, 5 insertions(+), 1 deletion(-)
diff --git a/.github/workflows/ci-amd.yml b/.github/workflows/ci-amd.yml
index c8782ee8ad1..7f3f761ac5e 100644
--- a/.github/workflows/ci-amd.yml
+++ b/.github/workflows/ci-amd.yml
@@ -598,6 +598,7 @@ jobs:
include-success-outputs: ${{
needs.build-info.outputs.include-success-outputs }}
run-coverage: ${{ needs.build-info.outputs.run-coverage }}
run-migration-tests: "true"
+ job-timeout-minutes: 80
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests
}}
use-uv: ${{ needs.build-info.outputs.use-uv }}
diff --git a/.github/workflows/ci-arm.yml b/.github/workflows/ci-arm.yml
index 042549f7f7f..2f257f32c5a 100644
--- a/.github/workflows/ci-arm.yml
+++ b/.github/workflows/ci-arm.yml
@@ -587,6 +587,7 @@ jobs:
include-success-outputs: ${{
needs.build-info.outputs.include-success-outputs }}
run-coverage: ${{ needs.build-info.outputs.run-coverage }}
run-migration-tests: "true"
+ job-timeout-minutes: 80
debug-resources: ${{ needs.build-info.outputs.debug-resources }}
skip-providers-tests: ${{ needs.build-info.outputs.skip-providers-tests
}}
use-uv: ${{ needs.build-info.outputs.use-uv }}
diff --git a/.github/workflows/run-unit-tests.yml
b/.github/workflows/run-unit-tests.yml
index 458ee33ca10..c93656ffd50 100644
--- a/.github/workflows/run-unit-tests.yml
+++ b/.github/workflows/run-unit-tests.yml
@@ -130,7 +130,9 @@ on: # yamllint disable-line rule:truthy
required: true
type: string
job-timeout-minutes:
- description: "How long the job may run before GitHub cancels it"
+ description: >
+ How long the job may run before GitHub cancels it. A caller raises
it when its job
+ carries steps expensive enough to squeeze the tests out of the
default.
required: false
default: 65
type: number
