-
Notifications
You must be signed in to change notification settings - Fork 30k
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
investigate flaky parallel/test-repl-envvars on AIX #16859
Comments
@nodejs/testing @nodejs/platform-aix |
ACK, trying to reproduce and debug. |
Ran a 1000 times locally, not reproducible. From the error description, looks like this line thew the error:
and from the test logic, it would hint to the first test, that correponds to:
(by matching the expected values, and the then state of environment). If there is some difference between my local and the CI, this reduced test should show that up. const stream = require('stream');
const REPL = require('internal/repl');
const assert = require('assert');
const env = {};
const expected = { terminal: true, useColors: true };
const opts = {
terminal: true,
input: new stream.Readable({ read() {} }),
output: new stream.Writable({ write() {} })
};
REPL.createInternalRepl(env, opts, function(err, repl) {
assert.strictEqual(expected.useColors, repl.useColors);
}); Can someone with access to the CI (AIX) run this a 1000 times please? |
@gireeshpunathil It reproduces reliably on one of the two hosts in CI. I took that host offline. @refack is looking at it now. If you want, hop into IRC #node-build and chat with him. |
I'm going to close this one in favor of nodejs/build#984 because I strongly suspect this is a build issue and not a test issue. |
https://ci.nodejs.org/job/node-test-commit-aix/10064/nodes=aix61-ppc64/console
The text was updated successfully, but these errors were encountered: