Skip to content

Commit 7025739

Browse files
author
Yoan Moscatelli
committed
Merge remote-tracking branch 'origin/improvement/bump-ingress-nginx-4.12.1' into w/130.0/improvement/bump-ingress-nginx-4.12.1
2 parents 767e6db + af9203e commit 7025739

File tree

57 files changed

+564
-752
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

57 files changed

+564
-752
lines changed

BUMPING.md

Lines changed: 26 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -4,26 +4,6 @@ Here is a short list of chart and component bumps and how to perform them
44

55
## Charts
66

7-
### General Outline
8-
9-
All charts are in the `charts/` directory, they are usually represented
10-
in one file and one directory:
11-
12-
- `$CHART_NAME/` contains the untouched chart files fetched using helm.
13-
- `$CHART_NAME.yaml` our personalized helm values file.
14-
15-
In order to Bump this chart, one has to:
16-
17-
- remove the current chart files:
18-
```rm -rf charts/$CHART_NAME/```
19-
- add the chart's repo using helm:
20-
```helm repo add $REPO_NAME $REPO_URL && helm repo update```
21-
- fetch the repo again:
22-
```helm fetch -d charts --untar $REPO_NAME/$CHART_NAME```
23-
- make any necessary patches to the chart (chart-specific).
24-
- generate the sls state from the chart:
25-
```./doit.sh codegen:chart_$CHART_NAME```
26-
277
### fluent-bit
288

299
```
@@ -81,9 +61,9 @@ set `$VERSION` with the appropriate value.
8161
run
8262

8363
```
84-
curl https://raw.githubusercontent.com/kubernetes/ingress-nginx/helm-chart-$VERSION/deploy/grafana/dashboards/nginx.json \
64+
curl https://raw.githubusercontent.com/kubernetes/ingress-nginx/refs/tags/controller-$VERSION/deploy/grafana/dashboards/nginx.json \
8565
-Lo salt/metalk8s/addons/nginx-ingress/deployed/files/ingress-nginx.json
86-
curl https://raw.githubusercontent.com/kubernetes/ingress-nginx/helm-chart-$VERSION/deploy/grafana/dashboards/request-handling-performance.json \
66+
curl https://raw.githubusercontent.com/kubernetes/ingress-nginx/refs/tags/controller-$VERSION/deploy/grafana/dashboards/request-handling-performance.json \
8767
-Lo salt/metalk8s/addons/nginx-ingress/deployed/files/ingress-nginx-performance.json
8868
```
8969

@@ -116,6 +96,24 @@ REPO_NAME=banzaicloud-stable
11696
REPO_URL=https://kubernetes-charts.banzaicloud.com/
11797
```
11898

99+
### General Outline
100+
101+
All charts are in the `charts/` directory, they are usually represented
102+
in one file and one directory:
103+
104+
- `$CHART_NAME/` contains the untouched chart files fetched using helm.
105+
- `$CHART_NAME.yaml` our personalized helm values file.
106+
107+
In order to Bump this chart, one has to:
108+
109+
- remove the current chart files:
110+
```rm -rf charts/$CHART_NAME/```
111+
- add the chart's repo using helm:
112+
```helm repo add $REPO_NAME $REPO_URL && helm repo update```
113+
- fetch the repo again:
114+
```helm fetch -d charts --untar $REPO_NAME/$CHART_NAME```
115+
- make any necessary patches to the chart (chart-specific).
116+
119117
## Images
120118

121119
A few tips to bump image versions and SHAs:
@@ -151,3 +149,9 @@ This guide is applied for both `metalk8s-operator` and `storage-operator`.
151149
## Containerd
152150

153151
Instructions to bump Containerd version are in its [spec file](./packages/redhat/common/containerd.spec)
152+
153+
## Update the sls state
154+
155+
- git add changes because codegen need to list them.
156+
- generate the sls state from the chart:
157+
```./doit.sh codegen:chart_$CHART_NAME```

CHANGELOG.md

Lines changed: 15 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,21 @@
3131
- Bump Calico version to [3.29.3](https://github.com/projectcalico/calico/releases/tag/v3.29.3)
3232
(PR[#4571](https://github.com/scality/metalk8s/pull/4571))
3333

34-
## Release 129.0.1 (in development)
34+
## Release 129.0.3 (in development)
35+
36+
### Enhancements
37+
38+
- Bump ingress-nginx chart version to
39+
[4.12.1](https://github.com/kubernetes/ingress-nginx/releases/tag/helm-chart-4.12.1)
40+
The controller image has been bumped accordingly to
41+
[v1.12.1](https://github.com/kubernetes/ingress-nginx/releases/tag/controller-v1.12.1)
42+
43+
- Bump nginx image to [1.27.5-alpine](https://github.com/nginx/nginx/releases/tag/release-1.27.5)
44+
(PR[#4544](https://github.com/scality/metalk8s/pull/4544))
45+
46+
## Release 129.0.2
47+
48+
## Release 129.0.1
3549

3650
### Enhancements
3751

buildchain/buildchain/versions.py

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -82,7 +82,7 @@ def load_version_information() -> None:
8282

8383
ETCD_VERSION: str = "3.5.21"
8484
ETCD_IMAGE_VERSION: str = f"{ETCD_VERSION}-0"
85-
NGINX_IMAGE_VERSION: str = "1.27.2-alpine"
85+
NGINX_IMAGE_VERSION: str = "1.27.5-alpine"
8686
NODEJS_IMAGE_VERSION: str = "20.11.1"
8787
KEEPALIVED_VERSION: str = "2.3.1"
8888
CERT_MANAGER_VERSION: str = "1.16.1"
@@ -187,8 +187,8 @@ def _version_prefix(version: str, prefix: str = "v") -> str:
187187
),
188188
Image(
189189
name="nginx-ingress-controller",
190-
version="v1.11.3",
191-
digest="sha256:d56f135b6462cfc476447cfe564b83a45e8bb7da2774963b00d12161112270b7",
190+
version="v1.12.1",
191+
digest="sha256:d2fbc4ec70d8aa2050dd91a91506e998765e86c96f32cffb56c503c9c34eed5b",
192192
),
193193
Image(
194194
name="node-exporter",

charts/ingress-nginx/Chart.yaml

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,9 @@
11
annotations:
22
artifacthub.io/changes: |
3-
- Update Ingress-Nginx version controller-v1.11.3
3+
- Update Ingress-Nginx version controller-v1.12.1
44
artifacthub.io/prerelease: "false"
55
apiVersion: v2
6-
appVersion: 1.11.3
6+
appVersion: 1.12.1
77
description: Ingress controller for Kubernetes using NGINX as a reverse proxy and
88
load balancer
99
home: https://github.com/kubernetes/ingress-nginx
@@ -15,11 +15,9 @@ kubeVersion: '>=1.21.0-0'
1515
maintainers:
1616
- name: cpanato
1717
- name: Gacko
18-
- name: puerco
19-
- name: rikatz
2018
- name: strongjz
2119
- name: tao12345666333
2220
name: ingress-nginx
2321
sources:
2422
- https://github.com/kubernetes/ingress-nginx
25-
version: 4.11.3
23+
version: 4.12.1

charts/ingress-nginx/OWNERS

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,4 @@
11
# See the OWNERS docs: https://www.kubernetes.dev/docs/guide/owners
22

3-
approvers:
4-
- ingress-nginx-helm-maintainers
5-
6-
reviewers:
7-
- ingress-nginx-helm-reviewers
8-
93
labels:
104
- area/helm

0 commit comments

Comments
 (0)