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
node-test-commit-linux-as-shared-lib build is failing #3619
Comments
I think we should change it to run on Ubuntu 22.04. It's using outdated GCC. |
I suspect it's the V8 update from nodejs/node#50115 and the job still building on Ubuntu 18.04 with gcc 8. I've moved the job to build on ubuntu-2204 (with gcc 11). |
I'm also wondering if we still need this separate job -- it runs: NODE_TEST_DIR=${HOME}/node-tmp FLAKY_TESTS=$FLAKY_TESTS_MODE CONFIG_FLAGS="--shared" $MAKE run-ci -j $JOBS e.g. NODE_TEST_DIR=/home/iojs/node-tmp FLAKY_TESTS=run CONFIG_FLAGS=--shared make run-ci -j 2 which is more or less what the ubuntu2204_sharedlibs_shared_x64 builds from node-test-commit-linux-containered are doing: NODE_TEST_DIR=${HOME}/node-tmp \
FLAKY_TESTS=$FLAKY_TESTS_MODE \
CONFIG_FLAGS="$CONFIG_FLAGS --shared" \
make run-ci -j $JOBS # --output-sync=target e.g. + NODE_TEST_DIR=/home/iojs/node-tmp
+ FLAKY_TESTS=dontcare
+ CONFIG_FLAGS=' --shared'
+ make run-ci -j 4 where the main difference between the two is treatment of flaky tests. |
fix patch
|
It seems fixed now that we run on ubuntu2204-64 |
Yes, the job is fixed -- I still question whether we need to maintain this separate job when we more or less cover what it is testing in one of the |
|
node-test-commit-linux-as-shared-lib has been broken since 5 Jan 2024.
It looks to be a fairly consistent compilation failure in turboshaft in V8.
e.g. https://ci.nodejs.org/job/node-test-commit-linux-as-shared-lib/nodes=ubuntu1804-64/2087/console
The text was updated successfully, but these errors were encountered: