-
Notifications
You must be signed in to change notification settings - Fork 1.2k
test_primary_storage_scope.py should only run with kvm, vmware and simulator #9323
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
Conversation
…t they are cleaned up in case of failure
@blueorangutan package |
@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 4.19 #9323 +/- ##
==========================================
Coverage 15.05% 15.05%
- Complexity 11135 11138 +3
==========================================
Files 5405 5405
Lines 472353 472353
Branches 61429 60748 -681
==========================================
+ Hits 71110 71129 +19
+ Misses 393324 393302 -22
- Partials 7919 7922 +3
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10227 |
@blueorangutan test matrix |
@abh1sar a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
[SF] Trillian Build Failed (tid-10702) |
[SF] Trillian Build Failed (tid-10704) |
[SF] Trillian Build Failed (tid-10703) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clgtm
[SF] Trillian Build Failed (tid-10714) |
[SF] Trillian Build Failed (tid-10715) |
@blueorangutan package |
@abh1sar a [SL] Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress. |
[SF] Trillian test result (tid-10716)
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 10235 |
@blueorangutan test matrix |
@sureshanaparti a [SL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
[SF] Trillian test result (tid-10719)
|
[SF] Trillian test result (tid-10720)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
clgtm
…mulator (apache#9323) * test_primary_storage_scope should only run with kvm, vmware and simulator * move cluster create and storage pool create from setup to test so that they are cleaned up in case of failure * fixed lint failure * using super class' tearDown
Description
This PR fixes the smoke failure in test_primary_storage_scope.py when run with XenServer.
#9315
Zone wide storage pools are not supported with Xenserver so storage pool creation was failing during setup, and not cleaning up the created cluster.
This causes cluster create to fail if the test is retried.
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?