Skip to content

Kubevirt datamover e2e#2174

Open
weshayutin wants to merge 20 commits into
openshift:oadp-1.6from
weshayutin:kubevirt-datamover-e2e
Open

Kubevirt datamover e2e#2174
weshayutin wants to merge 20 commits into
openshift:oadp-1.6from
weshayutin:kubevirt-datamover-e2e

Conversation

@weshayutin

@weshayutin weshayutin commented Apr 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Why the changes were made

Need a first pass at kubevirt-datamover tests

  1. installs an upstream build of HCO. This is required to catch and test the latest code from cnv
  2. setups the requirements for kubevirt datamover, volume policy , dpa etc.
  3. Uses a defined cirros vm w/ snapshotmovedata = true + volume policy.
  4. More verbose console dump of vmbackup, dataupload and velero details.

@coderabbitai

coderabbitai Bot commented Apr 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Repository: openshift/coderabbit/.coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: 77925efa-492c-4ac1-8829-89ae132ec4b7

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

@openshift-ci openshift-ci Bot requested review from eemcmullan and kaovilai April 22, 2026 01:51
@weshayutin

weshayutin commented Apr 22, 2026

Copy link
Copy Markdown
Contributor Author

@kaovilai if needed we could increase the storage capacity of the test profile in openshift/release for the kubevirt job right?

@kaovilai

kaovilai commented Apr 22, 2026

Copy link
Copy Markdown
Member

Not sure. But we can definitely have it use our own cluster which we can size how we want

sseago
sseago reviewed Apr 22, 2026
View reviewed changes
Comment thread tests/e2e/sample-applications/virtual-machines/kubevirt-dm/README.md
Comment thread tests/e2e/sample-applications/virtual-machines/kubevirt-dm/README.md Outdated
@weshayutin

weshayutin commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

/retest

shubham-pampattiwar
shubham-pampattiwar reviewed Apr 28, 2026
View reviewed changes
Comment thread tests/e2e/templates/default_settings.json
Comment thread tests/e2e/virt_backup_restore_suite_test.go
@openshift-ci openshift-ci Bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 28, 2026
@openshift-ci

openshift-ci Bot commented Apr 29, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: weshayutin
Once this PR has been reviewed and has the lgtm label, please assign dymurray for approval. For more information see the Code Review Process.

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

Details 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

@openshift-ci openshift-ci Bot removed the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Apr 29, 2026
@openshift-ci

openshift-ci Bot commented Apr 29, 2026

Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: weshayutin
Once this PR has been reviewed and has the lgtm label, please assign dymurray for approval. For more information see the Code Review Process.

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

Details 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

kaovilai
kaovilai reviewed Apr 30, 2026
View reviewed changes
Comment thread tests/e2e/lib/backup.go Outdated
weshayutin added 15 commits June 8, 2026 15:38
@weshayutin
DNM, first pass at kubevirt-datamover e2e test
7966e63 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
DNM, first pass at kubevirt-datamover e2e, working locally
8e5c00e 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
print out kubevirt objects, and directions
6f62e1c 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
add fedora
0f4795a 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
fix lint
d8f51cb 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
need the kubevirt 1.8 release for kubevirtdm
5a55941 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
ensure namespace is clean
5b7d414 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
update, remove fedora cbt, it's not ready for ci
21da6e8 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
clean up and retest
24a82f2 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
add clean up and centos
463d608 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
update readme
be65b53 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
use the fedora container dv
93eb762 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
used tagged version of fedora
2cbbe4b 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
add annotation shell script
5ff5ef5 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
update sc for vms
19a3343 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
weshayutin added 4 commits June 8, 2026 15:39
@weshayutin
update clean up script
aa72d16 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
update for win
02ff08a 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
fix fedora user login
a433859 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin
use fedora 44 containerdisk
893d4d2 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin weshayutin force-pushed the kubevirt-datamover-e2e branch from 307c17a to 893d4d2 Compare June 9, 2026 17:18
@weshayutin

weshayutin commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

/retest-required

@weshayutin

weshayutin commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

/test 5.0-e2e-test-kubevirt-aws

@weshayutin
update from comments
0ae37df 
Signed-off-by: Wesley Hayutin <weshayutin@gmail.com>
@weshayutin

weshayutin commented Jun 10, 2026

Copy link
Copy Markdown
Contributor Author

@kaovilai @shubham-pampattiwar ok.. ready for eyes again :) once e2e pass

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@sseago sseago sseago left review comments

@kaovilai kaovilai kaovilai left review comments

@shubham-pampattiwar shubham-pampattiwar shubham-pampattiwar left review comments

@eemcmullan eemcmullan Awaiting requested review from eemcmullan

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@weshayutin @kaovilai @sseago @shubham-pampattiwar