-
Notifications
You must be signed in to change notification settings - Fork 79
fix: add RHEL9 support for ppc64le and s390x MONGOSH-1680 #2504
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
Adds support for RHEL 9 on ppc64le and s390x across OpenSSL configs and Evergreen CI.
- Applies an OpenSSL patch to silently ignore the
rh-allow-sha1-signatures
option. - Introduces new RHEL 9 smoke test variants in
.evergreen/evergreen.yml.in
and the main CI file. - Extends E2E build variants in
.evergreen/build-variants/e2e-tests-build-variants.js
and.evergreen.yml
for RHEL 9 PPC and s390x.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
File | Description |
---|---|
scripts/nodejs-patches/007-ignore-rh-allow-sha1-signatures.patch | Ignores the RH-specific SHA1 signature flag in OpenSSL module. |
.evergreen/evergreen.yml.in | Adds RHEL 9 ppc64le and s390x package smoke test buildvariants. |
.evergreen/build-variants/e2e-tests-build-variants.js | Registers RHEL 9 PPC and s390x in the E2E build variants array. |
.evergreen.yml | Defines RHEL 9 E2E and smoke test buildvariants for CI. |
Comments suppressed due to low confidence (7)
.evergreen/build-variants/e2e-tests-build-variants.js:303
- [nitpick] Using 'PPC' is ambiguous; consider 'ppc64le' to match other variant names and avoid confusion.
displayName: 'RHEL 9 PPC',
.evergreen.yml:11481
- This entry is indented only 2 spaces but should align with the existing 8-space indent in the Evergreen YAML.
- name: e2e_tests_rhel9_power_small
.evergreen.yml:11517
- This entry is indented only 2 spaces but should align with the existing 8-space indent in the Evergreen YAML.
- name: e2e_tests_rhel9_zseries_small
.evergreen.yml:11889
- This entry is indented only 2 spaces but should align with the existing 8-space indent in the Evergreen YAML.
- name: pkg_smoke_tests_rhel9_s390x
.evergreen/evergreen.yml.in:1721
- This entry is indented only 2 spaces but should align with the existing 8-space indent for consistency in the CI YAML.
- name: pkg_smoke_tests_rhel9_s390x
.evergreen/evergreen.yml.in:1731
- This entry is indented only 2 spaces but should align with the existing 8-space indent for consistency in the CI YAML.
- name: pkg_smoke_tests_rhel9_ppc64le
.evergreen/build-variants/e2e-tests-build-variants.js:302
- [nitpick] The new variant object is indented 2 spaces but should match the surrounding 4-space indent for readability.
{
…penSslSharedConfig
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.
🚀
No description provided.