fix helm

README.md

@@ -198,13 +198,10 @@ helm repo update
 kubectl create namespace kubeinvaders
 
 # With ingress and TLS enabled
-helm install kubeinvaders --set-string config.target_namespace="namespace1\,namespace2" \
--n kubeinvaders kubeinvaders/kubeinvaders --set ingress.enabled=true --set ingress.hostName=kubeinvaders.io --set deployment.image.tag=latest
+helm install --set-string config.target_namespace="namespace1\,namespace2" --set ingress.enabled=true --set ingress.hostName=kubeinvaders.soukubedev.corp.sourcesense.com --set deployment.image.tag=latest -n kubeinvaders mychaoslab kubeinvaders/kubeinvaders --set ingress.tls_enabled=true
 
 # With ingress enabled but TLS disabled (in case you have a reverse-proxy that does TLS termination and nginx controller in http)
-
-helm install kubeinvaders --set-string config.target_namespace="namespace1\,namespace2" \
--n kubeinvaders kubeinvaders/kubeinvaders --set ingress.enabled=true --set ingress.tls_enabled=false --set ingress.hostName=kubeinvaders.io --set deployment.image.tag=latest
+helm install --set-string config.target_namespace="namespace1\,namespace2" --set ingress.enabled=true --set ingress.hostName=kubeinvaders.soukubedev.corp.sourcesense.com --set deployment.image.tag=latest -n kubeinvaders mychaoslab kubeinvaders/kubeinvaders/ --set ingress.tls_enabled=false
 
 ```
 

helm-charts/kubeinvaders/templates/ingress.yaml

@@ -17,8 +17,8 @@ metadata:
   {{- end }}
 spec:
   ingressClassName: {{ .Values.ingress.className }}
+  {{- if .Values.ingress.tls_enabled  }}
   tls:
-  {{- if .Values.ingress.tls_enabled -}}
     - hosts:
       - {{ .Values.ingress.hostName | quote }}
       secretName: {{ .Values.ingress.secretName }}

helm-charts/kubeinvaders/values.yaml

@@ -62,11 +62,9 @@ ingress:
   enabled: false
   legacyIngress: false
   className: "nginx"
-  annotations:
-    nginx.ingress.kubernetes.io/ssl-redirect: "true"
+  annotations: {}
   hostName: ""
   tls: {}
-  tls_enabled: true
 
 # Use route_host only if ingress is disabled - please specify also the port like route_host: "kubeinvaders.example.com:8080"
 # The port must be the same of the NodePort service