Skip to content

CI: Build CI image after Dockerfiles are updated in breeze ci upgrade#64777

Merged
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-ci-upgrade-image-order
Apr 6, 2026
Merged

CI: Build CI image after Dockerfiles are updated in breeze ci upgrade#64777
potiuk merged 1 commit into
apache:mainfrom
potiuk:fix-ci-upgrade-image-order

Conversation

@potiuk

@potiuk potiuk commented Apr 6, 2026

Copy link
Copy Markdown
Member

Previously, breeze ci upgrade built the CI image before running
upgrade-important-versions, which updates Dockerfiles. This meant
the update-uv-lock step (which runs inside the image) used a stale
image that didn't reflect the Dockerfile changes.

Now the upgrade commands are split into two phases:

  • Pre-image: autoupdate, update-chart-dependencies, upgrade-important-versions
  • Image build: breeze ci-image build --python 3.10
  • Post-image: update-uv-lock (runs inside the freshly built image)

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

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

@potiuk potiuk merged commit b875c40 into apache:main Apr 6, 2026
41 checks passed
@potiuk potiuk deleted the fix-ci-upgrade-image-order branch April 6, 2026 16:32
github-actions Bot pushed a commit that referenced this pull request Apr 6, 2026
…e ci upgrade (#64777)

(cherry picked from commit b875c40)

Co-authored-by: Jarek Potiuk <[email protected]>
@github-actions

github-actions Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-2-test

Note: As of Merging PRs targeted for Airflow 3.X
the committer who merges the PR is responsible for backporting the PRs that are bug fixes (generally speaking) to the maintenance branches.

In matter of doubt please ask in #release-management Slack channel.

Status Branch Result
v3-2-test PR Link

potiuk added a commit that referenced this pull request Apr 6, 2026
…e ci upgrade (#64777)

(cherry picked from commit b875c40)

Co-authored-by: Jarek Potiuk <[email protected]>
potiuk added a commit that referenced this pull request Apr 6, 2026
…e ci upgrade (#64777) (#64781)

(cherry picked from commit b875c40)

Co-authored-by: Jarek Potiuk <[email protected]>
else:
console_print(f"[info]Skipping {step_name} (disabled).[/]")

# Build the CI image for Python 3.10 after Dockerfiles have been updated so that

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is 3.10 mentioned specifically? Was this not an issue in other versions?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We decided to always use lowest version in CI to ensure that we are really compatible to the oldest supported Python version.

vatsrahul1001 pushed a commit that referenced this pull request Apr 8, 2026
…e ci upgrade (#64777) (#64781)

(cherry picked from commit b875c40)

Co-authored-by: Jarek Potiuk <[email protected]>
vatsrahul1001 pushed a commit that referenced this pull request Apr 15, 2026
…e ci upgrade (#64777) (#64781)

(cherry picked from commit b875c40)

Co-authored-by: Jarek Potiuk <[email protected]>
vatsrahul1001 pushed a commit that referenced this pull request Apr 15, 2026
…e ci upgrade (#64777) (#64781)

(cherry picked from commit b875c40)

Co-authored-by: Jarek Potiuk <[email protected]>
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.

3 participants