Skip to content

Fix tag-providers --dry-run failing to find the push remote - #71303

Open
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:fix-tag-providers-dry-run
Open

Fix tag-providers --dry-run failing to find the push remote#71303
potiuk wants to merge 1 commit into
apache:mainfrom
potiuk:fix-tag-providers-dry-run

Conversation

@potiuk

@potiuk potiuk commented Aug 7, 2026

Copy link
Copy Markdown
Member

breeze release-management tag-providers --dry-run always aborted with:

ValueError: Could not find the remote configured to push to apache/airflow

A dry run short-circuits every run_command, returning an empty stdout, so
the git remote get-url --push lookup that picks the apache/airflow remote
found nothing for all three candidate names (upstream, origin, apache).

The lookup is read-only, so it now runs even in dry-run mode. That makes
--dry-run usable for previewing the tags a provider wave will create,
which is precisely when a release manager wants to check them.


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

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

A dry run short-circuits every run_command, so the `git remote get-url
--push` lookup that picks the apache/airflow remote came back with empty
stdout for all three candidate names and the command aborted with
"Could not find the remote configured to push to apache/airflow".
That makes --dry-run useless for previewing a provider release's tags,
which is exactly when a release manager wants to check them.
@potiuk
potiuk force-pushed the fix-tag-providers-dry-run branch from 3b61fb0 to 057d80a Compare August 8, 2026 01:21
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.

1 participant