Closed mansam closed 3 years ago
@mansam: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.
[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: fdupont-redhat, mansam, pkliczewski
To complete the pull request process, please assign
You can assign the PR to them by writing /assign
in a comment when ready.
The full list of commands accepted by this bot can be found here.
This uses the
anyuid
SCC to set the fsGroup for the virt-v2v pod to the same one used by the CDI importer pod. This solves a permissions problem that was preventing the virtio overlay from being committed properly when importing a VM's disk to thenfs
storage class.The virt-v2v pod needs to be created by the vm-import-controller in order for it to have access to the
anyuid
SCC. If it is created via a Job, the job-controller is responsible for creating it and it will not have access to the needed SCC. Thus, all job related code has been refactored to manage a pod directly.Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1904797
Signed-off-by: Sam Lucidi slucidi@redhat.com