Skip to content

Commit 70cf46c

Browse files
authored
Merge pull request #445 from junior/helm-ingress-update
Helm ingress update
2 parents 97c79ca + 5531447 commit 70cf46c

File tree

14 files changed

+77
-67
lines changed

14 files changed

+77
-67
lines changed

deploy/complete/helm-chart/mushop/templates/cluster-issuers.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ spec:
1818
solvers:
1919
- http01:
2020
ingress:
21-
class: nginx
21+
ingressClassName: nginx
2222
---
2323
apiVersion: cert-manager.io/v1
2424
kind: ClusterIssuer
@@ -36,7 +36,7 @@ spec:
3636
solvers:
3737
- http01:
3838
ingress:
39-
class: nginx
39+
ingressClassName: nginx
4040
---
4141
apiVersion: cert-manager.io/v1
4242
kind: ClusterIssuer

deploy/complete/helm-chart/mushop/templates/ingress-edge.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -6,14 +6,14 @@ metadata:
66
labels:
77
{{ include "mushop.labels" . | indent 4 }}
88
annotations:
9-
kubernetes.io/ingress.class: nginx
109
{{- if .Values.ingress.tls }}
1110
nginx.ingress.kubernetes.io/ssl-redirect: "true"
1211
cert-manager.io/cluster-issuer: {{ include "mushop.fullname" . }}-{{ .Values.ingress.clusterIssuer }}
1312
cert-manager.io/acme-challenge-type: http01
1413
{{- end }}
15-
14+
1615
spec:
16+
ingressClassName: nginx
1717
rules:
1818
{{- if .Values.ingress.hosts }}
1919
{{- range $host := .Values.ingress.hosts }}

deploy/complete/helm-chart/mushop/templates/ingress-grafana.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -20,15 +20,15 @@ metadata:
2020
labels:
2121
{{ include "mushop.labels" . | indent 4 }}
2222
annotations:
23-
kubernetes.io/ingress.class: nginx
2423
nginx.ingress.kubernetes.io/rewrite-target: /$2
2524
{{- if .Values.ingress.tls }}
2625
nginx.ingress.kubernetes.io/ssl-redirect: "true"
2726
cert-manager.io/cluster-issuer: {{ include "mushop.fullname" . }}-{{ .Values.ingress.clusterIssuer }}
2827
cert-manager.io/acme-challenge-type: http01
2928
{{- end }}
30-
29+
3130
spec:
31+
ingressClassName: nginx
3232
rules:
3333
{{- if .Values.ingress.hosts }}
3434
{{- range $host := .Values.ingress.hosts }}

deploy/complete/helm-chart/setup/requirements.yaml

+6-6
Original file line numberDiff line numberDiff line change
@@ -1,31 +1,31 @@
1-
# Copyright (c) 2019-2021 Oracle and/or its affiliates. All rights reserved.
1+
# Copyright (c) 2019-2024 Oracle and/or its affiliates. All rights reserved.
22
# Licensed under the Universal Permissive License v 1.0 as shown at http://oss.oracle.com/licenses/upl.
33
#
44

55
dependencies:
66
# Prometheus
77
- name: prometheus
8-
version: 25.18.0
8+
version: 25.27.0
99
condition: prometheus.enabled
1010
repository: https://prometheus-community.github.io/helm-charts
1111
# Grafana
1212
- name: grafana
13-
version: 7.3.7
13+
version: 8.5.1
1414
condition: grafana.enabled
1515
repository: https://grafana.github.io/helm-charts
1616
# HPA Metrics
1717
- name: metrics-server
18-
version: 3.12.0
18+
version: 3.12.1
1919
condition: metrics-server.enabled
2020
repository: https://kubernetes-sigs.github.io/metrics-server
2121
# Ingress Controller
2222
- name: ingress-nginx
23-
version: 4.10.0
23+
version: 4.11.2
2424
condition: ingress-nginx.enabled
2525
repository: https://kubernetes.github.io/ingress-nginx
2626
# cert-manager
2727
- name: cert-manager
28-
version: 1.14.4
28+
version: 1.15.3
2929
condition: cert-manager.enabled
3030
repository: https://charts.jetstack.io
3131
# jenkins

deploy/complete/helm-chart/setup/values.yaml

+6-2
Original file line numberDiff line numberDiff line change
@@ -33,12 +33,16 @@ ingress-nginx:
3333
controller:
3434
metrics:
3535
enabled: true
36+
ingressClassResource:
37+
default: true
3638

3739
# https://github.com/jetstack/cert-manager/blob/master/README.md
3840
# https://artifacthub.io/packages/helm/jetstack/cert-manager
3941
cert-manager:
4042
enabled: true
41-
installCRDs: true
43+
crds:
44+
enabled: true
45+
keep: false
4246

4347
# https://github.com/grafana/helm-charts/blob/main/charts/grafana/README.md
4448
# https://artifacthub.io/packages/helm/grafana/grafana
@@ -136,10 +140,10 @@ jenkins:
136140
apiVersion: networking.k8s.io/v1
137141
metadata:
138142
annotations:
139-
kubernetes.io/ingress.class: nginx
140143
nginx.ingress.kubernetes.io/rewrite-target: /$2
141144
labels: {}
142145
spec:
146+
ingressClassName: nginx
143147
rules:
144148
- http:
145149
paths:

deploy/complete/kubernetes/ingress/mushop-dev.yaml

+1-2
Original file line numberDiff line numberDiff line change
@@ -2,9 +2,8 @@ apiVersion: extensions/v1beta1
22
kind: Ingress
33
metadata:
44
name: mushop-dev
5-
annotations:
6-
kubernetes.io/ingress.class: nginx
75
spec:
6+
ingressClassName: nginx
87
rules:
98
- http:
109
paths:

deploy/complete/kubernetes/ingress/mushop-grafana.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
1-
apiVersion: networking.k8s.io/v1beta1
1+
apiVersion: networking.k8s.io/v1
22
kind: Ingress
33
metadata:
44
name: grafana
55
annotations:
6-
kubernetes.io/ingress.class: nginx
76
nginx.ingress.kubernetes.io/ssl-redirect: "true"
87
cert-manager.io/cluster-issuer: letsencrypt-prod
98
cert-manager.io/acme-challenge-type: http01
109
spec:
10+
ingressClassName: nginx
1111
tls:
1212
- secretName: mushop-grafana-tls
1313
hosts:

deploy/complete/kubernetes/ingress/mushop-issuer.yaml

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
---
2-
apiVersion: cert-manager.io/v1alpha2
2+
apiVersion: cert-manager.io/v1
33
kind: ClusterIssuer
44
metadata:
55
name: letsencrypt-prod
@@ -15,9 +15,9 @@ spec:
1515
solvers:
1616
- http01:
1717
ingress:
18-
class: nginx
18+
ingressClassName: nginx
1919
---
20-
apiVersion: cert-manager.io/v1alpha2
20+
apiVersion: cert-manager.io/v1
2121
kind: ClusterIssuer
2222
metadata:
2323
name: letsencrypt-staging
@@ -33,9 +33,9 @@ spec:
3333
solvers:
3434
- http01:
3535
ingress:
36-
class: nginx
36+
ingressClassName: nginx
3737
---
38-
apiVersion: cert-manager.io/v1alpha2
38+
apiVersion: cert-manager.io/v1
3939
kind: ClusterIssuer
4040
metadata:
4141
name: selfsigned

deploy/complete/kubernetes/ingress/mushop-prod.yaml

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,15 +1,15 @@
1-
apiVersion: networking.k8s.io/v1beta1
1+
apiVersion: networking.k8s.io/v1
22
kind: Ingress
33
metadata:
44
name: mushop-prod
55
annotations:
6-
kubernetes.io/ingress.class: nginx
76
nginx.ingress.kubernetes.io/ssl-redirect: "true"
87
cert-manager.io/cluster-issuer: letsencrypt-prod
98
cert-manager.io/acme-challenge-type: http01
109
spec:
10+
ingressClassName: nginx
1111
tls:
12-
- secretName: mushop-prod-tls
12+
- secretName: mushop-prod-tls
1313
hosts:
1414
- mushop.ateam.cloud
1515
- insecure.mushop.ateam.cloud

deploy/complete/kubernetes/ingress/mushop-test.yaml

+2-2
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
1-
apiVersion: networking.k8s.io/v1beta1
1+
apiVersion: networking.k8s.io/v1
22
kind: Ingress
33
metadata:
44
name: mushop-test
55
annotations:
6-
kubernetes.io/ingress.class: nginx
76
nginx.ingress.kubernetes.io/ssl-redirect: "true"
87
cert-manager.io/cluster-issuer: letsencrypt-prod
98
cert-manager.io/acme-challenge-type: http01
109
spec:
10+
ingressClassName: nginx
1111
tls:
1212
- secretName: mushop-test-tls
1313
hosts:

deploy/complete/kubernetes/mushop.yaml

+1-2
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,8 @@ metadata:
1010
app.kubernetes.io/instance: mushop
1111
app.kubernetes.io/version: "1.0"
1212
app.kubernetes.io/managed-by: Helm
13-
annotations:
14-
kubernetes.io/ingress.class: nginx
1513
spec:
14+
ingressClassName: nginx
1615
rules:
1716
# change this for user-specific ingress
1817
- host: mushop.example.com

deploy/complete/terraform/VERSION

+1-1
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
3.2.0
1+
3.3.0

deploy/complete/terraform/mushop-utilities.tf

+16-6
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ resource "helm_release" "prometheus" {
1818
name = "prometheus"
1919
repository = local.helm_repository.prometheus
2020
chart = "prometheus"
21-
version = "25.18.0"
21+
version = "25.27.0"
2222
namespace = kubernetes_namespace.cluster_utilities_namespace.id
2323
wait = false
2424

@@ -37,7 +37,7 @@ resource "helm_release" "grafana" {
3737
name = "mushop-utils-grafana" # mushop-utils included to be backwards compatible to the docs and setup chart install
3838
repository = local.helm_repository.grafana
3939
chart = "grafana"
40-
version = "7.3.7"
40+
version = "8.5.1"
4141
namespace = kubernetes_namespace.cluster_utilities_namespace.id
4242
wait = false
4343

@@ -97,7 +97,7 @@ resource "helm_release" "metrics_server" {
9797
name = "metrics-server"
9898
repository = local.helm_repository.metrics_server
9999
chart = "metrics-server"
100-
version = "3.12.0"
100+
version = "3.12.1"
101101
namespace = kubernetes_namespace.cluster_utilities_namespace.id
102102
wait = false
103103

@@ -116,7 +116,7 @@ resource "helm_release" "ingress_nginx" {
116116
name = "mushop-utils-ingress-nginx" # mushop-utils included to be backwards compatible to the docs and setup chart install
117117
repository = local.helm_repository.ingress_nginx
118118
chart = "ingress-nginx"
119-
version = "4.10.0"
119+
version = "4.11.2"
120120
namespace = kubernetes_namespace.cluster_utilities_namespace.id
121121
wait = true
122122

@@ -140,6 +140,11 @@ resource "helm_release" "ingress_nginx" {
140140
type = "string"
141141
}
142142

143+
set {
144+
name = "controller.ingressClassResource.default"
145+
value = true
146+
}
147+
143148
timeout = 1800 # workaround to wait the node be active for other charts
144149

145150
depends_on = [kubernetes_deployment.cluster_autoscaler_deployment]
@@ -156,15 +161,20 @@ resource "helm_release" "cert_manager" {
156161
name = "cert-manager"
157162
repository = local.helm_repository.jetstack
158163
chart = "cert-manager"
159-
version = "1.14.4"
164+
version = "1.15.3"
160165
namespace = kubernetes_namespace.cluster_utilities_namespace.id
161166
wait = true # wait to allow the webhook be properly configured
162167

163168
set {
164-
name = "installCRDs"
169+
name = "crds.enabled"
165170
value = true
166171
}
167172

173+
set {
174+
name = "crds.keep"
175+
value = false
176+
}
177+
168178
set {
169179
name = "webhook.timeoutSeconds"
170180
value = "30"

src/docs/content/disaster-recovery/setup.md

+28-30
Original file line numberDiff line numberDiff line change
@@ -167,22 +167,21 @@ A TLS secret is used for SSL termination on the ingress controller. To generate
167167
apiVersion: networking.k8s.io/v1
168168
kind: Ingress
169169
metadata:
170-
name: mushop
171-
annotations:
172-
kubernetes.io/ingress.class: "nginx"
170+
name: mushop
173171
spec:
174-
tls:
175-
- secretName: tls-secret
176-
rules:
177-
- http:
178-
paths:
179-
- path: /
180-
pathType: Prefix
181-
backend:
182-
service:
183-
name: edge
184-
port:
185-
number: 80
172+
ingressClassName: nginx
173+
tls:
174+
- secretName: tls-secret
175+
rules:
176+
- http:
177+
paths:
178+
- path: /
179+
pathType: Prefix
180+
backend:
181+
service:
182+
name: edge
183+
port:
184+
number: 80
186185
EOF
187186
```
188187
@@ -301,22 +300,21 @@ oadbConnectionSecret: oadb-connection # Name of connection secret created earlie
301300
apiVersion: networking.k8s.io/v1
302301
kind: Ingress
303302
metadata:
304-
name: mushop
305-
annotations:
306-
kubernetes.io/ingress.class: "nginx"
303+
name: mushop
307304
spec:
308-
tls:
309-
- secretName: tls-secret
310-
rules:
311-
- http:
312-
paths:
313-
- path: /
314-
pathType: Prefix
315-
backend:
316-
service:
317-
name: edge
318-
port:
319-
number: 80
305+
ingressClassName: nginx
306+
tls:
307+
- secretName: tls-secret
308+
rules:
309+
- http:
310+
paths:
311+
- path: /
312+
pathType: Prefix
313+
backend:
314+
service:
315+
name: edge
316+
port:
317+
number: 80
320318
EOF
321319
```
322320

0 commit comments

Comments
 (0)