Skip to content

docker: update containers - gcc8 for cross, openssl3-a14 #2616

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 1 commit into from
Apr 9, 2021

Conversation

rvagg
Copy link
Member

@rvagg rvagg commented Apr 9, 2021

Builds on #2614, it turns out we didn't have gcc-8 ready in the cross compiler containers for host builds.

Also had to do the openssl-3-a14 update because a13 was having "not gzip content" errors (I couldn't replicate locally but upgrading to yesterday's release did the trick). I'm afraid this might have introduced new TLS failures from the one test run I've done so far. https://ci.nodejs.org/job/node-test-commit-linux-containered/26388/#showFailuresLink (test-crypto-dh-stateless & test-crypto-dh).

I'm also watching for some potential new failures from the gcc-8 armv7 builds, more runs need to shake out flakys. We'll also need to keep an eye on nightlies for the release version.

This is live now across the 4 containers in ci and 1 in ci-release.

@richardlau
Copy link
Member

Since this is already live I'll merge so that #2607 can be rebased on top of it.

@richardlau richardlau merged commit 53f0fb7 into master Apr 9, 2021
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