systemvm: add template_zone_ref record when add a new zone with same hypervisor type#8395
Conversation
|
@blueorangutan package |
|
@weizhouapache 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.18 #8395 +/- ##
=========================================
Coverage 13.12% 13.12%
Complexity 9141 9141
=========================================
Files 2720 2720
Lines 257723 257723
Branches 40177 40177
=========================================
Hits 33838 33838
Misses 219595 219595
Partials 4290 4290 ☔ View full report in Codecov by Sentry. |
|
Packaging result [SF]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8116 |
|
@blueorangutan LLtest matrix |
|
@blueorangutan LLtest matrix |
|
@rohityadavcloud a [LL] Trillian-Jenkins matrix job (centos7 mgmt + xenserver71, rocky8 mgmt + vmware67u3, centos7 mgmt + kvmcentos7) has been kicked to run smoke tests |
andrijapanicsb
left a comment
There was a problem hiding this comment.
LGTM
(updated the desc, so it becomes readable for regular mortals :)
|
[LL] Trillian Build Failed (tid-6904) |
|
[LL] Trillian Build Failed (tid-6906) |
|
[LL] Trillian Build Failed (tid-6905) |
|
@blueorangutan test |
|
@weizhouapache a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
[SF] Trillian Build Failed (tid-8657) |
|
[LL] Trillian Build Failed (tid-6907) |
|
[LL] Trillian Build Failed (tid-6908) |
|
@blueorangutan package |
|
@weizhouapache 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]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8125 |
|
@blueorangutan LLtest |
|
@weizhouapache a [LL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
@shwstppr a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
@blueorangutan package |
|
@rohityadavcloud 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]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 8521 |
|
@blueorangutan test |
|
@rohityadavcloud a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests |
|
[SF] Trillian test result (tid-9187)
|
|
@blueorangutan package |
|
@weizhouapache 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]: ✔️ el7 ✔️ el8 ✔️ el9 ✔️ debian ✔️ suse15. SL-JID 9390 |
|
@blueorangutan test rocky8 kvm-rocky8 |
|
@DaanHoogland a [SL] Trillian-Jenkins test job (rocky8 mgmt + kvm-rocky8) has been kicked to run smoke tests |
|
[SF] Trillian test result (tid-9981)
|
|
merging based on approvals and CI tests |
Description
This PR should fix #8388
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Bug Severity
The fix ensures that when you have at least one Zone with a specific hypervisor type (e.g. KVM), then when adding a new Zone with the same hypervisor type, even if the systemVM template for that hypervisor (when registered in the original Zone) is not marked as cross_zone=1 (in vm_template table) - i.e. when it's not marked for "ALL ZONES" - CloudStack will created the record in the template_zone_ref table, and the rest of the work will be done by the automatic systemVM template installation script (that runs when a first Secondary Storage is added inside a new Zone)