Commit cff04ab
authored
Remove legacy tag-triggered release workflow (#775)
NO_CHANGELOG=true
## Summary
The secure release pipeline in
[`databricks/secure-public-registry-releases-eng`](https://github.com/databricks/secure-public-registry-releases-eng)
now owns the Maven Central and GitHub release for this repo. It
dispatches `tagging.yml` (to create the tag) and `package.yml` (to build
+ sign artifacts), then does the actual publish + GH release from the
secure side.
The previous in-repo tag-triggered path in `release.yml` is the legacy
entry point that the secure pipeline has replaced. This PR removes it
and keeps only the release-build dry-run that runs on every branch push.
## Per-file decisions
- `.github/workflows/release.yml` -- **edited and renamed** to
`.github/workflows/release-build-check.yml` (top-level `name: Release
Build Check`).
- Removed: the `push.tags: ['v*']` trigger.
- Removed: the tag-gated steps (`if: startsWith(github.ref,
'refs/tags/')`) -- `Publish to Maven Central Repository (Deploy)`,
`Write release notes to file`, `Create GitHub release`.
- Removed: the `!startsWith(...)` guard on the dry-run step (no longer
needed now that the tag trigger is gone).
- Preserved: the branch-push trigger and the `Run Release Dry-Run
(Verify)` step, which is what "verify the release build still works"
means in this repo.
- `.github/workflows/debug-prelease.yml` -- **not present in the repo**,
nothing to do.
- `.github/workflows/tagging.yml` -- unchanged (dispatched by the secure
release pipeline).
- `.github/workflows/package.yml` -- unchanged (dispatched by the secure
release pipeline).
- All other workflows (`push.yml`, `conftest.yml`,
`integration-tests.yml`, `external-message.yml`, `next-changelog.yml`)
-- unchanged; they are PR / `merge_group` CI or unrelated automation.
## Heads up: required-check rename
The job name inside the renamed workflow is now constant (`Run Release
Dry-Run`) instead of the previous dynamic name (`${{
startsWith(github.ref, 'refs/tags/') && 'Publish Release' || 'Run
Release Dry-Run' }}`). The dry-run side produced the same `Run Release
Dry-Run` string before, so if anything on the branch side was using that
exact name as a required status check it should continue to match. If
branch protection was referencing the workflow file by path
(`release.yml`), that reference will need to be updated to
`release-build-check.yml`.
## Test plan
- [ ] Confirm this PR's own CI (`build` from `push.yml` on
`pull_request`) runs green.
- [ ] On merge to `main`, confirm the renamed workflow fires on the push
and the `Run Release Dry-Run` job succeeds.
- [ ] Confirm no tag-push events target this workflow any longer.
- [ ] Update any branch-protection required-check references from
`release.yml` to `release-build-check.yml` if they existed.1 parent 9f177da commit cff04ab
1 file changed
Lines changed: 7 additions & 40 deletions
Lines changed: 7 additions & 40 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
2 | 2 | | |
3 | 3 | | |
4 | 4 | | |
5 | | - | |
6 | | - | |
7 | | - | |
8 | | - | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
9 | 10 | | |
10 | 11 | | |
11 | 12 | | |
| |||
15 | 16 | | |
16 | 17 | | |
17 | 18 | | |
18 | | - | |
19 | | - | |
| 19 | + | |
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| |||
82 | 82 | | |
83 | 83 | | |
84 | 84 | | |
85 | | - | |
86 | 85 | | |
87 | | - | |
88 | 86 | | |
89 | 87 | | |
90 | 88 | | |
91 | 89 | | |
92 | 90 | | |
93 | | - | |
94 | | - | |
95 | | - | |
96 | | - | |
97 | | - | |
98 | | - | |
99 | | - | |
100 | | - | |
101 | | - | |
102 | | - | |
103 | | - | |
104 | | - | |
105 | | - | |
106 | | - | |
107 | | - | |
108 | | - | |
109 | | - | |
110 | | - | |
111 | | - | |
112 | | - | |
113 | | - | |
114 | | - | |
115 | | - | |
116 | | - | |
117 | | - | |
118 | | - | |
119 | | - | |
120 | | - | |
121 | | - | |
122 | | - | |
123 | | - | |
0 commit comments