Skip to content

[v3-3-test] Fix scheduled CI upgrade failing when its PR already exists (#69018) - #69055

Merged
potiuk merged 1 commit into
v3-3-testfrom
backport-419ac06-v3-3-test
Jun 26, 2026
Merged

[v3-3-test] Fix scheduled CI upgrade failing when its PR already exists (#69018)#69055
potiuk merged 1 commit into
v3-3-testfrom
backport-419ac06-v3-3-test

Conversation

@github-actions

Copy link
Copy Markdown
Contributor

The scheduled "breeze ci upgrade" job force-pushes a stable branch
(ci-upgrade-) and then looks for an existing PR before creating
one. The lookup passed the "owner:branch" head label to "gh pr list
--head", but that flag filters by the bare branch name only and does not
support the "owner:branch" syntax, so the existing PR was never found and
the job fell through to "gh pr create" — which rejected the duplicate and
failed the run.

Use the bare branch name for the "gh pr list" and "gh pr ready" lookups
(only "gh pr create --head" needs the cross-fork "owner:branch" label),
and treat an "already exists" creation error as success since the branch
has already been force-pushed.
(cherry picked from commit 419ac06)

Co-authored-by: Shahar Epstein 60007259+shahar1@users.noreply.github.com

…ts (#69018)
The scheduled "breeze ci upgrade" job force-pushes a stable branch
(ci-upgrade-<branch>) and then looks for an existing PR before creating
one. The lookup passed the "owner:branch" head label to "gh pr list
--head", but that flag filters by the bare branch name only and does not
support the "owner:branch" syntax, so the existing PR was never found and
the job fell through to "gh pr create" — which rejected the duplicate and
failed the run.
Use the bare branch name for the "gh pr list" and "gh pr ready" lookups
(only "gh pr create --head" needs the cross-fork "owner:branch" label),
and treat an "already exists" creation error as success since the branch
has already been force-pushed.
(cherry picked from commit 419ac06)
Co-authored-by: Shahar Epstein <60007259+shahar1@users.noreply.github.com>
@potiuk
potiuk marked this pull request as ready for review June 26, 2026 20:42
@potiuk
potiuk merged commit c2b9615 into v3-3-testJun 26, 2026
26 of 34 checks passed
@potiuk
potiuk deleted the backport-419ac06-v3-3-test branch June 26, 2026 20:42
Sign up for freeto 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.

2 participants

@potiuk@shahar1