From 8f33d51d9493d4f3e865e3c9b654708148025325 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 25 Nov 2025 15:51:32 +0000 Subject: [PATCH 1/3] Initial plan From a281f39db3bc932653a0c1902b2d0f447fe05979 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 25 Nov 2025 15:55:16 +0000 Subject: [PATCH 2/3] Fix github-repo-registrations-eventsource ingress to use spec.ingressClassName instead of deprecated annotation Co-authored-by: bwalsh <47808+bwalsh@users.noreply.github.com> --- .../events/eventsource-github-from-repo-registrations.yaml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml b/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml index 973b9276..87fc5449 100644 --- a/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml +++ b/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml @@ -78,9 +78,8 @@ metadata: namespace: {{ .Values.events.namespace | default "argo-events" }} labels: source: repo-registration - annotations: - kubernetes.io/ingress.class: {{ .Values.events.github.webhook.ingress.className | default "nginx" | quote }} spec: + ingressClassName: {{ .Values.events.github.webhook.ingress.className | default "nginx" }} rules: {{- range $h := (.Values.events.github.webhook.ingress.hosts | default list) }} - host: {{ $h }} From 5157cfdce2f9c671075ac25103a07df773094a1e Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Tue, 25 Nov 2025 15:56:46 +0000 Subject: [PATCH 3/3] Add quote to ingressClassName for proper YAML parsing Co-authored-by: bwalsh <47808+bwalsh@users.noreply.github.com> --- .../overlays/ingress-authz-overlay/templates/ingress-authz.yaml | 2 +- .../events/eventsource-github-from-repo-registrations.yaml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/helm/argo-stack/overlays/ingress-authz-overlay/templates/ingress-authz.yaml b/helm/argo-stack/overlays/ingress-authz-overlay/templates/ingress-authz.yaml index a056aef2..66e3cb57 100644 --- a/helm/argo-stack/overlays/ingress-authz-overlay/templates/ingress-authz.yaml +++ b/helm/argo-stack/overlays/ingress-authz-overlay/templates/ingress-authz.yaml @@ -40,7 +40,7 @@ metadata: nginx.ingress.kubernetes.io/rewrite-target: {{ $route.rewriteTarget | default "/$2" }} {{- end }} spec: - ingressClassName: {{ $config.ingressClassName | default "nginx" }} + ingressClassName: {{ $config.ingressClassName | default "nginx" | quote }} {{- if $config.tls.enabled }} tls: - hosts: diff --git a/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml b/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml index 87fc5449..fd47b9f1 100644 --- a/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml +++ b/helm/argo-stack/templates/events/eventsource-github-from-repo-registrations.yaml @@ -79,7 +79,7 @@ metadata: labels: source: repo-registration spec: - ingressClassName: {{ .Values.events.github.webhook.ingress.className | default "nginx" }} + ingressClassName: {{ .Values.events.github.webhook.ingress.className | default "nginx" | quote }} rules: {{- range $h := (.Values.events.github.webhook.ingress.hosts | default list) }} - host: {{ $h }}