fix(chart): provide empty default annotations map for api server ingr…

…ess (#2096) Signed-off-by: Kent Rancourt <kent.rancourt@gmail.com>
akuity · Jun 3, 2024 · 5e2dac5 · 5e2dac5
1 parent afdc0dd
commit 5e2dac5
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/charts/kargo/README.md b/charts/kargo/README.md
@@ -84,7 +84,7 @@ the Kargo controller is running.
 | `api.tls.selfSignedCert`                    | Whether to generate a self-signed certificate for use by the API server. If `true`, `cert-manager` CRDs **must** be present in the cluster. Kargo will create and use its own namespaced issuer. If `false`, a cert secret named `kargo-api-cert` **must** be provided in the same namespace as Kargo.                                                                                                                                                                                                                          | `true`                   |
 | `api.enablePermissiveCORSPolicy`            | Whether to enable a permissive CORS (Cross Origin Resource Sharing) policy. This is sometimes advantageous during local development, but otherwise, should generally be left disabled.                                                                                                                                                                                                                                                                                                                                          | `false`                  |
 | `api.ingress.enabled`                       | Whether to enable ingress. By default, this is disabled. Enabling ingress is advanced usage.                                                                                                                                                                                                                                                                                                                                                                                                                                    | `false`                  |
-| `api.ingress.annotations`                   | Annotations specified by your ingress controller to customize the behavior of the ingress resource.                                                                                                                                                                                                                                                                                                                                                                                                                             | `nil`                    |
+| `api.ingress.annotations`                   | Annotations specified by your ingress controller to customize the behavior of the ingress resource.                                                                                                                                                                                                                                                                                                                                                                                                                             | `{}`                     |
 | `api.ingress.ingressClassName`              | From Kubernetes 1.18+, this field is supported if implemented by your ingress controller. When set, you do not need to add the ingress class as annotation.                                                                                                                                                                                                                                                                                                                                                                     | `nil`                    |
 | `api.ingress.tls.enabled`                   | Whether to enable TLS for the ingress. All other settings in this section will be ignored when this is set to `false`.                                                                                                                                                                                                                                                                                                                                                                                                          | `true`                   |
 | `api.ingress.tls.selfSignedCert`            | Whether to generate a self-signed certificate for use with the API server's Ingress resource. If `true`, `cert-manager` CRDs **must** be present in the cluster. Kargo will create and use its own namespaced issuer. If `false`, a cert secret named `kargo-api-ingress-cert` **must** be provided in the same namespace as Kargo.                                                                                                                                                                                             | `true`                   |

diff --git a/charts/kargo/values.yaml b/charts/kargo/values.yaml
@@ -141,7 +141,7 @@ api:
     ## @param api.ingress.enabled Whether to enable ingress. By default, this is disabled. Enabling ingress is advanced usage.
     enabled: false
     ## @param api.ingress.annotations Annotations specified by your ingress controller to customize the behavior of the ingress resource.
-    annotations:
+    annotations: {}
       # kubernetes.io/ingress.class: nginx
     ## @param api.ingress.ingressClassName From Kubernetes 1.18+, this field is supported if implemented by your ingress controller. When set, you do not need to add the ingress class as annotation.
     ingressClassName: