Skip to content

CDRIVER-3620 Remove RHEL 6.2 tests and RHEL 7.0 CSE tests from Evergreen #1180

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 6 commits into from
Jan 13, 2023

Conversation

eramongodb
Copy link
Contributor

@eramongodb eramongodb commented Jan 13, 2023

Related to CDRIVER-3620 and resolves CDRIVER-3814. Verified by this patch.

Compilation on RHEL 6.2 has been consistently failing on Evergreen since 40a64c2 (Oct 2022) due to use of GCC pragmas within functions. Removing this variant removes testing with GCC 4.4. Given lack of complaints (so far?) regarding failure to support either RHEL 6.2 or GCC 4.4 since the release of 1.23.2, as well RHEL 6 being EOL and not supported by MongoDB Servers since 5.0, I propose removing this variant.

The only tasks not being covered by other variants were test-4.0-replica-set-(auth|noauth)-sasl-openssl. To maintain replica set test coverage with OpenSSL, these tasks was added to the GCC 7.5 variant instead.

CSE tests on RHEL 7.0 have been consistently failing on Evergreen since 958b871 (May 2022) and even earlier according to CDRIVER-3814 (Nov 2020). Given plenty of equivalent CSE test coverage on other variants and the issue appearing to be a RHEL 7.0 specific environment configuration problem, I propose removing these tasks.

@eramongodb eramongodb requested a review from kevinAlbs January 13, 2023 19:23
@eramongodb eramongodb self-assigned this Jan 13, 2023
Copy link
Collaborator

@kevinAlbs kevinAlbs left a comment

Choose a reason for hiding this comment

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

Please also update installing.rst to remove "RHEL 6.2" from the "Supported Platforms".

@@ -227,16 +227,16 @@ def days(n):
'.compression !.snappy',
'release-compile',
'debug-compile-nosasl-nossl',
'.debug-compile !.sspi .openssl',
'.debug-compile !.sspi .openssl !.client-side-encryption',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Suggested change
'.debug-compile !.sspi .openssl !.client-side-encryption',
# Skip client-side-encryption tests due to OCSP errors on rhel70 host. See: CDRIVER-3620
'.debug-compile !.sspi .openssl !.client-side-encryption',

@eramongodb
Copy link
Contributor Author

Please also update installing.rst to remove "RHEL 6.2" from the "Supported Platforms".

Created CDRIVER-4552 to document drop of support.

@eramongodb eramongodb merged commit ad85279 into mongodb:master Jan 13, 2023
@eramongodb eramongodb deleted the cdriver-rhel branch January 13, 2023 20:15
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.

2 participants