You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/08/07 17:21:20 UTC
[PR] Fix tag-providers --dry-run failing to find the push remote [airflow]
potiuk opened a new pull request, #71303:
URL: https://github.com/apache/airflow/pull/71303
`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?
- [X] Yes — Claude Code (Opus 5)
Generated-by: Claude Code (Opus 5) following [the guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix tag-providers --dry-run failing to find the push remote [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #71303:
URL: https://github.com/apache/airflow/pull/71303
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix tag-providers --dry-run failing to find the push remote [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #71303:
URL: https://github.com/apache/airflow/pull/71303#issuecomment-5320183165
@vatsrahul1001 @jason810496 — you both review a fair amount of the breeze and CI tooling; would either of you have a moment for this one?
It fixes `tag-providers --dry-run` failing to resolve the push remote, so the dry run no longer errors out in a case where the real run would have worked. Two files, release-manager-facing. #71304 is a sibling change in the same file.
---
Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix tag-providers --dry-run failing to find the push remote [airflow]
Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk commented on PR #71303:
URL: https://github.com/apache/airflow/pull/71303#issuecomment-5318139469
This has been waiting for a review for a couple of weeks. It fixes `tag-providers --dry-run` failing to resolve the push remote, so the dry run no longer errors out where the real run would have worked.
@shahar1 @jscheffl this is release-manager-facing and small — does either of you have a moment? (#71304 is a sibling change in the same file.)
---
Drafted-by: Claude Code (Opus 5); reviewed by @potiuk before posting
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] Fix tag-providers --dry-run failing to find the push remote [airflow]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #71303:
URL: https://github.com/apache/airflow/pull/71303#issuecomment-6253090869
### Backport successfully created: v3-4-test
Note: As of [Merging PRs targeted for Airflow 3.X](https://github.com/apache/airflow/blob/main/dev/README_AIRFLOW3_DEV.md#merging-prs-targeted-for-airflow-3x)
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](https://apache-airflow.slack.com/archives/C03G9H97MM2) Slack channel.
<table>
<tr>
<th>Status</th>
<th>Branch</th>
<th>Result</th>
</tr>
<tr>
<td>✅</td>
<td>v3-4-test</td>
<td><a href="https://github.com/apache/airflow/pull/75020"><img src="https://img.shields.io/badge/PR-75020-blue" alt="PR Link"></a></td>
</tr>
</table>
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: commits-unsubscribe@airflow.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org