Strict clusterIssuer also for cyberchef and urlaubsverwaltung
Per the maintainer decision (option a: ingress.clusterIssuer strictly required in ALL charts) the two remaining defaults are removed as well: - cyberchef: drop the letsencrypt default; bump raised from minor to major (6.0.0+up11.4.0) as this is a breaking values change. - urlaubsverwaltung: drop the letsencrypt-issuer default; major bump to 6.0.0+up6.7.0. - CI test-values for both charts now set clusterIssuer explicitly. A repo-wide grep confirms no chart carries a clusterIssuer default anymore.
This commit is contained in:
6 files changed
+11
-7
No files matched your search
@@ -1,5 +1,6 @@
|
||||
# cyberchef: stateless, no secrets. Only the ingress host is required
|
||||
# (clusterIssuer already has a default). The Ingress just has to apply -
|
||||
# there is no ingress-nginx/cert-manager in the CI cluster.
|
||||
# cyberchef: stateless, no secrets. Only the required ingress values are set.
|
||||
# The Ingress just has to apply - there is no ingress-nginx/cert-manager in
|
||||
# the CI cluster.
|
||||
ingress:
|
||||
domain: cyberchef.ci.local
|
||||
clusterIssuer: ci
|
||||
@@ -4,3 +4,6 @@ secrets:
|
||||
|
||||
ingress:
|
||||
domain: uv.ci.local
|
||||
# clusterIssuer is strictly required (no default); the Ingress just has to
|
||||
# apply - there is no cert-manager in the CI cluster.
|
||||
clusterIssuer: ci
|
||||
@@ -5,7 +5,7 @@ icon: https://raw.githubusercontent.com/gchq/CyberChef/master/src/web/static/ima
|
||||
|
||||
type: application
|
||||
|
||||
version: 5.2.0+up11.4.0
|
||||
version: 6.0.0+up11.4.0
|
||||
appVersion: "11.4.0"
|
||||
|
||||
maintainers:
|
||||
|
||||
@@ -60,7 +60,7 @@ cyberchef:
|
||||
|
||||
ingress:
|
||||
enabled: true
|
||||
clusterIssuer: letsencrypt
|
||||
clusterIssuer: null
|
||||
domain: null
|
||||
className: nginx
|
||||
# Extra annotations for the Ingress; merged with the cert-manager
|
||||
|
||||
@@ -4,5 +4,5 @@ description: A Helm chart to deploy urlaubsverwaltung on Kubernetes
|
||||
icon: https://raw.githubusercontent.com/urlaubsverwaltung/urlaubsverwaltung/main/src/main/resources/static/favicons/apple-touch-icon.png
|
||||
type: application
|
||||
|
||||
version: 5.0.0+up6.7.0
|
||||
version: 6.0.0+up6.7.0
|
||||
appVersion: 6.7.0
|
||||
@@ -10,7 +10,7 @@ secrets:
|
||||
|
||||
ingress:
|
||||
enabled: true
|
||||
clusterIssuer: "letsencrypt-issuer"
|
||||
clusterIssuer: null
|
||||
domain: null
|
||||
className: nginx
|
||||
# Extra annotations for the Ingress; merged with the cert-manager
|
||||
|
||||
Reference in new issue
Block a user