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.

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

Support kubevirt tests on sle-micro 6.0 #19148

Merged

Conversation

nanzhg
Copy link
Contributor

@nanzhg nanzhg commented Apr 19, 2024

  1. Add slem-kubevirt-tests.yaml
  2. Add live migration into core tests
  3. Enable NTP service
  4. Setup cnv-bridge containernetworking plugin
  5. Set the value of default storage class with true
  6. Optimize Whereabouts plugin downloading

@nanzhg nanzhg force-pushed the nanzhg/add_support_for_sle-micro_6.0 branch from 98bb07f to 792c5a4 Compare May 10, 2024 10:42
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.

Thanks for the great PR! I left only some minor comments inline.

But I may leave here a major comment:
in the html report http://10.67.129.96/tests/3173/file/all-tests.html, it is sooooo hard to find the short list that are really executed, no matter pass or fail. Besides, the uploaded log files, eg vnc_test.log, it's not very well readable. Would you please give it a high priority to enhance the report and logs from those regards? Not necessarily in this PR, but better with a following PR soon.

@nanzhg
Copy link
Contributor Author

nanzhg commented May 11, 2024

Thanks for the great PR! I left only some minor comments inline.

But I may leave here a major comment: in the html report http://10.67.129.96/tests/3173/file/all-tests.html, it is sooooo hard to find the short list that are really executed, no matter pass or fail. Besides, the uploaded log files, eg vnc_test.log, it's not very well readable. Would you please give it a high priority to enhance the report and logs from those regards? Not necessarily in this PR, but better with a following PR soon.

You can use command "iconv vnc_test.log" to output the original log. Anyway, will enhance the report in future PR. Thanks

@nanzhg nanzhg force-pushed the nanzhg/add_support_for_sle-micro_6.0 branch 2 times, most recently from 64725a8 to f243ba6 Compare May 13, 2024 06:55
@nanzhg nanzhg force-pushed the nanzhg/add_support_for_sle-micro_6.0 branch from f243ba6 to a81eee0 Compare May 13, 2024 11:38
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.

LGTM

@alice-suse alice-suse merged commit ee220c7 into os-autoinst:master May 13, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants