From ac343aa1a7fdaabe7a38c60fbda7ceef9e921750 Mon Sep 17 00:00:00 2001 From: Chris Pinola Date: Wed, 16 Apr 2025 09:43:53 -0400 Subject: [PATCH 1/2] Fix helm lint parsing error --- helm/ingress-azure/templates/crds.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/helm/ingress-azure/templates/crds.yaml b/helm/ingress-azure/templates/crds.yaml index 242d86926..d43440dbb 100644 --- a/helm/ingress-azure/templates/crds.yaml +++ b/helm/ingress-azure/templates/crds.yaml @@ -10,7 +10,7 @@ spec: paths: - /* {{- end -}} -{{- range .Values.appgw.prohibitedTargets -}} +{{- range .Values.appgw.prohibitedTargets }} apiVersion: appgw.ingress.k8s.io/v1 kind: AzureIngressProhibitedTarget metadata: From c54547a7761924ef43759e591ce6720fcdf58841 Mon Sep 17 00:00:00 2001 From: Chris Pinola Date: Wed, 16 Apr 2025 10:51:08 -0400 Subject: [PATCH 2/2] Test multi prohibited targets --- .../fixtures/sample-config-prohibited-target.json | 6 ++++++ .../ingress-azure/templates/crds.yaml | 11 ++++++++++- 2 files changed, 16 insertions(+), 1 deletion(-) diff --git a/helm/ingress-azure/tests/fixtures/sample-config-prohibited-target.json b/helm/ingress-azure/tests/fixtures/sample-config-prohibited-target.json index 8e9fd7356..55a316a13 100644 --- a/helm/ingress-azure/tests/fixtures/sample-config-prohibited-target.json +++ b/helm/ingress-azure/tests/fixtures/sample-config-prohibited-target.json @@ -12,6 +12,12 @@ "paths": [ "/blacklist/*" ] + }, + { + "name": "prohibit-another-backend-ns", + "paths": [ + "/deny/*" + ] } ] }, diff --git a/helm/ingress-azure/tests/snapshots/sample-config-prohibited-target/ingress-azure/templates/crds.yaml b/helm/ingress-azure/tests/snapshots/sample-config-prohibited-target/ingress-azure/templates/crds.yaml index 198e6642d..1acbe443a 100644 --- a/helm/ingress-azure/tests/snapshots/sample-config-prohibited-target/ingress-azure/templates/crds.yaml +++ b/helm/ingress-azure/tests/snapshots/sample-config-prohibited-target/ingress-azure/templates/crds.yaml @@ -6,4 +6,13 @@ metadata: name: prohibit-backend-ns spec: paths: - - /blacklist/* \ No newline at end of file + - /blacklist/* +--- +# Source: ingress-azure/templates/crds.yaml +apiVersion: appgw.ingress.k8s.io/v1 +kind: AzureIngressProhibitedTarget +metadata: + name: prohibit-another-backend-ns +spec: + paths: + - /deny/* \ No newline at end of file