containerized-data-importer icon indicating copy to clipboard operation
containerized-data-importer copied to clipboard

[release-v1.49] Manual backport of 'Allow creating clones without source PVC (#2306)'

Open alromeros opened this issue 2 years ago • 6 comments

What this PR does / why we need it:

Manual backport of #2306

Release note:

NONE

alromeros avatar Jul 21 '22 09:07 alromeros

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please assign awels after the PR has been reviewed. You can assign the PR to them by writing /assign @awels in a comment when ready.

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

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment Approvers can cancel approval by writing /approve cancel in a comment

kubevirt-bot avatar Jul 21 '22 09:07 kubevirt-bot

/test pull-cdi-apidocs

awels avatar Jul 21 '22 18:07 awels

/test pull-containerized-data-importer-e2e-ceph-gc

alromeros avatar Jul 22 '22 08:07 alromeros

/override pull-containerized-data-importer-e2e-ceph-gc

awels avatar Jul 25 '22 15:07 awels

@awels: Overrode contexts on behalf of awels: pull-containerized-data-importer-e2e-ceph-gc

In response to this:

/override pull-containerized-data-importer-e2e-ceph-gc

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

kubevirt-bot avatar Jul 25 '22 15:07 kubevirt-bot

/lgtm looks good, but I think you need to coordinate changes with @ShellyKa13, on this PR https://github.com/kubevirt/containerized-data-importer/pull/2406 I do not now which change should go first. Probably this one

brybacki avatar Aug 26 '22 09:08 brybacki

/hold

Holding since we have yet to reach a consensus on whether to backport this feature or not. The general idea seemed to be not doing it, which I'm not against, but I'll leave the PR open for some time just in case.

alromeros avatar Aug 26 '22 10:08 alromeros

@alromeros: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

kubevirt-bot avatar Aug 29 '22 19:08 kubevirt-bot

Since https://github.com/kubevirt/containerized-data-importer/pull/2406 was merged, this backport is no longer necessary. I'm closing the PR.

alromeros avatar Aug 31 '22 14:08 alromeros