Skip to content

[Backport 8.1.x] release: Hand "Update main" PRs over to the releaseworker - #493

Merged
michaellzc merged 1 commit into
8.1.xfrom
backport-492-to-8.1.x
Oct 21, 2026
Merged

michaellzc merged 1 commit into
8.1.xfrom
backport-492-to-8.1.x

Conversation

@michaellzc

Copy link
Copy Markdown
Member

Backports #492 so 8.1.x patch releases use the same releaseworker-owned "Update main" flow as main. This is an unchanged cherry-pick: remove the old finalize PR creation and exclude update-main branches from promote steps.

Requires sourcegraph/sourcegraph#17347 to be deployed before the next release. Companion backports are deploy-sourcegraph-helm#974 and the deploy-sourcegraph-docker backport of #1443.

Promote finalize opens a draft "Update main: build vX" PR after every
release, but none has ever been merged, so `main` here lags many
releases behind. Its Buildkite build also always fails:
`promote/release-vX-update-main` matches the `^promote\/release-.*`
release steps, and `(promote) Release: test` stops with "did not find
serialized config" (for example, [deploy-sourcegraph-k8s build
1187](https://buildkite.com/sourcegraph/deploy-sourcegraph-k8s/builds/1187)).

The releaseworker now opens the "Update main" PR itself after the
promote build passes, with squash auto-merge enabled and the Platform
team requested as reviewers
([sourcegraph/sourcegraph#17347](sourcegraph/sourcegraph#17347)).
This PR:

- removes the "Update main" steps from `promoteToPublic.finalize` in
`release.yaml`, so the releaseworker and Buildkite don't both create it;
- skips the `(promote) Release: test` and `(promote) Release: finalize`
steps on `*-update-main` branches in `.buildkite/pipeline.yaml`.

Ship this together with sourcegraph/sourcegraph#17347. Until both are
live, a release either gets no "Update main" PR or a duplicate attempt
that the releaseworker logs and skips.

Part of
[PLAT-881](https://linear.app/sourcegraph/issue/PLAT-881/close-release-generated-prs-when-release-is-complete).
Matching changes:
[docker#1443](sourcegraph/deploy-sourcegraph-docker#1443),
[helm#972](sourcegraph/deploy-sourcegraph-helm#972).

## Checklist

- [ ] Update CHANGELOG.md: not needed, release tooling only
- [ ] Update K8s Upgrade notes: not needed
- [ ] Kustomize-specific changes: none
- [x] Update sister repository:
[deploy-sourcegraph-helm](https://github.com/sourcegraph/deploy-sourcegraph-helm)
(see below)
- [x] Update sister repository:
[deploy-sourcegraph-docker](https://github.com/sourcegraph/deploy-sourcegraph-docker)
(see below)
- [x] Verify all images have a valid tag and SHA256 sum (no image
changes)

### Test plan

Both YAML files parse, and `promoteToPublic.finalize` now contains only
the tag and release registry steps. The pipeline conditions still match
`promote/release-v8.1.0` and no longer match
`promote/release-v8.1.0-update-main`. The next release's promote build
and "Update main" PR will exercise this end to end.

(cherry picked from commit bfa9a36)
@github-actions

github-actions Bot commented Oct 21, 2026 •

Copy link
Copy Markdown

@michaellzc
michaellzc enabled auto-merge (squash) October 21, 2026 17:38
@michaellzc
michaellzc requested a review from DaedalusG October 21, 2026 17:39
@michaellzc
michaellzc merged commit 5db4559 into 8.1.x Oct 21, 2026
4 checks passed
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