-
Notifications
You must be signed in to change notification settings - Fork 314
[develop] Add support for Rocky Linux 8 #5423
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
Codecov ReportAll modified lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #5423 +/- ##
===========================================
- Coverage 89.89% 89.63% -0.27%
===========================================
Files 179 174 -5
Lines 15432 14842 -590
===========================================
- Hits 13873 13303 -570
+ Misses 1559 1539 -20
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
eeb9807 to
1e9d40a
Compare
a456eb5 to
dd52ff0
Compare
560bc5b to
ef8b822
Compare
|
Hi Enrico, |
|
Hi @niallguerin, the support of Rocky Linux 8 wasn't scheduled for 3.7.0 release, where we added Ubuntu22 support. We have to complete internal testing and will evaluate to support it in the future releases of ParallelCluster. |
|
I'm looking forward to the release of this PR. This will be a good test bed for MPI applications that built for EL8 systems. |
4c003ba to
658be42
Compare
Added new os in: * Image Builder components * constants used by pcluster commands * CloudWatch dashboard builder * list of OS supported by DCV * pcluster configure and other unit tests * system-analyzer.sh * pc_support json file * changelog ### References Using `Rocky-8-EC2-Base-8.*` as base AMI from official Rocky Linux account: https://rockylinux.org/cloud-images/ Signed-off-by: Enrico Usai <usai@amazon.com>
658be42 to
d224beb
Compare
|
Thank you Enrico-san! @enrico-usai |
Description of changes
Added new os in:
Tests
### References