Skip to content

Commit ff14611

Browse files
committed
.github/ISSUE_TEMPLATE: replace templates with new instructions
The new instructions have been used for a couple releases already so we can safely use them and remove the old instructions. Signed-off-by: André Martins <[email protected]>
1 parent 19fafa9 commit ff14611

File tree

3 files changed

+106
-177
lines changed

3 files changed

+106
-177
lines changed

.github/ISSUE_TEMPLATE/release_template_patch.md

Lines changed: 92 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
name: Release a new patch version of Cilium from a stable branch
2+
name: NEW Release a new patch version of Cilium from a stable branch
33
about: Create a checklist for an upcoming release
44
title: 'vX.Y.Z release'
55
labels: kind/release
@@ -10,75 +10,79 @@ assignees: ''
1010
## Setup preparation
1111

1212
- [ ] Depending on your OS, make sure Docker is running
13-
- [ ] Export a [`GITHUB_TOKEN`](https://github.com/settings/tokens/new?description=Cilium%20Release%20Script&scopes=write:org,public_repo) that has access to the repository. Only classic tokens are
14-
[supported at the moment][GitHub PAT tracker], the needed scope is `public_repo`.
15-
- [ ] Make sure a setup (GPG, SSH, S/MIME) is in place for [signing tags] with Git and install [Hub](https://github.com/github/hub).
16-
- [ ] Make sure the `GOPATH` environment variable is set and pointing to the relevant path
13+
- [ ] Export a [`GITHUB_TOKEN`](https://github.com/settings/tokens/new?description=Cilium%20Release%20Script&scopes=write:org,public_repo,project) that has access to the repository. Only classic tokens are
14+
[supported at the moment][GitHub PAT tracker], the needed scopes are `write:org`, `public_repo` and `project`.
15+
- [ ] Make sure a setup (GPG, SSH, S/MIME) is in place for [signing tags]
16+
with Git and install [gh](https://cli.github.com).
1717
- [ ] Make sure the [Cilium helm charts][Cilium charts] and [release][Cilium release-notes tool] repositories are installed locally:
1818
- [ ] Run `git clone https://github.com/cilium/charts.git "$GOPATH/src/github.com/cilium/charts"`
1919
- [ ] Run `git clone https://github.com/cilium/release.git "$GOPATH/src/github.com/cilium/release"`
2020
- [ ] If you already have the repo checked out, make sure the `release` binary is up to date:
2121

2222
git checkout master && git pull && make
23+
- [ ] Read the documentation of `release start --help` tool to understand what
24+
each automated step does.
2325

24-
## Pre-release
26+
## Pre-check (run ~1 week before release date)
2527

2628
- [ ] When you create a GitHub issue using this issue template, GitHub Slack app posts a
2729
message in #launchpad Slack channel. Create a thread for that message and ping the
2830
current backporter to merge the outstanding [backport PRs] and stop merging any new
2931
backport PRs until the GitHub issue is closed (to avoid generating incomplete
3032
release notes).
31-
- [ ] Check that there are no [release blockers] for the targeted release version.
32-
- [ ] Ensure that outstanding [backport PRs] are merged (these may be skipped on
33-
case by case basis in coordination with the backporter).
34-
- [ ] Check with @cilium/security team if there are any security fixes to include
35-
in the release.
36-
- [ ] Push a PR to cilium/cilium including the changes necessary for the new release:
37-
- [ ] Change directory to the local copy of cilium/cilium repository and pull latest
38-
changes from the branch being released
39-
- [ ] Run `../release/internal/start-release.sh X.Y.Z`. You can ignore
40-
warnings about commits with no related PR found.
41-
Note that this script produces some files at the root of the Cilium
42-
repository, and that these files are required at a later step for
43-
tagging the release. Do not commit them.
44-
- [ ] Commit all changed files with title `Prepare for release vX.Y.Z`. New
45-
generated files, such as release-state.json and vX.Y.Z-changes.txt
46-
should not be committed. Tip: use `git add -p` to review the changes and
47-
compare them with the previous release PR.
48-
- [ ] Submit PR (`../release/internal/submit-release.sh`). Note that only the smoke tests
49-
need to succeed in order to merge this PR. Full e2e test runs are not required.
33+
- [ ] Run `./release start --steps 1-pre-check --target-version vX.Y.Z`
34+
- [ ] Check that there are no [release blockers] for the targeted release
35+
version.
36+
- [ ] Ensure that outstanding [backport PRs] are merged (these may be
37+
skipped on case by case basis in coordination with the backporter).
38+
- [ ] Check with @cilium/security team in case there are any CVEs found in the
39+
docker image.
40+
- [ ] Check with @cilium/security team if there are any security fixes to
41+
include in the release.
42+
43+
## Preparation PR (run ~1 day before release date. It can be re-run multiple times.)
44+
45+
- [ ] Go to [release workflow] and Run the workflow from "Branch: main", for
46+
step "2-prepare-release" and version for vX.Y.Z
47+
- [ ] Check if the workflow was successful and check the PR opened by the
48+
Release bot.
5049
- [ ] Merge PR
50+
51+
## Tagging
52+
5153
- [ ] Ask a maintainer if there are any known issues that should hold up the release
5254
- [ ] FYI, do not wait too much time between a tag is created and the helm charts are published.
5355
Once the tags are published the documentation will be pointing to them. Until we release
5456
the helm chart, users will face issues while trying out our documentation.
55-
- [ ] Create and push *both* tags to GitHub (`vX.Y.Z`, `X.Y.Z`)
56-
- [ ] Pull latest `upstream/vX.Y` branch locally
57-
- [ ] Run `../release/internal/tag-release.sh`.
57+
- [ ] Run `./release start --steps 3-tag --target-version vX.Y.Z`
5858
- [ ] Ask a maintainer to approve the build in the following link (keep the URL
5959
of the GitHub run to be used later):
6060
[Cilium Image Release builds](https://github.com/cilium/cilium/actions?query=workflow:%22Image+Release+Build%22)
61-
- [ ] Check if all docker images are available before announcing the release:
62-
`make -C install/kubernetes/ check-docker-images`
63-
- [ ] Get the image digests from the build process and make a commit and PR with
64-
these digests.
65-
- [ ] Run `../release/internal/post-release.sh URL` to fetch the image
66-
digests and submit a PR to update these, use the `URL` of the GitHub
67-
run here
68-
- [ ] Get someone to review the PR. Do not trigger the full CI suite, but
69-
wait for the automatic checks to complete. Merge the PR.
70-
- [ ] Update helm charts
71-
- [ ] Create helm charts artifacts in [Cilium charts] repository using
72-
[cilium helm release tool] for the `vX.Y.Z` release
73-
and create a PR with these changes against the charts repository.
74-
Note: If you handle several patch releases at once,
75-
create one PR per release, to make sure that the corresponding workflow
76-
action run for each commit. Wait for your PR to be merged before
77-
creating the other ones for other patch releases, or they will
78-
conflict.
79-
- [ ] Have a maintainer review and merge your PR.
80-
- [ ] Check the output of the [chart workflow] and see if the test was
81-
successful.
61+
62+
## Post Tagging (run after docker images are published)
63+
64+
- [ ] Go to [release workflow] and Run the workflow from "Branch: main", for
65+
step "4-post-release" and version for vX.Y.Z
66+
- [ ] Check if the workflow was successful and check the PR opened by the
67+
Release bot.
68+
- [ ] Merge PR
69+
70+
## Publish helm (run after docker images are published)
71+
72+
- [ ] Update helm charts `./release start --steps 5-publish-helm --target-version vX.Y.Z`
73+
- [ ] Open [Charts Workflow] and check if the workflow run is successful.
74+
75+
## Publish docs (only for pre/rc releases)
76+
77+
- [ ] Check [read the docs] configuration:
78+
- [ ] Set a new build as active and hidden in [active versions].
79+
- [ ] Deactivate previous RCs.
80+
- [ ] Update algolia configuration search in [docsearch-scraper-webhook].
81+
- Update the versions in `docsearch.config.json`, commit them and push a
82+
trigger the workflow [here](https://github.com/cilium/docsearch-scraper-webhook/actions/workflows/update-algolia-index.yaml)
83+
84+
## Post-release
85+
8286
- [ ] Check draft release from [releases] page
8387
- [ ] Update the text at the top with 2-3 highlights of the release
8488
- [ ] Check with @cilium/security if the release addresses any open security
@@ -90,18 +94,54 @@ assignees: ''
9094
_latest_ release displayed on GitHub (e.g. 1.11.13 does not become the
9195
new latest release over 1.12.5, but version 1.12.6 will).
9296
- [ ] Publish the release
93-
- [ ] Announce the release in #general on Slack (do not use [@]channel)
97+
- [ ] Announce the release in #general on Slack (do not use [@]channel).
98+
See below for templates.
99+
- [ ] Prepare post-release changes to main branch using `../release/internal/bump-readme.sh`.
94100

95-
## Post-release
101+
---
96102

97-
- [ ] Prepare post-release changes to main branch using `../release/internal/bump-readme.sh`.
103+
## Slack example text templates
104+
105+
### Patch releases
106+
107+
```
108+
:confetti_ball: :cilium-radiant: Release Announcement :cilium-radiant::confetti_ball:
109+
110+
Cilium vX.Y.Z, vA.B.C, and vD.E.F have been released. Thanks all for your contributions! Please see the release notes below for details :cilium-gopher:
111+
112+
vX.Y.Z: https://github.com/cilium/cilium/releases/tag/vX.Y.Z
113+
vA.B.C: https://github.com/cilium/cilium/releases/tag/vA.B.C
114+
vD.E.F: https://github.com/cilium/cilium/releases/tag/vD.E.F
115+
```
116+
117+
### First pre-release
118+
119+
```
120+
:cilium-new: *Cilium vX.Y.Z-rc.W has been released:*
121+
https://github.com/cilium/cilium/releases/tag/vX.Y.Z-rc.W
122+
123+
This is the first monthly snapshot for the vX.Y development cycle. There are [vX.Y.Z-rc.W OSS docs](https://docs.cilium.io/en/vX.Y.Z-rc.W) available if you want to pull this version & try it out.
124+
```
125+
126+
### Subsequent pre-/rc- releases
127+
128+
```
129+
*Announcement* :tada: :tada:
130+
131+
:cilium-new: *Cilium vX.Y.Z-rc.W has been released:*
132+
https://github.com/cilium/cilium/releases/tag/vX.Y.Z-rc.W
133+
134+
Thank you for the testing and contributing to the previous pre-releases. There are [vX.Y.Z-rc.W OSS docs](https://docs.cilium.io/en/vX.Y.Z-rc.W) available if you want to pull this version & try it out.
135+
```
98136

137+
[release workflow]: https://github.com/cilium/cilium/actions/workflows/release.yaml
99138
[GitHub PAT tracker]: https://github.com/orgs/community/discussions/36441
100139
[signing tags]: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-tags
101140
[release blockers]: https://github.com/cilium/cilium/labels/release-blocker%2FX.Y
102141
[backport PRs]: https://github.com/cilium/cilium/pulls?q=is%3Aopen+is%3Apr+draft%3Afalse+label%3Abackport%2FX.Y
103142
[Cilium release-notes tool]: https://github.com/cilium/release
104143
[Cilium charts]: https://github.com/cilium/charts
144+
[Charts Workflow]: https://github.com/cilium/charts/actions/workflows/validate-cilium-chart.yaml
105145
[releases]: https://github.com/cilium/cilium/releases
106146
[cilium helm release tool]: https://github.com/cilium/charts/blob/master/RELEASE.md
107147
[cilium-runtime images]: https://quay.io/repository/cilium/cilium-runtime

.github/ISSUE_TEMPLATE/release_template_patch_new.md renamed to .github/ISSUE_TEMPLATE/release_template_pre_main.md

Lines changed: 14 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
---
2-
name: NEW Release a new patch version of Cilium from a stable branch
2+
name: Release a new pre-release version of Cilium from the main branch
33
about: Create a checklist for an upcoming release
4-
title: 'vX.Y.Z release'
4+
title: 'vX.Y.Z-pre.W release'
55
labels: kind/release
66
assignees: ''
77

@@ -30,7 +30,7 @@ assignees: ''
3030
current backporter to merge the outstanding [backport PRs] and stop merging any new
3131
backport PRs until the GitHub issue is closed (to avoid generating incomplete
3232
release notes).
33-
- [ ] Run `./release start --steps 1-pre-check --target-version vX.Y.Z`
33+
- [ ] Run `./release start --steps 1-pre-check --target-version vX.Y.Z-pre.W`
3434
- [ ] Check that there are no [release blockers] for the targeted release
3535
version.
3636
- [ ] Ensure that outstanding [backport PRs] are merged (these may be
@@ -43,7 +43,7 @@ assignees: ''
4343
## Preparation PR (run ~1 day before release date. It can be re-run multiple times.)
4444

4545
- [ ] Go to [release workflow] and Run the workflow from "Branch: main", for
46-
step "2-prepare-release" and version for vX.Y.Z
46+
step "2-prepare-release" and version for vX.Y.Z-pre.W
4747
- [ ] Check if the workflow was successful and check the PR opened by the
4848
Release bot.
4949
- [ ] Merge PR
@@ -54,22 +54,14 @@ assignees: ''
5454
- [ ] FYI, do not wait too much time between a tag is created and the helm charts are published.
5555
Once the tags are published the documentation will be pointing to them. Until we release
5656
the helm chart, users will face issues while trying out our documentation.
57-
- [ ] Run `./release start --steps 3-tag --target-version vX.Y.Z`
57+
- [ ] Run `./release start --steps 3-tag --target-version vX.Y.Z-pre.W`
5858
- [ ] Ask a maintainer to approve the build in the following link (keep the URL
5959
of the GitHub run to be used later):
6060
[Cilium Image Release builds](https://github.com/cilium/cilium/actions?query=workflow:%22Image+Release+Build%22)
6161

62-
## Post Tagging (run after docker images are published)
63-
64-
- [ ] Go to [release workflow] and Run the workflow from "Branch: main", for
65-
step "4-post-release" and version for vX.Y.Z
66-
- [ ] Check if the workflow was successful and check the PR opened by the
67-
Release bot.
68-
- [ ] Merge PR
69-
7062
## Publish helm (run after docker images are published)
7163

72-
- [ ] Update helm charts `./release start --steps 5-publish-helm --target-version vX.Y.Z`
64+
- [ ] Update helm charts `./release start --steps 5-publish-helm --target-version vX.Y.Z-pre.W`
7365
- [ ] Open [Charts Workflow] and check if the workflow run is successful.
7466

7567
## Publish docs (only for pre/rc releases)
@@ -107,31 +99,31 @@ assignees: ''
10799
```
108100
:confetti_ball: :cilium-radiant: Release Announcement :cilium-radiant::confetti_ball:
109101
110-
Cilium vX.Y.Z, vA.B.C, and vD.E.F have been released. Thanks all for your contributions! Please see the release notes below for details :cilium-gopher:
102+
Cilium vX.Y.Z-pre.W, vA.B.C, and vD.E.F have been released. Thanks all for your contributions! Please see the release notes below for details :cilium-gopher:
111103
112-
vX.Y.Z: https://github.com/cilium/cilium/releases/tag/vX.Y.Z
104+
vX.Y.Z-pre.W: https://github.com/cilium/cilium/releases/tag/vX.Y.Z-pre.W
113105
vA.B.C: https://github.com/cilium/cilium/releases/tag/vA.B.C
114106
vD.E.F: https://github.com/cilium/cilium/releases/tag/vD.E.F
115107
```
116108

117109
### First pre-release
118110

119111
```
120-
:cilium-new: *Cilium vX.Y.Z-rc.W has been released:*
121-
https://github.com/cilium/cilium/releases/tag/vX.Y.Z-rc.W
112+
:cilium-new: *Cilium vX.Y.Z-pre.W has been released:*
113+
https://github.com/cilium/cilium/releases/tag/vX.Y.Z-pre.W
122114
123-
This is the first monthly snapshot for the vX.Y development cycle. There are [vX.Y.Z-rc.W OSS docs](https://docs.cilium.io/en/vX.Y.Z-rc.W) available if you want to pull this version & try it out.
115+
This is the first monthly snapshot for the vX.Y development cycle. There are [vX.Y.Z-pre.W OSS docs](https://docs.cilium.io/en/vX.Y.Z-pre.W) available if you want to pull this version & try it out.
124116
```
125117

126118
### Subsequent pre-/rc- releases
127119

128120
```
129121
*Announcement* :tada: :tada:
130122
131-
:cilium-new: *Cilium vX.Y.Z-rc.W has been released:*
132-
https://github.com/cilium/cilium/releases/tag/vX.Y.Z-rc.W
123+
:cilium-new: *Cilium vX.Y.Z-pre.W has been released:*
124+
https://github.com/cilium/cilium/releases/tag/vX.Y.Z-pre.W
133125
134-
Thank you for the testing and contributing to the previous pre-releases. There are [vX.Y.Z-rc.W OSS docs](https://docs.cilium.io/en/vX.Y.Z-rc.W) available if you want to pull this version & try it out.
126+
Thank you for the testing and contributing to the previous pre-releases. There are [vX.Y.Z-pre.W OSS docs](https://docs.cilium.io/en/vX.Y.Z-pre.W) available if you want to pull this version & try it out.
135127
```
136128

137129
[release workflow]: https://github.com/cilium/cilium/actions/workflows/release.yaml

0 commit comments

Comments
 (0)