xenserver: destroy halted vm on expunge #10833
Conversation
Addresses apache#9175 (comment) Signed-off-by: Abhishek Kumar <[email protected]>
|
@Pearl1594 kindly check. Close this PR and pick changes if needed @blueorangutan package |
|
@shwstppr 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 ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 4.19 #10833 +/- ##
============================================
- Coverage 15.17% 15.17% -0.01%
Complexity 11339 11339
============================================
Files 5414 5415 +1
Lines 475185 475228 +43
Branches 57991 57997 +6
============================================
+ Hits 72105 72106 +1
- Misses 395018 395059 +41
- Partials 8062 8063 +1
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:
|
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13299 |
|
@blueorangutan test ol8 xcpng82 |
|
@weizhouapache a [SL] Trillian-Jenkins test job (ol8 mgmt + xcpng82) has been kicked to run smoke tests |
...a/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixCleanupVMCommandWrapper.java
Show resolved
Hide resolved
...a/com/cloud/hypervisor/xenserver/resource/wrapper/xenbase/CitrixCleanupVMCommandWrapper.java
Outdated
Show resolved
Hide resolved
…or/xenserver/resource/wrapper/xenbase/CitrixCleanupVMCommandWrapper.java
|
@blueorangutan package |
|
@shwstppr 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. |
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 13315 |
|
it looks like some vms are not cleaned |
Description
Addresses #9175 (comment)
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?