feat: make addpodidbinding optional (#9)

nuuday · Mar 3, 2023 · 15061ba · 15061ba
1 parent c935a7f
commit 15061ba
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 19 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1,4 +1,7 @@
 # macOS
 .DS_Store
 .AppleDouble
-.LSOverride
+.LSOverride
+
+# Helm Chart packages
+*.tgz
diff --git a/charts/app/Chart.yaml b/charts/app/Chart.yaml
@@ -2,5 +2,4 @@ apiVersion: v2
 name: app
 description: A Helm chart for Kubernetes for running an application
 type: application
-version: 2.0.5
-#appVersion: "1.3.0"
+version: 2.1.0
diff --git a/charts/app/templates/_helpers.tpl b/charts/app/templates/_helpers.tpl
@@ -2,7 +2,7 @@
 Expand the name of the service.
 */}}
 {{- define "nmp-chart.serviceName" -}}
-{{- default .Values.serviceName .Values.serviceNameNameOverride | trunc 63 | trimSuffix "-" }}
+{{- default .Values.serviceName .Values.serviceNameOverride | trunc 63 | trimSuffix "-" }}
 {{- end }}
 
 {{/*
@@ -75,6 +75,8 @@ Selector labels
 {{- define "nmp-chart.selectorLabels" -}}
 app.kubernetes.io/name: {{ include "nmp-chart.deploymentName" . }}
 app.kubernetes.io/instance: {{ include "nmp-chart.deploymentName" . }}
+{{- if .Values.aadpodidbinding }}
 aadpodidbinding: {{ .Values.aadpodidbinding }}
 {{- end }}
+{{- end }}
 
diff --git a/charts/app/templates/tests/test-connection.yaml b/charts/app/templates/tests/test-connection.yaml