Skip to content

fix: switch auto-merge-release-pr to pull_request trigger - #455

Merged
sebastiancorrea81 merged 2 commits into
mainfrom
fix/auto-merge-pull-request-trigger
Jul 30, 2026
Merged

fix: switch auto-merge-release-pr to pull_request trigger#455
sebastiancorrea81 merged 2 commits into
mainfrom
fix/auto-merge-pull-request-trigger

Conversation

@sebastiancorrea81

Copy link
Copy Markdown
Collaborator

Summary

Supersedes #454. That workflow_run-triggered version fails with startup_failure in practice (confirmed on main after merge — 0 jobs scheduled, no further detail available via the API; possibly an org Actions policy restricting workflow_run, but I don't have the access to confirm).

Reverts to the pull_request trigger already proven working in nullplatform/services-postgresql-rds and the other services-* repos, and instead handles the generate-readmes race with concurrency: cancel-in-progress keyed on the PR number:

  • The README-regeneration commit lands ~38s after the release PR is opened (measured on the real chore(main): release 6.7.1 #453 release).
  • That push fires a new synchronize event → a fresh run starts for the same group:.
  • GitHub cancels whatever older run for that PR is still in-flight (waiting on checks), so a stale run can never reach the merge step — only the run for the final commit does.

Test plan

  • Merge this PR.
  • On the next real release-please PR, confirm: the older run (from the initial commit) gets cancelled once the README commit lands, the newer run waits for checks and merges, and the release/tag is created.

…flow_run

The workflow_run-triggered version merged in #454 fails immediately
with startup_failure (0 jobs ever scheduled) for reasons that aren't
visible via the API — likely an org-level Actions policy restricting
workflow_run, but that needs admin access to confirm.

Switch back to the pull_request trigger proven to work in the other
services-* repos, and instead solve the generate-readmes race with
concurrency: cancel-in-progress keyed on the PR number. When the
README-regeneration commit lands (observed ~38s after the release PR
is opened, on a real release), it fires a new 'synchronize' event;
GitHub cancels whatever older run for the same PR is still waiting on
checks, so only the run for the final commit can ever reach the merge
step.
@sebastiancorrea81
sebastiancorrea81 merged commit 6475fb6 into main Jul 30, 2026
45 checks passed
@sebastiancorrea81
sebastiancorrea81 deleted the fix/auto-merge-pull-request-trigger branch July 30, 2026 13:07
sebastiancorrea81 added a commit that referenced this pull request Jul 30, 2026
🤖 I have created a release *beep* *boop*
---


##
[6.7.2](v6.7.1...v6.7.2)
(2026-07-30)


### Bug Fixes

* allow gateway egress to additional VPC CIDR blocks
([#452](#452))
([c6a9f3a](c6a9f3a))
* use pull_request trigger with per-PR concurrency instead of
workflow_run
([#455](#455))
([6475fb6](6475fb6))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants