ソースを参照

Double-check https ingress trailing dash

Dash on end was breaking manifest, tested with helm template
gmsantos 4 年 前
コミット
d9fa32fe6b

+ 8 - 8
tools/kubernetes/helm/hue/templates/ingress-https-v1.yaml

@@ -13,11 +13,11 @@ metadata:
     nginx.ingress.kubernetes.io/session-cookie-name: "hue-balancer"
     nginx.ingress.kubernetes.io/session-cookie-expires: "172800"
     nginx.ingress.kubernetes.io/session-cookie-max-age: "172800"
-    {{- if .Values.ingress.hasAuth -}}
+    {{- if .Values.ingress.hasAuth }}
     nginx.ingress.kubernetes.io/auth-type: basic
     nginx.ingress.kubernetes.io/auth-secret: basic-auth-hue
     nginx.ingress.kubernetes.io/auth-realm: 'Authentication Required'
-    {{- end -}}
+    {{- end }}
 {{- with .Values.ingress.annotations }}
 {{ toYaml . | indent 4 }}
 {{- end }}
@@ -26,7 +26,7 @@ spec:
   - host: {{ .Values.ingress.domain }}
     http:
       paths:
-      {{ if .Values.websocket.enabled }}
+      {{- if .Values.websocket.enabled }}
       - backend:
           service:
             name: daphne-websocket
@@ -34,7 +34,7 @@ spec:
               number: 8001
         pathType: ImplementationSpecific
         path: /(ws/.*)
-      {{- end -}}
+      {{- end }}
       - backend:
           service:
             name: hue-balancer
@@ -69,11 +69,11 @@ spec:
   tls:
   - hosts:
     - {{ .Values.ingress.domain }}
-    {{- range .Values.ingress.extraHosts -}}
+    {{- range .Values.ingress.extraHosts }}
     - {{ . | quote }}
-    {{- end -}}
-    {{- if .Values.api.enabled -}}
+    {{- end }}
+    {{- if .Values.api.enabled }}
     - {{ .Values.api.domain }}
-    {{- end -}}
+    {{- end }}
     secretName: letsencrypt-hue-prod-tls
 {{- end -}}

+ 10 - 10
tools/kubernetes/helm/hue/templates/ingress-https-v1beta.yaml

@@ -13,11 +13,11 @@ metadata:
     nginx.ingress.kubernetes.io/session-cookie-name: "hue-balancer"
     nginx.ingress.kubernetes.io/session-cookie-expires: "172800"
     nginx.ingress.kubernetes.io/session-cookie-max-age: "172800"
-    {{- if .Values.ingress.hasAuth -}}
+    {{- if .Values.ingress.hasAuth }}
     nginx.ingress.kubernetes.io/auth-type: basic
     nginx.ingress.kubernetes.io/auth-secret: basic-auth-hue
     nginx.ingress.kubernetes.io/auth-realm: 'Authentication Required'
-    {{- end -}}
+    {{- end }}
 {{- with .Values.ingress.annotations }}
 {{ toYaml . | indent 4 }}
 {{- end }}
@@ -26,12 +26,12 @@ spec:
   - host: {{ .Values.ingress.domain }}
     http:
       paths:
-      {{ if .Values.websocket.enabled }}
+      {{- if .Values.websocket.enabled }}
       - backend:
           serviceName: daphne-websocket
           servicePort: 8001
         path: /(ws/.*)
-      {{- end -}}
+      {{- end }}
       - backend:
           serviceName: hue-balancer
           servicePort: 80
@@ -45,7 +45,7 @@ spec:
           servicePort: 80
         path: /
   {{- end }}
-  {{ if .Values.api.enabled }}
+  {{- if .Values.api.enabled }}
   - host: {{ .Values.api.domain }}
     http:
       paths:
@@ -53,15 +53,15 @@ spec:
           serviceName: hue-api
           servicePort: 8005
         path: /
-  {{ end }}
+  {{- end }}
   tls:
   - hosts:
     - {{ .Values.ingress.domain }}
-    {{- range .Values.ingress.extraHosts -}}
+    {{- range .Values.ingress.extraHosts }}
     - {{ . | quote }}
-    {{- end -}}
-    {{- if .Values.api.enabled -}}
+    {{- end }}
+    {{- if .Values.api.enabled }}
     - {{ .Values.api.domain }}
-    {{- end -}}
+    {{- end }}
     secretName: letsencrypt-hue-prod-tls
 {{- end -}}