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

spurious lint failures in CI #906

Closed
Trott opened this issue Oct 8, 2017 · 7 comments
Closed

spurious lint failures in CI #906

Trott opened this issue Oct 8, 2017 · 7 comments

Comments

@Trott
Copy link
Member

Trott commented Oct 8, 2017

Key line seems to be:

ok 19 - /usr/home/iojs/build/workspace/node-test-lintercat: stdout: Resource temporarily unavailable

See https://ci.nodejs.org/job/node-test-linter/12318/console for an example.

/cc @refack

@gibfahn
Copy link
Member

gibfahn commented Oct 8, 2017

Hmm, maybe the machine can't handle that much console logging.

@gibfahn
Copy link
Member

gibfahn commented Oct 8, 2017

Probably caused by #720 (comment)

@refack
Copy link
Contributor

refack commented Oct 8, 2017

AFAICT it happened only one (and #720 (comment) is a genius idea!).
RE: "that much console logging" shall I remind you of the 130MB console output for even Windows build ;)

@refack
Copy link
Contributor

refack commented Oct 8, 2017

I think there's an stdout race of some sort between cat and Jenkins. I just rebuilt that job (https://ci.nodejs.org/job/node-test-linter/12355/console) and got a different truncation:

ok 35 - /usr/home/iojs/buBuild step 'Execute shell' marked build as failure

@refack
Copy link
Contributor

refack commented Oct 8, 2017

Got an incantation - replace cat with grep -v -P "^(ok|1\.\.|TAP|$)" test-eslint.tap" (no GNU options on smartOS/BSD machines 🤦‍♂️ ), and maybe add an echo --------- for good measure.

@refack
Copy link
Contributor

refack commented Oct 8, 2017

@Trott to the point of job 12318's failure, see nodejs/node#15805 (comment)

@gibfahn
Copy link
Member

gibfahn commented Oct 8, 2017

Yeah, it's probably due to the speed at which this spewed out 5000 lines of text.

This should be fixed by #720 (comment). I'll close this, reopen if it reoccurs.

@gibfahn gibfahn closed this as completed Oct 8, 2017
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

No branches or pull requests

3 participants