Skip to content

Commit 813909d

Browse files
authored
Merge pull request #569 from qiuming-best/adhere-server
2 parents dde9531 + c7ceea8 commit 813909d

File tree

4 files changed

+44
-4
lines changed

4 files changed

+44
-4
lines changed

.github/PULL_REQUEST_TEMPLATE.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,6 @@
33
#### Checklist
44
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]
55
- [ ] [DCO](https://github.com/helm/charts/blob/master/CONTRIBUTING.md#sign-your-work) signed
6-
- [ ] Chart Version bumped
6+
- [ ] Chart Version bumped, please refer to the [chart version instruction](../RELEASE-INSTRUCT.md#guidelines)
77
- [ ] Variables are documented in the values.yaml or README.md
88
- [ ] Title of the PR starts with chart name (e.g. `[velero]`)

.github/workflows/lint-test.yaml

+3-3
Original file line numberDiff line numberDiff line change
@@ -42,14 +42,14 @@ jobs:
4242
- name: Run chart-testing (list-changed)
4343
id: list-changed
4444
run: |
45-
changed=$(ct list-changed --target-branch ${{ github.event.repository.default_branch }})
45+
changed=$(ct list-changed --target-branch ${{ github.base_ref }})
4646
if [[ -n "$changed" ]]; then
4747
echo "changed=true" >> "$GITHUB_OUTPUT"
4848
fi
4949
5050
- name: Run chart-testing (lint)
5151
if: steps.list-changed.outputs.changed == 'true'
52-
run: ct lint --target-branch ${{ github.event.repository.default_branch }}
52+
run: ct lint --target-branch ${{ github.base_ref }}
5353

5454
- name: Create kind cluster
5555
uses: helm/[email protected]
@@ -60,4 +60,4 @@ jobs:
6060

6161
- name: Run chart-testing (install+upgrade)
6262
if: steps.list-changed.outputs.changed == 'true'
63-
run: ct install --upgrade --target-branch ${{ github.event.repository.default_branch }}
63+
run: ct install --upgrade --target-branch ${{ github.base_ref }}

.github/workflows/release.yml

+1
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@ on:
44
push:
55
branches:
66
- main
7+
- velero-helm-charts-v*
78

89
permissions:
910
contents: write

RELEASE-INSTRUCT.md

+39
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
Here is an instruction about the Velero helm charts release.
2+
3+
It involves Helm charts version control and branch control.
4+
5+
### Helm Charts Version Control
6+
7+
Our Helm charts are released under two scenarios:
8+
- **Velero Release:** When Velero itself is released.
9+
10+
- **Chart Updates:** When there are updates specific to the Helm charts.
11+
12+
The version is defined in [Chart.yaml](charts/velero/Chart.yaml).
13+
14+
#### Guidelines
15+
To comply with the [Semantic Versioning (semver)](https://semver.org/#summary) rule, follow these instructions:
16+
- **Velero Release:**
17+
- For each Velero minor version release, increase the major version of the Helm charts. For example, if Velero v1.13.0 is released, the Helm charts version becomes 6.0.0; for Velero v1.14.0, it becomes 7.0.0.
18+
19+
- For each Velero patch version release, increase the minor version. For instance, for Velero v1.13.1, the Helm charts version is 6.1.0; for Velero v1.13.2, it's 6.2.0.
20+
- **Helm Charts Updates:**
21+
- For breaking changes, increase the major version.
22+
23+
- For added functionality, increase the minor version.
24+
25+
- For bug fixes, increase the patch version.
26+
27+
#### Note:
28+
Breaking changes are only allowed for the **latest Helm version**. This restriction ensures that older Helm versions do not have higher major versions than newer ones, preventing helm upgrade issues.
29+
30+
### Branch Control
31+
32+
Follow these rules for branch control:
33+
- Our main branch should always align with the latest Velero major version. For example, if Velero v1.13 is the latest major version, any updates in the main branch should target Velero v1.13.
34+
35+
- Create a new branch prefixed with `velero-helm-charts-v` when Velero releases a new major version. For instance, if the main branch refers to Velero v1.13 and Velero v1.14 is released, name the new branch `velero-helm-charts-v7`. The branch version should match the major version of the Helm charts.
36+
37+
- Submit PRs for updates related to the newest Velero major version to the main branch. For updates related to older Velero major versions, submit PRs to the corresponding Helm charts branch (e.g., PRs for Velero v1.13.4 go to the `velero-helm-charts-v6` branch if it exists).
38+
39+
These guidelines ensure consistency and version control for our Helm charts releases.

0 commit comments

Comments
 (0)