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

AIX failure after system mainteance. #21451

Closed
mhdawson opened this issue Jun 21, 2018 · 2 comments
Closed

AIX failure after system mainteance. #21451

mhdawson opened this issue Jun 21, 2018 · 2 comments

Comments

@mhdawson
Copy link
Member

  • Version: master
  • Platform: AIX
  • Subsystem: repl

First guess is that it has nothing to do with the update, but instead with the agent being automatically restarted on reboot as we only very rarely reboot the AIX machines.

https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/15915/console

14:36:21 not ok 1434 parallel/test-repl-envvars
14:36:21   ---
14:36:21   duration_ms: 0.324
14:36:21   severity: fail
14:36:21   exitcode: 1
14:36:21   stack: |-
14:36:21     assert.js:80
14:36:21       throw new AssertionError(obj);
14:36:21       ^
14:36:21     
14:36:21     AssertionError [ERR_ASSERTION]: Expected { terminal: true, useColors: true } with { GIT_SSL_CAINFO: '/home/iojs/ca-bundle.crt',
14:36:21       BUILD_URL:
14:36:21        'https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/15915/',
14:36:21       BUILD_TAG: 'jenkins-node-test-commit-aix-nodes=aix61-ppc64-15915',
14:36:21       LOCPATH: '/usr/lib/nls/loc',
14:36:21       NODE_COMMON_PIPE: '/home/iojs/test.pipe',
14:36:21       GITHUB_ORG: 'nodejs',
14:36:21       jenkins_log_file: '/home/iojs/jenkins_console.log',
14:36:21       SHELL: '/bin/bash',
14:36:21       IBM_JAVA_COMMAND_LINE:
14:36:21        '/home/iojs/jdk8/jre/bin/java -Xmx128m -Dorg.jenkinsci.plugins.gitclient.Git.timeOut=30 -jar /home/iojs/slave.jar -secret b9dc892bc932e06f3d0784216ad9b09d5e2fa7e7ec4ff80702eed2d67729e761 -jnlpUrl https://ci.nodejs.org/computer/test-osuosl-aix61-ppc64_be-1/slave-agent.jnlp',
14:36:21       AIXTHREAD_SCOPE: 'S',
14:36:21       GIT_REMOTE_REF: 'refs/heads/master',
14:36:21       ROOT_BUILD_CAUSE_MANUALTRIGGER: 'true',
14:36:21       nodes: 'aix61-ppc64',
14:36:21       MFLAGS: '-w --jobserver-fds=3,4 -j',
14:36:21       ROOT_BUILD_CAUSE: 'MANUALTRIGGER',
14:36:21       BUILD_NUMBER: '15915',
14:36:21       JOB_URL:
14:36:21        'https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/',
14:36:21       POST_STATUS_TO_PR: 'false',
14:36:21       CORE_MMAP: 'yes',
14:36:21       JOB_BASE_NAME: 'nodes=aix61-ppc64',
14:36:21       IBM_JVM_AIXTHREAD_SCOPE_NEW_VALUE: 'S',
14:36:21       IBM_JVM_CORE_MMAP_NEW_VALUE: 'yes',
14:36:21       NODE_TEST_DIR: '/home/iojs/node-tmp',
14:36:21       JOB_DISPLAY_URL:
14:36:21        'https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/display/redirect',
14:36:21       CXX: 'ccache /home/iojs/gcc-6.3.0-1/opt/freeware/bin/g++',
14:36:21       EXECUTOR_NUMBER: '0',
14:36:21       IBM_JVM_CHANGED_ENVVARS_5701668: 'AIXTHREAD_SCOPE,NULLPTR,CORE_MMAP',
14:36:21       BUILD_CAUSE: 'UPSTREAMTRIGGER',
14:36:21       JOBS: '5',
14:36:21       HUDSON_SERVER_COOKIE: '77c5c04cfa349d13',
14:36:21       USER: 'iojs',
14:36:21       AUTHSTATE: 'files',
14:36:21       RUN_DISPLAY_URL:
14:36:21        'https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/15915/display/redirect',
14:36:21       LC__FASTMSG: 'true',
14:36:21       SHLVL: '3',
14:36:21       CXX_host: 'ccache /home/iojs/gcc-6.3.0-1/opt/freeware/bin/g++',
14:36:21       SSL_CERT_FILE: '/home/iojs/ca-bundle.crt',
14:36:21       REPO_NAME: 'node',
14:36:21       NULLPTR: 'NOSEGV',
14:36:21       GIT_COMMIT: '40b1f08a0f559a11434347368f802061f80181c0',
14:36:21       RUN_CHANGES_DISPLAY_URL:
14:36:21        'https://ci.nodejs.org/job/node-test-commit-aix/nodes=aix61-ppc64/15915/display/redirect?page=changes',
14:36:21       _: '/usr/bin/gmake',
14:36:21       BUILD_CAUSE_UPSTREAMTRIGGER: 'true',
14:36:21       ODMDIR: '/etc/objrepos',
14:36:21       GIT_BRANCH: 'origin/_jenkins_local_branch',
14:36:21       IGNORE_FLAKY_TESTS: 'true',
14:36:21       JENKINS_HOME: '/var/lib/jenkins',
14:36:21       HUDSON_URL: 'https://ci.nodejs.org/',
14:36:21       GIT_PREVIOUS_COMMIT: 'a627bc1a50df9a2cec32228f1231bac6cd8e64e4',
14:36:21       WORKSPACE:
14:36:21        '/home/iojs/build/workspace/node-test-commit-aix/nodes/aix61-ppc64',
14:36:21       HOME: '/home/iojs',
14:36:21       JOB_NAME: 'node-test-commit-aix/nodes=aix61-ppc64',
14:36:21       FLAKY_TESTS: 'dontcare',
14:36:21       LANG: 'en_US',
14:36:21       IBM_JVM_NULLPTR_NEW_VALUE: 'NOSEGV',
14:36:21       HUDSON_HOME: '/var/lib/jenkins',
14:36:21       TEST_THREAD_ID: '4',
14:36:21       CLCMD_PASSTHRU: '1',
14:36:21       CONFIG_FLAGS: ' --dest-cpu=ppc64',
14:36:21       OSTYPE: 'aix',
14:36:21       HUDSON_COOKIE: '46acb670-a24b-4289-a87d-d1248d95d2a1',
14:36:21       NODES_SUBSET: 'io.js',
14:36:21       NODE_NAME: 'test-osuosl-aix61-ppc64_be-1',
14:36:21       MAKELEVEL: '2',
14:36:21       LAST_TRIGGERED_JOB_NAME: 'post_build_status_update',
14:36:21       PYTHON: '/usr/bin/python',
14:36:21       JENKINS_URL: 'https://ci.nodejs.org/',
14:36:21       LOGNAME: 'iojs',
14:36:21       NODE_LABELS: 'aix61-ppc64 test-osuosl-aix61-ppc64_be-1',
14:36:21       PATH:
14:36:21        '/home/iojs/gcc-6.3.0-1/opt/freeware/bin:/opt/freeware/bin/ccache:/home/iojs/jdk8/jre/bin:/usr/bin:/etc:/usr/sbin:/usr/ucb:/usr/bin/X11:/sbin:/usr/java6/jre/bin:/usr/java6/bin',
14:36:21       LOGIN: 'iojs',
14:36:21       MAKEFLAGS: 'w --jobserver-fds=3,4 -j',
14:36:21       TERM: 'dumb',
14:36:21       TZ: 'CST6CDT',
14:36:21       GIT_PREVIOUS_SUCCESSFUL_COMMIT: 'c850c14a9cdf6303a48b309bf207be2e951ab68d',
14:36:21       TEST_PARALLEL: '1',
14:36:21       CC: 'ccache /home/iojs/gcc-6.3.0-1/opt/freeware/bin/gcc',
14:36:21       BUILD_ID: '15915',
14:36:21       TEMP_REPO: 'binary_tmp@147.75.70.237:binary_tmp.git',
14:36:21       GIT_URL: 'git@github.com:nodejs/node.git',
14:36:21       NODE_OPTIONS: '',
14:36:21       PWD:
14:36:21        '/home/iojs/build/workspace/node-test-commit-aix/nodes/aix61-ppc64',
14:36:21       JENKINS_SERVER_COOKIE: '77c5c04cfa349d13',
14:36:21       BUILD_DISPLAY_NAME: '#15915',
14:36:21       TEMP_REPO_SERVER: '147.75.70.237',
14:36:21       NLSPATH: '/usr/lib/nls/msg/%L/%N:/usr/lib/nls/msg/%L/%N.cat',
14:36:21       LIBPATH:
14:36:21        '/home/iojs/gmake/opt/freeware/lib:/home/iojs/gcc-6.3.0-1/opt/freeware/lib/gcc/powerpc-ibm-aix6.1.0.0/6.3.0/pthread/ppc64:/home/iojs/gcc-6.3.0-1/opt/freeware/lib' }
14:36:21         at /home/iojs/build/workspace/node-test-commit-aix/nodes/aix61-ppc64/test/parallel/test-repl-envvars.js:52:12
14:36:21         at REPLServer.<anonymous> (internal/repl.js:150:9)
14:36:21         at Object.onceWrapper (events.js:273:13)
14:36:21         at REPLServer.emit (events.js:182:13)
14:36:21         at REPLServer.EventEmitter.emit (domain.js:442:20)
14:36:21         at onwritten (internal/repl.js:186:14)
14:36:21         at FSReqWrap.wrapper [as oncomplete] (fs.js:509:5)
14:36:21   ...
@mhdawson
Copy link
Member Author

Restarted the agents the way we normally do after logging in. Started job to see if that resolves:https://ci.nodejs.org/job/node-test-commit-aix/15917/

This was referenced Jun 21, 2018
Closed
Closed
@mhdawson
Copy link
Member Author

Looks like the restart of the agent resolved the issue. Will close this and open an issue in build repo (nodejs/build#1377) to tweak the auto startup on boot.

MylesBorins added a commit to MylesBorins/node that referenced this issue Dec 26, 2018
@MylesBorins
test: mark test-repl-envvars as flaky on AIX
47364b7 
Refs: nodejs#21451
Refs: nodejs/build#1377
@MylesBorins MylesBorins mentioned this issue Dec 26, 2018
Closed
addaleax added a commit to addaleax/node that referenced this issue Dec 26, 2018
@addaleax
test: fix test-repl-envvars
1f02e58 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: nodejs#21451
Fixes: nodejs/build#1377
Refs: nodejs#25219
@addaleax addaleax mentioned this issue Dec 26, 2018
Closed
3 tasks
@cjihrig cjihrig mentioned this issue Dec 26, 2018
Merged
2 tasks
Trott pushed a commit to Trott/io.js that referenced this issue Dec 27, 2018
@addaleax @Trott
test: fix test-repl-envvars
728777d 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: nodejs#21451
Fixes: nodejs/build#1377
Refs: nodejs#25219

PR-URL: nodejs#25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
addaleax added a commit that referenced this issue Dec 30, 2018
@addaleax
test: fix test-repl-envvars
b7b1d7e 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
addaleax added a commit that referenced this issue Dec 30, 2018
@addaleax
test: fix test-repl-envvars
1bceb9d 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
addaleax added a commit that referenced this issue Dec 30, 2018
@addaleax
test: fix test-repl-envvars
48da8be 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
addaleax added a commit that referenced this issue Dec 30, 2018
@addaleax
test: fix test-repl-envvars
e2faf8c 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
gabrielschulhof pushed a commit to gabrielschulhof/node that referenced this issue Jan 2, 2019
@addaleax
test: fix test-repl-envvars
f2e4510 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: nodejs#21451
Fixes: nodejs/build#1377
Refs: nodejs#25219

PR-URL: nodejs#25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
refack pushed a commit to refack/node that referenced this issue Jan 14, 2019
@refack
test: fix test-repl-envvars
9b28eb5 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: nodejs#21451
Fixes: nodejs/build#1377
Refs: nodejs#25219

PR-URL: nodejs#25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
rvagg pushed a commit that referenced this issue Feb 28, 2019
@addaleax @rvagg
test: fix test-repl-envvars
fbafe8d 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
rvagg pushed a commit that referenced this issue Feb 28, 2019
@addaleax @rvagg
test: fix test-repl-envvars
2948c07 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
BethGriggs pushed a commit that referenced this issue Mar 7, 2019
@addaleax @BethGriggs
test: fix test-repl-envvars
b5d4649 
In 180f865, the test was changed
so that the `env` argument of `createInternalRepl()` also contained
external environment variables, because keeping them can be necessary
for spawning processes on some systems.

However, this test does not spawn new processes, and relies on the
fact that the environment variables it tests are not already set
(and fails otherwise); therefore, reverting to the original state
should fix this.

Fixes: #21451
Fixes: nodejs/build#1377
Refs: #25219

PR-URL: #25226
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: fix test-repl-envvars addaleax/node
1 participant
@mhdawson