Skip to content

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

Merged
merged 3 commits into from
Jul 11, 2025
Merged

Conversation

lerouxb
Copy link
Contributor

@lerouxb lerouxb commented Jul 11, 2025

No description provided.

@Copilot Copilot AI review requested due to automatic review settings July 11, 2025 09:18
@lerouxb lerouxb requested a review from a team as a code owner July 11, 2025 09:18
Copy link

@Copilot Copilot AI left a 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.
  {

Copy link
Collaborator

@addaleax addaleax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🚀

@lerouxb
Copy link
Contributor Author

lerouxb commented Jul 11, 2025

@lerouxb lerouxb merged commit 46e4cbb into main Jul 11, 2025
124 of 127 checks passed
@lerouxb lerouxb deleted the RHEL9 branch July 11, 2025 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants