Merge pull request #258 from jck-ccl/master

Use correct templateoptions for helm charts
arangodb · Oct 23, 2018 · c02b089 · c02b089
2 parents 0a32c84 + c2ec908
commit c02b089
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/tools/manifests/manifest_builder.go b/tools/manifests/manifest_builder.go
@@ -148,6 +148,7 @@ ImagePullPolicy: {{ .ImagePullPolicy | quote }}
 RBAC:
   Create: {{ .RBAC }}
 Storage:
+  Create: {{ .Storage.Create }}
   User:
     ServiceAccountName: {{ .Storage.User.ServiceAccountName | quote }}
   Operator:
@@ -403,7 +404,9 @@ func main() {
 			OperatorDeploymentName: "arango-deployment-replication-operator", // Fixed name because only 1 is allowed per namespace
 		},
 		Storage: ResourceOptions{
-			Create: "{{ .Values.Storage.Create }}",
+			Create:      "{{ .Values.Storage.Create }}",
+			FilterStart: "{{- if .Values.Storage.Create }}",
+			FilterEnd:   "{{- end }}",
 			User: CommonOptions{
 				Namespace:          "{{ .Release.Namespace }}",
 				RoleName:           `{{ printf "%s-%s" .Release.Name "storages" | trunc 63 | trimSuffix "-" }}`,
@@ -475,7 +478,7 @@ func main() {
 					}
 					// Execute to tmp buffer
 					tmpBuf := &bytes.Buffer{}
-					t.Execute(tmpBuf, templateOptions)
+					t.Execute(tmpBuf, chartTemplateOptions)
 					// Add tmp buffer to output, unless empty
 					if strings.TrimSpace(tmpBuf.String()) != "" {
 						if output.Len() > 0 {