Skip to content

ci: convert release builder to workflow_dispatch (v0.35)#4073

Merged
sydorovdmytro merged 2 commits into
v0.35from
dmytrosydorov/devops-1050-release-dispatch-v0.35
Jul 21, 2026
Merged

ci: convert release builder to workflow_dispatch (v0.35)#4073
sydorovdmytro merged 2 commits into
v0.35from
dmytrosydorov/devops-1050-release-dispatch-v0.35

Conversation

@sydorovdmytro

@sydorovdmytro sydorovdmytro commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

Part of the DEVOPS-1050 phase-4 fan-out (loft-sh/vcluster legacy line). Builder-only conversion.

What

Converts only the builder release.yaml:

  • on: release:[created]on: workflow_dispatch (release-as-output: goreleaser creates the release at the end of a green build, dispatched by the shared vcluster-release action via --ref <tag>).
  • Decoupled github.event.release.*github.ref_name.
  • Branch-local previous_tag (git describe --match 'v*' <sha>^).

Validators intentionally unchanged

Legacy lines get no post-build validation wired — this matches current behavior (no regression). workflow_run only ever fires from the default branch, so a legacy-branch validator trigger would never fire anyway. Release validation for new lines (>= v0.37) is handled by main's validators, gated to skip legacy tags (reference PR #4072).

Actionlint-clean. Draft until the GitHub Actions incident clears and CI can run.


Note

Medium Risk
Touches the production release entrypoint and goreleaser tag inputs; behavior now depends on dispatch with the correct tag ref instead of the release event.

Overview
Release builder no longer runs on release: created; it is started only via workflow_dispatch (expected from the shared vcluster-release action with --ref <tag>), so the GitHub Release from goreleaser is an output and cannot re-trigger the workflow.

Checkout, minimum-version checks, and failure notifications now use github.ref_name instead of github.event.release.tag_name.

previous_tag for goreleaser is computed from the tag on the commit’s parent on the current branch (git describe … "${GITHUB_SHA}^"), replacing the repo-wide “second newest tag” logic so release-branch builds get the correct prior version.

Reviewed by Cursor Bugbot for commit c519466. Bugbot is set up for automated code reviews on this repo. Configure here.

Convert the release builder to workflow_dispatch (the GitHub Release is now a
pipeline output created by goreleaser, not a trigger) and drive e2e, unit-test
and ginkgo validation off workflow_run of the Release builder completing. On the
release path change detection is skipped, jobs run only for a green build, and
the built tag is checked out. Previous-tag derivation is made branch-local.

Closes DEVOPS-1050
Comment thread .github/workflows/e2e-ginkgo.yaml Fixed
@github-actions

github-actions Bot commented Jul 13, 2026

Copy link
Copy Markdown

E2E Ginkgo Tests

Status Commit Run
Passed c5194660b112ff25c1af92943ece2c8d8d1c0b65 View run #29255331970

Legacy lines keep their existing validators untouched: release validation is
handled by main's workflow_run validators, gated to new (>= v0.37) lines. This
branch only converts the builder release.yaml (release-as-output + dispatch).
@sydorovdmytro sydorovdmytro changed the title ci: release dispatch + workflow_run validation (v0.35) [DEVOPS-1050] ci: release dispatch + workflow_run validation (v0.35) Jul 13, 2026
@sydorovdmytro sydorovdmytro changed the title ci: release dispatch + workflow_run validation (v0.35) ci: convert release builder to workflow_dispatch (v0.35) Jul 14, 2026
@sydorovdmytro
sydorovdmytro marked this pull request as ready for review July 21, 2026 14:00
@sydorovdmytro
sydorovdmytro requested review from a team as code owners July 21, 2026 14:00

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes using high effort and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit c519466. Configure here.

Comment thread .github/workflows/release.yaml
@sydorovdmytro
sydorovdmytro merged commit 00ff495 into v0.35 Jul 21, 2026
39 of 43 checks passed
@sydorovdmytro
sydorovdmytro deleted the dmytrosydorov/devops-1050-release-dispatch-v0.35 branch July 21, 2026 14:13
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.

3 participants