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.

Sign up for GitHub

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

Jump to bottom

ansible: add Debian 10 #2612

Merged
merged 4 commits into from
Apr 9, 2021
Merged

ansible: add Debian 10 #2612

merged 4 commits into from
Apr 9, 2021

Conversation

richardlau
Copy link
Member

Replace test-softlayer-debian8-x86-1 with a Debian 10 based image.
openjdk-8-jre-headless is no longer available from the Debian 10
repositories, so replace with openjdk-11-jre-headless.

Refs: https://lists.debian.org/debian-java/2019/04/msg00036.html
Refs: #2600

@richardlau
Copy link
Member Author

Changes have been run against test-softlayer-debian10-x64-1 (which is the old test-softlayer-debian8-x86-1 but reimaged ("Load selected image") with "Debian GNU/Linux 10.x Buster/Stable - Minimal Install (64 bit)" through the IBM Cloud interface).

Test build in progress: https://ci.nodejs.org/job/node-test-commit-linux-richardlau/4/nodes=debian10-x64/
(am not intending to update the main node-test-commit-linux job until we have a second debian 10 host.)

@richardlau
Copy link
Member Author

We are mostly running Java 8 elsewhere... if desired I can try getting Java 8 from the adoptopenjdk project instead of pulling from Debian repositories.

@richardlau richardlau mentioned this pull request Apr 7, 2021
Closed
@richardlau
Copy link
Member Author

Test build and second test build (https://ci.nodejs.org/job/node-test-commit-linux-richardlau/5/) failed lots of tests with "key too small" errors. I think this is related to nodejs/node#27862.

@richardlau
Copy link
Member Author

Added a commit to drop the OpenSSL security level back to the OpenSSL default of 1 (Debian increased it to 2 but that breaks over a hundred Node.js tests). Longer term we should look to resolve nodejs/node#27862 but this will get us going.

Test build: https://ci.nodejs.org/job/node-test-commit-linux-richardlau/7/nodes=debian10-x64/
is now down to a single failure, parallel/test-zlib-unused-weak, which is failing on other platforms and may be fixed by nodejs/node#38149.

@richardlau
Copy link
Member Author

A test build of nodejs/node#38149 passed (🎉): https://ci.nodejs.org/job/node-test-commit-linux-richardlau/8/nodes=debian10-x64/

@richardlau richardlau changed the title ansible: replace test-softlayer-debian8-x86-1 ansible: add Debian 10 Apr 8, 2021
@richardlau
Copy link
Member Author

I've reimaged test-rackspace-debian8-x64-2 as test-rackspace-debian10-x64-1 and run the scripts in this PR against it.
Test build: https://ci.nodejs.org/job/node-test-commit-linux-richardlau/9/nodes=debian10-x64/

@richardlau
ansible: replace test-softlayer-debian8-x86-1
7804962 
Replace `test-softlayer-debian8-x86-1` with a Debian 10 based image.
`openjdk-8-jre-headless` is no longer available from the Debian 10
repositories, so replace with `openjdk-11-jre-headless`.

Refs: https://lists.debian.org/debian-java/2019/04/msg00036.html
@richardlau
ansible: reset OpenSSL security level on Debian 10
36344c2 
Debian 10 raises the security level for OpenSSL to 2 which breaks
over one hundred Node.js tests. Lower the security level back down
to the OpenSSL default of 1 to allow tests to pass.
@richardlau
ansible: replace test-rackspace-debian8-x64-2
ac5abb5 
Replace `test-rackspace-debian8-x64-2` with a Debian 10 based image.
@richardlau
jenkins: stop testing on Debian 9 from Node.js 16
8765fd7 
Python and gcc/g++ in Debian 9 are not recent enough for the proposed
minumum supported versions for Node.js 16.
@richardlau
Copy link
Member Author

Added VersionSelectorScript changes. This is ready for review.

@richardlau richardlau merged commit 3e7f097 into nodejs:master Apr 9, 2021
@richardlau richardlau deleted the debian10 branch April 9, 2021 19:05
@richardlau richardlau mentioned this pull request Sep 2, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@AshCripps AshCripps AshCripps approved these changes

Assignees
No one assigned
Labels
ansible platform:linux-intel
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@richardlau @AshCripps