Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Support vgpu automation test #19718

Merged
merged 1 commit into from
Aug 5, 2024
Merged

Support vgpu automation test #19718

merged 1 commit into from
Aug 5, 2024

Conversation

Julie-CAO
Copy link
Contributor

@Julie-CAO Julie-CAO commented Jul 11, 2024 •
edited
Loading

  • Related ticket: https://progress.opensuse.org/issues/92990
  • Although MIG mode is supported in test scrips but it is not verified because it is beyond the test scope in this ticket. These scripts were written in the past, so keep them for future support.
  • Add VGPU_TEST=short/full to allow run short(no passthough) or full vgpu tests.
  • move set_ssh_console_timeout() into autoyast for sle15+.
  • make SERIALDEV optional and set default value is ttyS1
  • allow to run test from install_package.pm directly
  • remove obsolete scripts from install_package.pm

Verification run:
vGPU tests:
short_test_on_OSD
vgpu_only_full_test_on_OSD
full_test_on_OSD
short_test_on_ph052
a_fail_test_to_check_uploaded_logs

other tests:
gi-guest_developing-on-host_developing-kvm
gi-guest_developing-on-host_sles15sp5-xen
prj4_guest_upgrade_sles12sp5_on_sles12sp5-kvm
prj2_host_upgrade_sles15sp5_to_developing_kvm
uefi-gi-guest_developing-on-host_developing-kvm

@github-actions GitHub Actions
Copy link

Great PR! Please pay attention to the following items before merging:

Files matching lib/**.pm:

  • Consider adding or extending unit tests in t/

This is an automatically generated QA checklist based on modified files.

@Julie-CAO Julie-CAO force-pushed the julie/vgpu branch 2 times, most recently from 6246c5a to 2b74b7d Compare July 15, 2024 06:27
@alice-suse
Copy link
Contributor

The design is quite clear, good job!

@Julie-CAO Julie-CAO force-pushed the julie/vgpu branch 2 times, most recently from f220d7e to a6549c5 Compare July 19, 2024 05:24
@Julie-CAO Julie-CAO force-pushed the julie/vgpu branch 4 times, most recently from cf67414 to a1dd7cb Compare July 31, 2024 05:19
@Julie-CAO Julie-CAO force-pushed the julie/vgpu branch 2 times, most recently from b051ace to 1098bf2 Compare August 1, 2024 04:50
@Julie-CAO Julie-CAO changed the title [WIP]Support vgpu automation test Support vgpu automation test Aug 1, 2024
@Julie-CAO
Copy link
Contributor Author

@alice-suse @guoxuguang @waynechen55 @nanzhg @tbaev @RoyCai7 @martinsmarcelo Welcome review!

alice-suse
alice-suse requested changes Aug 2, 2024
View reviewed changes
Copy link
Contributor

@alice-suse alice-suse left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks great. I only left some minor comment.

@Julie-CAO Julie-CAO force-pushed the julie/vgpu branch 3 times, most recently from 84a93db to c27a6c2 Compare August 2, 2024 12:07
@Julie-CAO
Copy link
Contributor Author

The PR is ready for merge!

@alice-suse alice-suse merged commit ca95fc1 into os-autoinst:master Aug 5, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@guoxuguang guoxuguang guoxuguang left review comments

@alice-suse alice-suse alice-suse approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Julie-CAO @alice-suse @guoxuguang