Skip to content

[v3-4-test] Derive the expected CI image name from the branch in Breeze CI image tests (#76206) - #76207

Merged
potiuk merged 1 commit into
apache:v3-4-testfrom
potiuk:fix-ci-image-test-branch-v3-4
Nov 1, 2026
Merged

potiuk merged 1 commit into
apache:v3-4-testfrom
potiuk:fix-ci-image-test-branch-v3-4

Conversation

@potiuk

@potiuk potiuk commented Nov 1, 2026

Copy link
Copy Markdown
Member

Backport of #76206, which fixes the failing "Breeze unit tests" job on v3-4-test.

test_ci_image_commands.py compared the image Breeze inspects against a hard-coded ghcr.io/apache/airflow/main/ci/python3.11. On a release branch Breeze names the image after the branch, so after the backport of #74173 (#76183) test_load_restores_snapshot_with_the_checks_of_an_image_file_load fails on v3-4-test, which fails the "Breeze unit tests" job there. The test now builds the expected name with BuildCiParams(python="3.11").airflow_image_name, the same derivation the command uses, so it holds on main and on every release branch.

Verified: the test file passes on main (22 passed), and on v3-4-test the failing test fails before this change and the whole file passes after it.


Was generative AI tooling used to co-author this PR?
  • Yes — Claude Code (Claude Opus 5)

Generated-by: Claude Code (Claude Opus 5) following the guidelines

…tests

The test hard-coded the main image name, so it fails on release branches such as v3-4-test, where Breeze names the image after the branch.

Generated-by: Claude Opus 5
@potiuk
potiuk requested a review from ashb as a code owner November 1, 2026 10:53
@potiuk
potiuk enabled auto-merge (squash) November 1, 2026 10:54
@potiuk
potiuk disabled auto-merge November 1, 2026 10:55
@potiuk
potiuk merged commit 4e566b0 into apache:v3-4-test Nov 1, 2026
49 of 52 checks passed
@potiuk
potiuk deleted the fix-ci-image-test-branch-v3-4 branch November 1, 2026 10:56
vatsrahul1001 pushed a commit that referenced this pull request Nov 4, 2026
…tests (#76207)

The test hard-coded the main image name, so it fails on release branches such as v3-4-test, where Breeze names the image after the branch.

Generated-by: Claude Opus 5
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant