Start VMs so WaitForFirstConsumer DVs can bind

kubevirt / vm-import-operator

Operator for importing VMs from RHV to KubeVirt

Apache License 2.0

16 stars 23 forks source link

Start VMs so WaitForFirstConsumer DVs can bind #483

Closed mansam closed 3 years ago

mansam commented 3 years ago

Adds a check to see whether the DVs are in the WaitForFirstConsumer phase. If so, the Kubevirt VM will be started so that the DVs can bind. On the subsequent reconcile the VM's running state will be restored to the original mapped value.

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1943304

Signed-off-by: Sam Lucidi slucidi@redhat.com

kubevirt-bot commented 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.

Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.

sonarcloud[bot] commented 3 years ago

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

mansam commented 3 years ago

/retest

kubevirt-bot commented 3 years ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: fdupont-redhat, mansam

The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files: - **[OWNERS](https://github.com/kubevirt/vm-import-operator/blob/master/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment