Skip to content

Commit 9fc918d

Browse files
committed
restructure release docs team roles
1 parent fdd1bb9 commit 9fc918d

File tree

4 files changed

+378
-7
lines changed

4 files changed

+378
-7
lines changed

docs/release/release-team.md

Lines changed: 13 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
# Cluster API Release Team
2+
13
<!-- START doctoc generated TOC please keep comment here to allow auto update -->
24
<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
35

@@ -19,8 +21,6 @@
1921

2022
<!-- END doctoc generated TOC please keep comment here to allow auto update -->
2123

22-
# Cluster API Release Team
23-
2424
## Overview
2525

2626
In the past, releasing Cluster API has been mostly ad-hoc and relied on one or more contributors to do most of the chore work necessary to prepare the release. One of the major downsides of this approach is that it is often difficult for users and providers to plan around Cluster API releases as they often have little visibility around when a release will happen.
@@ -67,11 +67,17 @@ As noted above, making changes to the CAPI release cadence is out of scope for
6767

6868
## Team Roles
6969

70-
- **Release Lead**: responsible for coordinating release activities, assembling the release team, taking ultimate accountability for all release tasks to be completed on time, and ensuring that a retrospective happens. The lead is also responsible for ensuring a successor is selected and trained for future release cycles.
71-
- **Communications/Docs/Release Notes Manager**: Responsible for communicating key dates to the community, improving release process documentation, and polishing release notes. Also responsible for ensuring the user-facing Netlify book and provider upgrade documentation are up to date.
72-
- **CI Signal/Bug Triage/Automation Manager**: Assumes the responsibility of the quality gate for the release and makes sure blocking issues and bugs are triaged and dealt with in a timely fashion. Helps improve release automation and tools.
73-
- **Team member**: Any Release Team lead or manager may select one or more additional members to help with their tasks. These team members will help fulfill future Release Team staffing requirements and continue to grow the CAPI community in general.
74-
*Note*: This is also documented in [Release tasks](./release-tasks.md) together with a mapping to specific tasks.
70+
**Notes**:
71+
72+
* The examples in these documents are based on the v1.6 release cycle.
73+
* The documents focus on tasks that are done for every release. One-time improvement tasks are out of scope.
74+
* If a task is prefixed with `[Track]` it means it should be ensured that this task is done, but the folks with the corresponding role are not responsible to do it themselves.
75+
76+
| Role | Handbook |
77+
|---|---|
78+
| Release Lead | [Lead Handbook](role-handbooks/release-lead/README.md) |
79+
| CI Signal | [CI Signal Handbook](role-handbooks/ci-signal/README.md) |
80+
| Communications | [Communications Handbook](role-handbooks/communications/README.md) |
7581

7682
## Team repo permissions
7783
- Release notes (`CHANGELOG` folder)
Lines changed: 95 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,95 @@
1+
# CI Signal/Bug Triage/Automation Manager
2+
3+
## Overview
4+
5+
<!-- START doctoc generated TOC please keep comment here to allow auto update -->
6+
<!-- DON'T EDIT THIS SECTION, INSTEAD RE-RUN doctoc TO UPDATE -->
7+
8+
- [Responsibilities](#responsibilities)
9+
- [Tasks](#tasks)
10+
- [Setup jobs and dashboards for a new release branch](#setup-jobs-and-dashboards-for-a-new-release-branch)
11+
- [[Continuously] Monitor CI signal](#continuously-monitor-ci-signal)
12+
- [[Continuously] Reduce the amount of flaky tests](#continuously-reduce-the-amount-of-flaky-tests)
13+
- [[Continuously] Bug triage](#continuously-bug-triage)
14+
15+
<!-- END doctoc generated TOC please keep comment here to allow auto update -->
16+
17+
## Responsibilities
18+
19+
* Signal:
20+
* Responsibility for the quality of the release
21+
* Continuously monitor CI signal, so a release can be cut at any time
22+
* Add CI signal for new release branches
23+
* Bug Triage:
24+
* Make sure blocking issues and bugs are triaged and dealt with in a timely fashion
25+
* Automation:
26+
* Maintain and improve release automation, tooling & related developer docs
27+
28+
## Tasks
29+
30+
### Setup jobs and dashboards for a new release branch
31+
32+
The goal of this task is to have test coverage for the new release branch and results in testgrid.
33+
While we add test coverage for the new release branch we will also drop the tests for old release branches if necessary.
34+
35+
1. Create new jobs based on the jobs running against our `main` branch:
36+
1. Copy the `main` branch entry as `release-1.6` in the `cluster-api-prowjob-gen.yaml` file in [test-infra](https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes-sigs/cluster-api/).
37+
2. Modify the following at the `release-1.6` branch entry:
38+
* Change intervals (let's use the same as for `release-1.5`).
39+
2. Create a new dashboard for the new branch in: `test-infra/config/testgrids/kubernetes/sig-cluster-lifecycle/config.yaml` (`dashboard_groups` and `dashboards`).
40+
3. Remove old release branches and unused versions from the `cluster-api-prowjob-gen.yaml` file in [test-infra](https://github.com/kubernetes/test-infra/blob/master/config/jobs/kubernetes-sigs/cluster-api/) according to our policy documented in [Support and guarantees](../../CONTRIBUTING.md#support-and-guarantees). For example, let's assume we just added `release-1.6`, then we can now drop test coverage for the `release-1.3` branch.
41+
4. Regenerate the prowjob configuration running `make generate-test-infra-prowjobs` command from cluster-api repository. Before running this command, ensure to export the `TEST_INFRA_DIR` variable, specifying the location of the [test-infra](https://github.com/kubernetes/test-infra/) repository in your environment. For further information, refer to this [link](https://github.com/kubernetes-sigs/cluster-api/pull/9937).
42+
43+
```sh
44+
TEST_INFRA_DIR=../../k8s.io/test-infra make generate-test-infra-prowjobs
45+
```
46+
5. Verify the jobs and dashboards a day later by taking a look at: `https://testgrid.k8s.io/sig-cluster-lifecycle-cluster-api-1.6`
47+
6. Update `.github/workflows/weekly-security-scan.yaml` - to setup Trivy and govulncheck scanning - `.github/workflows/weekly-md-link-check.yaml` - to setup link checking in the CAPI book - and `.github/workflows/weekly-test-release.yaml` - to verify the release target is working - for the currently supported branches.
48+
7. Update the [PR markdown link checker](https://github.com/kubernetes-sigs/cluster-api/blob/main/.github/workflows/pr-md-link-check.yaml) accordingly (e.g. `main` -> `release-1.6`).
49+
<br>Prior art: [Update branch for link checker](https://github.com/kubernetes-sigs/cluster-api/pull/9206)
50+
51+
52+
Prior art:
53+
54+
* [Add jobs for CAPI release 1.6](https://github.com/kubernetes/test-infra/pull/31208)
55+
56+
### [Continuously] Monitor CI signal
57+
58+
The goal of this task is to keep our tests running in CI stable.
59+
60+
**Note**: To be very clear, this is not meant to be an on-call role for Cluster API tests.
61+
62+
1. Add yourself to the [Cluster API alert mailing list](https://github.com/kubernetes/k8s.io/blob/151899b2de933e58a4dfd1bfc2c133ce5a8bbe22/groups/sig-cluster-lifecycle/groups.yaml#L20-L35)
63+
<br\>**Note**: An alternative to the alert mailing list is manually monitoring the [testgrid dashboards](https://testgrid.k8s.io/sig-cluster-lifecycle-cluster-api)
64+
(also dashboards of previous releases). Using the alert mailing list has proven to be a lot less effort though.
65+
2. Subscribe to `CI Activity` notifications for the Cluster API repo.
66+
3. Check the existing **failing-test** and **flaking-test** issue templates under `.github/ISSUE_TEMPLATE/` folder of the repo, used to create an issue for failing or flaking tests respectively. Please make sure they are up-to-date and if not, send a PR to update or improve them.
67+
4. Check if there are any existing jobs that got stuck (have been running for more than 12 hours) in a ['pending'](https://prow.k8s.io/?repo=kubernetes-sigs%2Fcluster-api&state=pending) state:
68+
- If that is the case, notify the maintainers and ask them to manually cancel and re-run the stuck jobs.
69+
5. Triage CI failures reported by mail alerts or found by monitoring the testgrid dashboards:
70+
1. Create an issue using an appropriate template (failing-test) in the Cluster API repository to surface the CI failure.
71+
2. Identify if the issue is a known issue, new issue or a regression.
72+
3. Mark the issue as `release-blocking` if applicable.
73+
6. Triage periodic GitHub actions failures, with special attention to image scan results;
74+
Eventually open issues as described above.
75+
7. Run periodic deep-dive sessions with the CI team to investigate failing and flaking tests. Example session recording: https://www.youtube.com/watch?v=YApWftmiDTg
76+
77+
#### [Continuously] Reduce the amount of flaky tests
78+
79+
The Cluster API tests are pretty stable, but there are still some flaky tests from time to time.
80+
81+
To reduce the amount of flakes please periodically:
82+
83+
1. Take a look at recent CI failures via `k8s-triage`:
84+
* [main: e2e, e2e-mink8s, test, test-mink8s](https://storage.googleapis.com/k8s-triage/index.html?job=.*cluster-api.*(test%7Ce2e)-(mink8s-)*main&xjob=.*-provider-.*)
85+
2. Open issues using an appropriate template (flaking-test) for occurring flakes and ideally fix them or find someone who can.
86+
**Note**: Given resource limitations in the Prow cluster it might not be possible to fix all flakes.
87+
Let's just try to pragmatically keep the amount of flakes pretty low.
88+
89+
### [Continuously] Bug triage
90+
91+
The goal of bug triage is to triage incoming issues and if necessary flag them with `release-blocking`
92+
and add them to the milestone of the current release.
93+
94+
We probably have to figure out some details about the overlap between the bug triage task here, release leads
95+
and Cluster API maintainers.

0 commit comments

Comments
 (0)