Skip to content

Show prepare breeze timing in CI duration alerts#69732

Merged
potiuk merged 1 commit into
apache:mainfrom
gopidesupavan:improve-duration-script
Jul 11, 2026
Merged

Show prepare breeze timing in CI duration alerts#69732
potiuk merged 1 commit into
apache:mainfrom
gopidesupavan:improve-duration-script

Conversation

@gopidesupavan

Copy link
Copy Markdown
Member

CI duration alerts currently identify slower jobs but do not surface whether
the setup phase accounts for the change. Showing the prepare breeze step timing
makes common CI image/cache slowdowns visible without adding noisy per-step
analysis for every job.

image

The first job spent most of its extra time preparing the image, and in some cases that includes downloading the stash. Adding the Prepare breeze & CI image timing makes that setup cost visible directly in the duration alert.


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
  • codex

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@gopidesupavan

Copy link
Copy Markdown
Member Author

it produces now something like this:

image

@gopidesupavan
gopidesupavan force-pushed the improve-duration-script branch from 8f7b18c to 3120d43 Compare July 10, 2026 21:33
@potiuk
potiuk merged commit 2e59622 into apache:main Jul 11, 2026
65 checks passed
@github-actions

Copy link
Copy Markdown
Contributor

Backport successfully created: v3-3-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-3-test PR Link

@potiuk

potiuk commented Jul 11, 2026

Copy link
Copy Markdown
Member

Let's see - sometimes that duration will be longer - mostly when we need to rebuild the image. I think we should simply exclude the image build time and track it separately.,

potiuk pushed a commit that referenced this pull request Jul 11, 2026
…#69753)

(cherry picked from commit 2e59622)

Co-authored-by: GPK <gopidesupavan@gmail.com>
joshuabvarghese pushed a commit to joshuabvarghese/airflow that referenced this pull request Jul 16, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants