Files
charts/bitnami/apache/templates
Andreas Lindhé 8d539d5d21 [bitnami/apache] Rewrite Ingress TLS logic (#17124)
* [bitnami/apache] Make `ingress.tls` into a bool

Its contents were not parsed anyway since chart version
[v9.0.0](https://github.com/bitnami/charts/pull/7491/), so this change
has no functional effect.

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* [bitnami/apache] Cleanup messy ingress.tls logic

I have convinced myself that this change has no functional effect.

* `ingress.tls` always had to be set, so that logic is the same.
* It is now possible to set `ingress.tls` without setting
  `ingress.certManager` or `ingress.selfSigned`. What's the harm in
  that? Perhaps someone would like to provide a TLS secret manually...

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* [bitnami/apache] Remove unused value ingress.hosts

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* [bitnami/apache] Bump chart version

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* [bitnami/apache] Remove ingress.certManager

This change removes the long deprecated annotation for cert-manager from
the Ingress resource.

NOTE: Technically, this is a breaking change. If someone is using the
`ingress.certManager` value, they will lose the annotation. I imagine
there are few cases where the annotation has any effect, but there's
always someone out there running some really odd configuration...

It's of course still possible to have the annotation, it just has to be
included in `ingress.annotations`, along with the rest of the
annotations.

Since the value is no longer in use anywhere, we also remove it from
values.yaml.

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* [bitnami/apache] Bump chart version

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* [bitnami/apache] Revert breaking change by checking ingress.selfSigned

Before this change, I had accidentally missed the case where the user
has `--set 'ingress.enabled=true,ingress.selfSigned=true'`, which
inadvertently introduced a breaking change.

This change makes sure this case is covered as before the PR was merged.

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>

* Full standardization of ingress

Signed-off-by: Andrés Bono <andresbono@vmware.com>

---------

Signed-off-by: Andreas Lindhé <andreas@lindhe.io>
Signed-off-by: Andrés Bono <andresbono@vmware.com>
Co-authored-by: Andrés Bono <andresbono@vmware.com>
2023-07-18 14:12:04 +02:00
..
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00
2023-06-26 10:28:56 +02:00