Merge pull request #143 from ncalibey/nick/api-pathing

fix: revert pathing changes to API Ingress
Flagsmith · Jul 24, 2023 · 25cd405 · 25cd405
2 parents 07a55cd + c65ea77
commit 25cd405
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 4 deletions.
diff --git a/.gitignore b/.gitignore
@@ -5,3 +5,4 @@ venv/
 .vscode
 charts/*/charts
 .direnv
+.idea/
diff --git a/charts/flagsmith/Chart.yaml b/charts/flagsmith/Chart.yaml
@@ -2,7 +2,7 @@ apiVersion: v2
 name: flagsmith
 description: Flagsmith
 type: application
-version: 0.20.0
+version: 0.21.0
 appVersion: 2.57.0
 dependencies:
   - name: postgresql

diff --git a/charts/flagsmith/templates/ingress-api.yaml b/charts/flagsmith/templates/ingress-api.yaml
@@ -36,10 +36,11 @@ spec:
   {{- end }}
   rules:
     {{- range .Values.ingress.api.hosts }}
-    - host: {{ . | quote }}
+    - host: {{ .host | quote }}
       http:
         paths:
-          - path: /
+          {{- range .paths }}
+          - path: {{ . }}
             {{- if semverCompare ">=1.19-0" $.Capabilities.KubeVersion.GitVersion }}
             pathType: Prefix
             backend:
@@ -52,5 +53,6 @@ spec:
               serviceName: {{ $fullName }}-api
               servicePort: {{ $svcPort }}
             {{- end }}
+          {{- end }}
     {{- end }}
   {{- end }}
diff --git a/charts/flagsmith/values.yaml b/charts/flagsmith/values.yaml
@@ -302,7 +302,8 @@ ingress:
     # kubernetes.io/ingress.class: nginx
     # kubernetes.io/tls-acme: "true"
     hosts:
-      - chart-example.local
+      - host: chart-example.local
+        paths: [ ]
     tls: []
     #  - secretName: chart-example-tls
     #    hosts: