Merge pull request #17060 from sabre1041/volumesource-error-msg

openshift-merge-robot · web-flow · commit 67305cec9702 · 2023-01-11T21:02:01.000-05:00
Updated error message for supported VolumeSource types
diff --git a/pkg/specgen/generate/kube/volume.go b/pkg/specgen/generate/kube/volume.go
@@ -245,7 +245,7 @@ func VolumeFromSource(volumeSource v1.VolumeSource, configMaps []v1.ConfigMap, s
 	case volumeSource.EmptyDir != nil:
 		return VolumeFromEmptyDir(volumeSource.EmptyDir, volName)
 	default:
-		return nil, errors.New("HostPath, ConfigMap, EmptyDir, and PersistentVolumeClaim are currently the only supported VolumeSource")
+		return nil, errors.New("HostPath, ConfigMap, EmptyDir, Secret, and PersistentVolumeClaim are currently the only supported VolumeSource")
 	}
 }
 

Original file line number	Diff line number	Diff line change
`@@ -245,7 +245,7 @@ func VolumeFromSource(volumeSource v1.VolumeSource, configMaps []v1.ConfigMap, s`
`245`	`245`	`case volumeSource.EmptyDir != nil:`
`246`	`246`	`return VolumeFromEmptyDir(volumeSource.EmptyDir, volName)`
`247`	`247`	`default:`
`248`		`- return nil, errors.New("HostPath, ConfigMap, EmptyDir, and PersistentVolumeClaim are currently the only supported VolumeSource")`
	`248`	`+ return nil, errors.New("HostPath, ConfigMap, EmptyDir, Secret, and PersistentVolumeClaim are currently the only supported VolumeSource")`
`249`	`249`	`}`
`250`	`250`	`}`
`251`	`251`