search

kubevirt / kubevirt-velero-plugin

Plugin to Velero which automates backing up and restoring KubeVirt/CDI objects
Apache License 2.0
32 stars 28 forks source link

Update cdi, kv #45

Closed brybacki closed 2 years ago

brybacki commented 2 years ago

What this PR does / why we need it:

Update CDI and KV

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): Fixes #

Special notes for your reviewer:

Release note:

NONE
kubevirt-bot commented 2 years ago

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: To complete the pull request process, please ask for approval from brybacki after the PR has been reviewed.

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/kubevirt-velero-plugin/blob/main/OWNERS)** Approvers can indicate their approval by writing `/approve` in a comment Approvers can cancel approval by writing `/approve cancel` in a comment
brybacki commented 2 years ago

/test pull-kvp-functional-test

akalenyu commented 2 years ago

https://storage.googleapis.com/kubevirt-prow/pr-logs/pull/kubevirt_kubevirt-velero-plugin/45/pull-kvp-functional-test/1504024106427748352/artifacts/pods/1_kvp-e2e-tests-8w67d_importer-test-dv-importer.log sounds like https://github.com/kubevirt/containerized-data-importer/issues/2160

brybacki commented 2 years ago

Yes, but this time is happens third time in a row in the same test.

akalenyu commented 2 years ago

Yes, but this time is happens third time in a row in the same test.

Think its related to CDI bump? maybe try bump to something more stable like 1.43.z?

brybacki commented 2 years ago

Yes, but this time is happens third time in a row in the same test.

Think its related to CDI bump? maybe try bump to something more stable like 1.43.z?

I'll try 1.40 again

brybacki commented 2 years ago

/test pull-kvp-functional-test

kubevirt-bot commented 2 years ago

@brybacki: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pull-kvp-functional-test e1edbca5a6f80412af826b58b3b201cc24f8f3bf link true /test pull-kvp-functional-test
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. I understand the commands that are listed [here](https://go.k8s.io/bot-commands).
kubevirt-bot commented 2 years ago

@brybacki: PR needs rebase.

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.