-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Add support EL10 & support java 21 for EL10 #11546
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
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.
Pull Request Overview
This PR adds support for building EL10 (Enterprise Linux 10) compatible packages by creating a symbolic link from el10 to el8 packaging configuration and updating the EL8 spec file to support newer Java versions and system dependencies.
- Creates a symbolic link structure to reuse EL8 packaging for EL10
- Updates Java dependencies to include Java 21 support alongside existing Java 11/17
- Adds compatibility for newer system libraries and utilities in EL10
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| packaging/el8/cloud.spec | Updates dependencies and exclusions to support EL10 compatibility |
| packaging/el10 | Creates symbolic link to reuse EL8 packaging configuration |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #11546 +/- ##
=========================================
Coverage 17.36% 17.36%
Complexity 15237 15237
=========================================
Files 5888 5888
Lines 525737 525737
Branches 64164 64164
=========================================
Hits 91274 91274
Misses 424163 424163
Partials 10300 10300
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:
|
a377df8 to
e77c4b4
Compare
e77c4b4 to
e03fa85
Compare
|
@blueorangutan package |
|
@vishesh92 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 14806 |
|
[SF] Trillian test result (tid-14174)
|
|
[SF] Trillian test result (tid-14172)
|
|
[SF] Trillian test result (tid-14173)
|
rohityadavcloud
left a comment
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
|
[SF] Trillian test result (tid-14288)
|
|
@blueorangutan package |
|
@vishesh92 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 15056 |
|
[SF] Trillian test result (tid-14370)
|
|
@blueorangutan test suse15 kvm-suse15 keepEnv |
|
@vladimirpetrov a [SL] Trillian-Jenkins test job (suse15 mgmt + kvm-suse15) has been kicked to run smoke tests |
|
@blueorangutan test ol10 kvm-ol10 keepEnv |
|
@vladimirpetrov a [SL] Trillian-Jenkins test job (ol10 mgmt + kvm-ol10) has been kicked to run smoke tests |
|
[SF] Trillian test result (tid-14427)
|
|
[SF] Trillian test result (tid-14428)
|
|
@vladimirpetrov for OpenSuse test results, the fix for |
vladimirpetrov
left a comment
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 based on manual testing.
weizhouapache
left a comment
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
|
Merging this based on reviews, manual QA and smoketests. |
Description
This PR adds support for building EL10 compatible packages and also adds support for Java 21 for EL10 packages.
Doc PR: apache/cloudstack-documentation#557
Types of changes
Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale
Screenshots (if appropriate):
How Has This Been Tested?
How did you try to break this feature and the system with this change?