fix: don't add secretName: "<nil>" to ingress if no value defined (#1635)

Author: lemeurherve

charts/jenkins/CHANGELOG.md

@@ -12,6 +12,10 @@ Use the following links to reference issues, PRs, and commits prior to v2.6.0.
 The changelog until v1.5.7 was auto-generated based on git commits.
 Those entries include a reference to the git commit to be able to get more details.
 
+## 5.9.6
+
+Don't add `secretName: "<nil>"` to ingress if no value defined
+
 ## 5.9.5
 
 Don't render `tls` ingress section if its value is `{}`

charts/jenkins/Chart.yaml

@@ -2,7 +2,7 @@ apiVersion: v2
 name: jenkins
 type: application
 home: https://www.jenkins.io/
-version: 5.9.5
+version: 5.9.6
 appVersion: 2.541.2
 description: >
   Jenkins - Build great things at any scale! As the leading open source automation server, Jenkins provides over 2000 plugins to support building, deploying and automating any project.

charts/jenkins/templates/jenkins-controller-ingress.yaml

@@ -81,8 +81,10 @@ spec:
 {{- if $.Values.controller.ingress.resourceRootUrl }}
       - {{ tpl $.Values.controller.ingress.resourceRootUrl $ | quote }}
 {{- end }}
+{{- if .secretName }}
     secretName: {{ tpl (.secretName | toString) $ | quote }}
 {{- end }}
 {{- end }}
 {{- end }}
 {{- end }}
+{{- end }}