Skip to content

Testing OpenSSL 3.5.2 vs OpenSSL 3.5.3+ stability issues#316

Merged
rockhowse merged 13 commits intorapidsai:mainfrom
rockhowse:review-openssl-3.5.3-plus-issue
Nov 19, 2025
Merged

Testing OpenSSL 3.5.2 vs OpenSSL 3.5.3+ stability issues#316
rockhowse merged 13 commits intorapidsai:mainfrom
rockhowse:review-openssl-3.5.3-plus-issue

Conversation

@rockhowse
Copy link
Contributor

We have seen some instability when allowing the conda environment when using OpenSSL 3.5.3+

Due to these issues, pinned this repo to ensure we are using OpenSSL 3.5.2 in our conda environments:

This PR is being used to dump some more detailed information about how mamba connectivity is being done at the lowest level to ideally help us identify issues with the newer version of OpenSSL 3.5.3+

@rockhowse rockhowse requested a review from a team as a code owner October 28, 2025 20:16
@rockhowse rockhowse requested review from msarahan and removed request for a team October 28, 2025 20:16
@rockhowse rockhowse added the DO NOT MERGE Hold off on merging; see PR for details label Oct 28, 2025
@rockhowse
Copy link
Contributor Author

@msarahan this is not intended to be reviewed... need an active PR to trigger the builds fail with OpenSSL 3.5.3+. I am using it to get some verbose connectivity logging with OpenSS 3.5.2 (doesn't exhibit the issue) and OpenSSL 3.5.4 (the version that was causing issues in PRs/nightlies)

@rockhowse rockhowse force-pushed the review-openssl-3.5.3-plus-issue branch from 7bff4d6 to 8ed427b Compare November 14, 2025 02:28
@copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 14, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@rockhowse
Copy link
Contributor Author

/ok to test

@copy-pr-bot
Copy link

copy-pr-bot bot commented Nov 17, 2025

/ok to test

@rockhowse, there was an error processing your request: E1

See the following link for more information: https://docs.gha-runners.nvidia.com/cpr/e/1/

@rockhowse
Copy link
Contributor Author

/ok to test e31b06d

@rockhowse
Copy link
Contributor Author

/ok to test 955515d

@rockhowse rockhowse force-pushed the review-openssl-3.5.3-plus-issue branch from 955515d to 6384ba5 Compare November 17, 2025 01:47
@rockhowse rockhowse added bug Something isn't working non-breaking Introduces a non-breaking change and removed DO NOT MERGE Hold off on merging; see PR for details labels Nov 17, 2025
@rockhowse rockhowse requested a review from gforsyth November 17, 2025 22:51
Copy link
Contributor

@gforsyth gforsyth left a comment

Choose a reason for hiding this comment

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

Great work tracking this down @rockhowse -- I'd say let's go ahead and commit your suggestions to remove the (commented out) pins and extra workaround comments, and then merge this in.

@rockhowse rockhowse merged commit aef989c into rapidsai:main Nov 19, 2025
314 checks passed
@rockhowse
Copy link
Contributor Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working non-breaking Introduces a non-breaking change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants