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

node-test-commit-v8-linux #1370

Closed
mhdawson opened this issue Jun 19, 2018 · 4 comments
Closed

node-test-commit-v8-linux #1370

mhdawson opened this issue Jun 19, 2018 · 4 comments
Labels

Comments

@mhdawson
Copy link
Member

mhdawson commented Jun 19, 2018

Looks like node-test-commit-v8-linux is failing on 10.x for x86.

https://ci.nodejs.org/job/node-test-commit-v8-linux/1452/

I think it may be related to recent updates to tweak what tests run. @refack can you take a look as I think you made those updates. The job failing is: node-test-commit-custom-suites

@mmarchini
Copy link
Contributor

test/v8-updates is not on v10.x yet and the job is trying to run them anyway. We might want to limit this job to run only on master for now.

@refack
Copy link
Contributor

refack commented Jun 20, 2018

Updated node-test-commit-custom-suites to skip test if if [ "$CI_JS_SUITES" = "v8-updates" ] && [ "$NODEJS_MAJOR_VERSION" -lt "11" ]
tested with:
https://ci.nodejs.org/job/node-test-commit-custom-suites/43/ (should skip)
https://ci.nodejs.org/job/node-test-commit-custom-suites/44/ (should run and pass)

Full code

#/bin/bash

echo $NODE_NAME

rm -rf build
git clone https://github.com/nodejs/build.git
. ./build/jenkins/scripts/select-compiler.sh

SKIP_MESSAGE="ok"
REVELANT_VERSION=1
if [ "$NODEJS_MAJOR_VERSION" -lt "6" ]
then
  REVELANT_VERSION=
  # 390 is only supported on v6 and later for now
  SKIP_MESSAGE="ok # skipped s390 not supported for versions less than 6, skipping"
fi
RELEVANT_SUITE=1
if [ "$CI_JS_SUITES" = "v8-updates" ] && [ "$NODEJS_MAJOR_VERSION" -lt "11" ]
then
  RELEVANT_SUITE=
fi
JOBS=$(getconf _NPROCESSORS_ONLN)

if [ $REVELANT_VERSION ] && [ $RELEVANT_SUITE ]
then
  NODE_TEST_DIR=${HOME}/node-tmp
  NODE_COMMON_PORT=15000
  PYTHON=python
  FLAKY_TESTS=$FLAKY_TESTS_MODE
  make build-ci -j $JOBS
  python tools/test.py -j $JOBS -p tap --logfile test.tap --mode=release --flaky-tests=dontcare $TEST_ARGS $CI_JS_SUITES
else
  # fake out so we don't get failures
  mkdir out
  echo "1..1" >test.tap
  echo "ok 1 # skip $SKIP_MESSAGE" >> test.tap
  echo "  ---" >> test.tap
  echo "  duration_ms: 4.925" >> test.tap
  echo "  ..." >> test.tap
fi

@github-actions
Copy link

github-actions bot commented Mar 9, 2020

This issue is stale because it has been open many days with no activity. It will be closed soon unless the stale label is removed or a comment is made.

@github-actions github-actions bot added the stale label Mar 9, 2020
@mhdawson
Copy link
Member Author

mhdawson commented Mar 9, 2020

Closing this is stale

@mhdawson mhdawson closed this as completed Mar 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants