#349 Fix Helm upgrade issue with PVC

jenkinsci · Jun 7, 2020 · 87ee491 · 87ee491
1 parent fcfeb30
commit 87ee491
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 4 deletions.
diff --git a/chart/jenkins-operator/templates/backup.yaml b/chart/jenkins-operator/templates/backup.yaml
@@ -8,7 +8,13 @@ metadata:
     app: jenkins-operator
     jenkins-cr: {{ .Values.jenkins.name }}
 spec:
-  storageClassName: {{ .Values.jenkins.backup.pvc.className }}
+  {{- if .Values.jenkins.backup.pvc.className }}
+  {{- if (eq "-" .Values.jenkins.backup.pvc.className) }}
+  storageClassName: ""
+  {{- else }}
+  storageClassName: "{{ .Values.jenkins.backup.pvc.className }}"
+  {{- end }}
+  {{- end }}
   accessModes:
     - ReadWriteOnce
   resources:

diff --git a/chart/jenkins-operator/values.yaml b/chart/jenkins-operator/values.yaml
@@ -168,7 +168,7 @@ jenkins:
 
       # className is storageClassName for PVC
       # See https://kubernetes.io/docs/concepts/storage/persistent-volumes/#class-1 for more details
-      className: ""
+      className: "-"
 
     # env contains container environment variables
     # PVC backup provider handles these variables:

diff --git a/test/e2e/helm_test.go b/test/e2e/helm_test.go
@@ -49,11 +49,19 @@ func TestDeployHelmChart(t *testing.T) {
 		},
 	}
 
-	cmd := exec.Command("helm", "install", "helm", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
-		"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace))
+	cmd := exec.Command("helm", "upgrade", "jenkins", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
+		"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace, "--install"))
 	output, err := cmd.CombinedOutput()
 	require.NoError(t, err, string(output))
 
+	waitForJenkinsBaseConfigurationToComplete(t, jenkins)
+	waitForJenkinsUserConfigurationToComplete(t, jenkins)
+
+	cmd = exec.Command("helm", "upgrade", "jenkins", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
+		"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace, "--install"))
+	output, err = cmd.CombinedOutput()
+	require.NoError(t, err, string(output))
+
 	// Then
 	waitForJenkinsBaseConfigurationToComplete(t, jenkins)
 	waitForJenkinsUserConfigurationToComplete(t, jenkins)