Skip to content
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

ansible: update sharedlibs containers #3758

Merged
merged 2 commits into from
Jun 14, 2024

Conversation

richardlau
Copy link
Member

The earliest supported Node.js version we still build in the CI is 18.x.
For Node.js 18.x, the minimum supported version of ICU is 69.
@richardlau
Copy link
Member Author

richardlau commented Jun 13, 2024

I've deployed this onto the three x64 docker hosts in the test CI.

I'm testing the OpenSSL 3.2 addition in https://ci.nodejs.org/job/richardlau-node-test-commit-linux-containered/26/ but there are expected to be failures (see nodejs/node#53382) -- adding OpenSSL 3.2 to the containers and in a copy of the job (https://ci.nodejs.org/job/richardlau-node-test-commit-linux-containered) will allow easier testing of Node.js compiled with OpenSSL 3.2.

(For the moment, the plan is to stick with OpenSSL 3.0 as long as possible as it's the current OpenSSL LTS version. There's no indication yet what the next OpenSSL LTS version will be, but testing against later OpenSSL should hopefully ease the upgrade path when it happens.)

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

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

LGTM

@richardlau richardlau merged commit 19af747 into nodejs:main Jun 14, 2024
1 check passed
@richardlau richardlau deleted the openssl-containers branch June 14, 2024 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants