fix predefined secretname for ingress not to include suffix tls (#62)

* fix predefined secretname for ingress not to include suffix tls

* update chart version

* add ability to add ingress labels

Author: KoomeKiriinya

standard-app/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: standard-app
 description: A Helm chart library by Cloudkite
 type: application
-version: 0.9.7
+version: 0.9.8
 maintainters:
   - email: [email protected]
     name: cloudkite

standard-app/example.values.yaml

@@ -147,7 +147,7 @@ apps:
         args:
           - /etc/scripts/script1.sh
         secrets:
-          dataFrom: true
+          - secretKey: example
     containers:
       example-container-1:
         image: us-central1-docker.pkg.dev/cloudkite-infra-ops/cloudkite-docker-images/app-1

standard-app/templates/network/ingress.yaml

@@ -9,6 +9,9 @@ metadata:
     {{- with $.Values.labels }}
     {{- toYaml . | nindent 4 }}
     {{- end }}
+    {{- with $.Values.ingress.labels }}
+    {{- toYaml . | nindent 4 }}
+    {{- end }}
   annotations:
     {{- range $key, $value := .Values.ingress.annotations }}
     {{ $key }}: {{ $value | quote }}
@@ -52,7 +55,7 @@ spec:
     {{- range .Values.ingress.hosts }}
     - hosts:
         - {{ . | quote }}
-      secretName: {{ $.Values.ingress.secretName | default . | replace "." "-" | replace "*" "star" }}-tls
+      secretName: {{ default ( . | replace "." "-" | replace "*" "star" |  printf "%s-tls")  $.Values.ingress.secretName }}
     {{- end }}
   {{- end }}
 ---