From 5eab1c4e0ce9e59705100e8bda19782d6add81cf Mon Sep 17 00:00:00 2001 From: ja14000 Date: Thu, 12 Sep 2024 03:07:27 +0100 Subject: [PATCH] add 'pathType' field to 'dagsterWebserver' ingress (#24419) ## Summary & Motivation The 'PathType' field is missing from the ingress template for 'succeedingPaths' in 'dagsterWebserver' paths causing the helm deployment to fail if a 'succeedingPath' has been declared in the input values file. [Issue Link](https://github.com/dagster-io/dagster/issues/24420) ## How I Tested These Changes - Added the 'pathType' field to and re-deployed the helm chart without error. - Confirmed that the resulting ingress resource was created successfully. ## Changelog Insert changelog entry or "NOCHANGELOG" here. - [x] `BUGFIX` add 'pathType' field to 'dagsterWebserver' ingress in ingress.yaml --- helm/dagster/templates/ingress.yaml | 1 + 1 file changed, 1 insertion(+) diff --git a/helm/dagster/templates/ingress.yaml b/helm/dagster/templates/ingress.yaml index b0f36c2b5f0f8..554658c0fbfaf 100644 --- a/helm/dagster/templates/ingress.yaml +++ b/helm/dagster/templates/ingress.yaml @@ -58,6 +58,7 @@ spec: number: {{ $_.Values.dagsterWebserver.service.port | default 80 }} {{- range $_.Values.ingress.dagsterWebserver.succeedingPaths }} - path: {{ .path }} + pathType: {{ .pathType }} backend: service: name: {{ .serviceName }}