-
Notifications
You must be signed in to change notification settings - Fork 1.2k
PR #11778 with changes for main branch #11781
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
This bumps the systemvmtemplate version to 4.22 for use with the main/4.22 branch. Signed-off-by: Rohit Yadav <[email protected]>
|
I've used @weizhouapache 's PR to add some custom changes -- but we could in theory just use the 4.20.2.0 template for main/4.22, I don't notice huge difference b/w 4.20 & main for systemvmtemplate use -- please advise @harikrishna-patnala @weizhouapache Kicked build job for this PR, if/when successful (in 1-2 hrs+) the templates will be avaliable here: https://build.yadav.cloud/systemvm/ |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11781 +/- ##
============================================
- Coverage 17.50% 17.50% -0.01%
- Complexity 15427 15428 +1
============================================
Files 5894 5894
Lines 526880 526880
Branches 64338 64338
============================================
- Hits 92243 92242 -1
Misses 424260 424260
- Partials 10377 10378 +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:
|
thanks @rohityadavcloud we do need to rebuild 4.22 system templates due to #11726 |
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.
lgtm
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.
lgtm
|
@blueorangutan package |
|
@harikrishna-patnala 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 ✔️ el10 ✖️ debian ✔️ suse15. SL-JID 15309 |
|
@blueorangutan test matrix |
|
@harikrishna-patnala a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke tests |
|
[SF] Trillian Build Failed (tid-14558) |
|
[SF] Trillian Build Failed (tid-14557) |
|
[SF] Trillian Build Failed (tid-14555) |
|
@blueorangutan test |
|
@harikrishna-patnala a [SL] Trillian-Jenkins test job (ol8 mgmt + kvm-ol8) has been kicked to run smoke tests |
|
[SF] Trillian Build Failed (tid-14597) |
|
Do we need manual building of the packages? And should we upload the 4.22/main systemvmtemplates/files on download.cloudstack.org @weizhouapache @harikrishna-patnala ? |
|
@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. |
@rohityadavcloud Wei has copied the recently built templates for 4.22 internally. So the tests should run now. Once we get the good results, we can merge this PR and then move the templates to download.cloudstack.org. Does this sound good @weizhouapache @rohityadavcloud ? |
@harikrishna-patnala |
|
Packaging result [SF]: ✔️ el8 ✔️ el9 ✔️ el10 ✔️ debian ✔️ suse15. SL-JID 15390 |
|
@blueorangutan test matrix |
|
@harikrishna-patnala a [SL] Trillian-Jenkins matrix job (EL8 mgmt + EL8 KVM, Ubuntu22 mgmt + Ubuntu22 KVM, EL8 mgmt + VMware 7.0u3, EL9 mgmt + XCP-ng 8.2 ) has been kicked to run smoke tests |
|
[SF] Trillian Build Failed (tid-14614) |
|
[SF] Trillian test result (tid-14617)
|
|
[SF] Trillian test result (tid-14615)
|
|
[SF] Trillian test result (tid-14616)
|
|
These tests are failing on health check PR as well. |
* systemvmtemplate: Bump Debian version to 12.12.0 * systemvmtemplate: bump version to 4.22 This bumps the systemvmtemplate version to 4.22 for use with the main/4.22 branch. Signed-off-by: Rohit Yadav <[email protected]> --------- Signed-off-by: Rohit Yadav <[email protected]> Co-authored-by: Wei Zhou <[email protected]>
This continues PR #11778, but for main branch with 4.22 specific changes.
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?