-
Notifications
You must be signed in to change notification settings - Fork 70
Add virtualization-kvm as part of requires of job verify_lxd_vm (New) #2143
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
base: main
Are you sure you want to change the base?
Conversation
…fy_lxd_vm This can prevent the job execute on the system that not support KVM instead of execute the job and get fail in later stage
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2143 +/- ##
=======================================
Coverage 52.77% 52.77%
=======================================
Files 395 395
Lines 42563 42563
Branches 7892 7892
=======================================
Hits 22461 22461
Misses 19302 19302
Partials 800 800
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
This will remove a red flag for systems that do support KVM but it's disabled in BIOS. I have concern that simply skipping something that's expected to run and pass will lead to legitimate failures not being caught. For example, when run on a system that supports virtualization: and on the same system with virtualization "accidentally" disabled in BIOS: In the latter case, the test would simply be skipped saying "echo "kvm: unsupported" which is incorrect for the hardware... Perhaps a better way would be to not skip this based on something easily changed by misconfiguration, and rather depend on a manifest entry somewhere in the virtualization chain that says "yes, we expect virtualization to be supported" Additionally, has this been tested on architectures that are not x86 to be sure that the expected results carry over on things like arm64, Power, and s390? |
You make the point. I'll think about how to resolve the situation that you mentioned. |
|
This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in a week. |
Add virtualization-kvm as part of requires of job virtualization/verify_lxd_vm
This can prevent the job execute on the system that not support KVM instead of execute the job and get fail in later stage
Description
Resolved issues
Documentation
Tests
Sideload result
The machine not support kvm
https://certification.canonical.com/hardware/202503-36377/submission/454484/test-results/
The machine support kvm
https://certification.canonical.com/hardware/202010-28347/submission/454489/test-results/