kubevirt/containerized-data-importer

The DataVolume "test-vm-storage" is invalid: spec.source.pvc.namespace: Required value

Opened this issue · 1 comments

Is your feature request related to a problem? Please describe:
Cannot clone/create a DataVolume from existing pvc in the current namespace.

Describe the solution you'd like:
namespace field would be optional. When empty, current namespace would be used.
Exactly the same as typical k8s manifests.

Describe alternatives you've considered:
Detecting current namespace and injecting it - not exactly feasible.

Additional context:
I don't know (and don't care) in which namespace my resources are deployed. I create one "template" DataVolume (OS image) that is used for VMs to be created (cloned) from.

We discussed this on the SIG-Storage meeting and it seems reasonable to make namespace optional (and default to the same ns as the source PVC).