From 36bd81996e9beb49ce15180cf215af08ba30b117 Mon Sep 17 00:00:00 2001 From: Dennis Marttinen Date: Wed, 9 Oct 2024 09:57:06 +0300 Subject: [PATCH] fix: add Helm labels and annotations to `cilium` GatewayClass This fixes the deployment flakiness of and Helm's complaints about the `cilium` GatewayClass. Signed-off-by: Dennis Marttinen --- work/manifests/gateway-api/manifests/gateway-class.yaml | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/work/manifests/gateway-api/manifests/gateway-class.yaml b/work/manifests/gateway-api/manifests/gateway-class.yaml index 1985e8c..f1cf4fe 100644 --- a/work/manifests/gateway-api/manifests/gateway-class.yaml +++ b/work/manifests/gateway-api/manifests/gateway-class.yaml @@ -1,6 +1,15 @@ +# The `cilium` GatewayClass should technically be deployed automatically, +# however, that is not reliable in practice. We can specify the resource here, +# but the Helm labels and annotations must also be present, otherwise Cilium +# won't deploy without complaining. apiVersion: gateway.networking.k8s.io/v1 kind: GatewayClass metadata: name: cilium + labels: + app.kubernetes.io/managed-by: Helm + annotations: + meta.helm.sh/release-name: cilium + meta.helm.sh/release-namespace: kube-system spec: controllerName: io.cilium/gateway-controller