Skip to content

Fix release dry-run fails with a non-existing workflow - #71131

Merged
potiuk merged 1 commit into
apache:mainfrom
FrankYang0529:claude/airflow-release-dry-run-fix-9df7e4
Aug 6, 2026
Merged

Fix release dry-run fails with a non-existing workflow#71131
potiuk merged 1 commit into
apache:mainfrom
FrankYang0529:claude/airflow-release-dry-run-fix-9df7e4

Conversation

@FrankYang0529

@FrankYang0529 FrankYang0529 commented Aug 5, 2026

Copy link
Copy Markdown
Member

Why

How

  • trigger_workflow_and_monitor() returns right after the dispatch in a dry run.

Verification

  • uv run --project dev/breeze --frozen --no-sync pytest dev/breeze/tests/test_gh_workflow_utils.py dev/breeze/tests/test_release_constraints.py dev/breeze/tests/test_workflow_commands.py -v
  • env CI=true GITHUB_REPOSITORY=apache/airflow breeze release-management start-rc-process --version 3.2.0rc1 --previous-version 3.1.0 --task-sdk-version 1.1.0rc1 --sync-branch v3-2-test --answer yes --dry-run

Was generative AI tooling used to co-author this PR?
  • Yes - Claude Code

  • 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.

@FrankYang0529
FrankYang0529 force-pushed the claude/airflow-release-dry-run-fix-9df7e4 branch 2 times, most recently from 58cd344 to dd31ce6 Compare August 5, 2026 06:43
@FrankYang0529 FrankYang0529 changed the title Fix release dry-run aborting on a workflow run it never dispatched Fix release dry-run fails with a non-existing workflow Aug 5, 2026
@FrankYang0529
FrankYang0529 force-pushed the claude/airflow-release-dry-run-fix-9df7e4 branch from dd31ce6 to 6ce5fb3 Compare August 5, 2026 23:09
Signed-off-by: PoAn Yang <payang@apache.org>
@potiuk
potiuk merged commit de5c819 into apache:main Aug 6, 2026
302 checks passed
@github-actions

github-actions Bot commented Aug 6, 2026

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

github-actions Bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Aug 6, 2026
…ispatched (apache#71131)

(cherry picked from commit de5c819)

Co-authored-by: PoAn Yang <payang@apache.org>
Signed-off-by: PoAn Yang <payang@apache.org>
aws-airflow-bot pushed a commit to aws-mwaa/upstream-to-airflow that referenced this pull request Aug 6, 2026
…ispatched (apache#71131)

(cherry picked from commit de5c819)

Co-authored-by: PoAn Yang <payang@apache.org>
Signed-off-by: PoAn Yang <payang@apache.org>
@FrankYang0529
FrankYang0529 deleted the claude/airflow-release-dry-run-fix-9df7e4 branch August 7, 2026 03:52
potiuk pushed a commit that referenced this pull request Aug 7, 2026
…ispatched (#71131) (#71233)

(cherry picked from commit de5c819)

Signed-off-by: PoAn Yang <payang@apache.org>
Co-authored-by: PoAn Yang <payang@apache.org>
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