#349 Fix Helm upgrade issue with PVC
This commit is contained in:
parent
87ee491a74
commit
8c62b6909d
|
|
@ -168,7 +168,7 @@ jenkins:
|
||||||
|
|
||||||
# className is storageClassName for PVC
|
# className is storageClassName for PVC
|
||||||
# See https://kubernetes.io/docs/concepts/storage/persistent-volumes/#class-1 for more details
|
# See https://kubernetes.io/docs/concepts/storage/persistent-volumes/#class-1 for more details
|
||||||
className: "-"
|
className: ""
|
||||||
|
|
||||||
# env contains container environment variables
|
# env contains container environment variables
|
||||||
# PVC backup provider handles these variables:
|
# PVC backup provider handles these variables:
|
||||||
|
|
|
||||||
|
|
@ -50,7 +50,7 @@ func TestDeployHelmChart(t *testing.T) {
|
||||||
}
|
}
|
||||||
|
|
||||||
cmd := exec.Command("helm", "upgrade", "jenkins", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
|
cmd := exec.Command("helm", "upgrade", "jenkins", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
|
||||||
"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace, "--install"))
|
"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace), "--install")
|
||||||
output, err := cmd.CombinedOutput()
|
output, err := cmd.CombinedOutput()
|
||||||
require.NoError(t, err, string(output))
|
require.NoError(t, err, string(output))
|
||||||
|
|
||||||
|
|
@ -58,7 +58,7 @@ func TestDeployHelmChart(t *testing.T) {
|
||||||
waitForJenkinsUserConfigurationToComplete(t, jenkins)
|
waitForJenkinsUserConfigurationToComplete(t, jenkins)
|
||||||
|
|
||||||
cmd = exec.Command("helm", "upgrade", "jenkins", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
|
cmd = exec.Command("helm", "upgrade", "jenkins", "./chart/jenkins-operator", "--namespace", namespace, "--debug",
|
||||||
"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace, "--install"))
|
"--set-string", fmt.Sprintf("jenkins.namespace=%s", namespace), "--install")
|
||||||
output, err = cmd.CombinedOutput()
|
output, err = cmd.CombinedOutput()
|
||||||
require.NoError(t, err, string(output))
|
require.NoError(t, err, string(output))
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Reference in New Issue