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,test: skip test due to file size limit #16273

Closed
wants to merge 2 commits into from
Closed

aix,test: skip test due to file size limit #16273

wants to merge 2 commits into from

Conversation

jBarz
Copy link
Contributor

@jBarz jBarz commented Oct 18, 2017 •
edited
Loading

The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label Oct 18, 2017
@jBarz jBarz mentioned this pull request Oct 18, 2017
Closed
@jBarz
aix,test: skip test due to file size limit
b9b278b 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.
@addaleax addaleax added the aix Issues and PRs related to the AIX platform. label Oct 18, 2017
MylesBorins
MylesBorins approved these changes Oct 18, 2017
View reviewed changes
Copy link
Contributor

@MylesBorins MylesBorins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MylesBorins
Copy link
Contributor

I don't think this should need 48 hours to land. Any one else agree?

@hiroppy
Copy link
Member

hiroppy commented Oct 18, 2017

CI: https://ci.nodejs.org/job/node-test-pull-request/10798/

@jBarz
Copy link
Contributor Author

jBarz commented Oct 18, 2017

My bad, I forgot to multiply by block size (to get the value in bytes).
I fixed this in the second commit.

@gibfahn
Copy link
Member

gibfahn commented Oct 18, 2017

CI 2: https://ci.nodejs.org/job/node-test-commit/13260/

@jBarz jBarz mentioned this pull request Oct 18, 2017
Closed
jasnell
jasnell approved these changes Oct 18, 2017
View reviewed changes
Copy link
Member

@jasnell jasnell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM with green CI

mhdawson
mhdawson approved these changes Oct 18, 2017
View reviewed changes
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@richardlau richardlau mentioned this pull request Oct 19, 2017
Closed
3 tasks
@mhdawson
Copy link
Member

OSX failure was unrelated, opened #16321

@mhdawson
Copy link
Member

Arm failures not related, going to land.

@mhdawson
Copy link
Member

Landed as 31cde1c

@mhdawson mhdawson closed this Oct 19, 2017
mhdawson pushed a commit that referenced this pull request Oct 19, 2017
@jBarz @mhdawson
test: skip test due to file size limit
31cde1c 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: #16273
Fixes: #16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@jBarz jBarz deleted the aixfilesize branch October 19, 2017 14:08
MylesBorins pushed a commit that referenced this pull request Oct 23, 2017
@jBarz @MylesBorins
test: skip test due to file size limit
0a0fa2b 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: #16273
Fixes: #16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
addaleax pushed a commit to ayojs/ayo that referenced this pull request Oct 26, 2017
@jBarz @addaleax
test: skip test due to file size limit
240fbe5 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: nodejs/node#16273
Fixes: nodejs/node#16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
MylesBorins pushed a commit that referenced this pull request Nov 16, 2017
@jBarz @MylesBorins
test: skip test due to file size limit
a20ea14 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: #16273
Fixes: #16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Nov 21, 2017
Merged
MylesBorins pushed a commit that referenced this pull request Nov 21, 2017
@jBarz @MylesBorins
test: skip test due to file size limit
2b4c981 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: #16273
Fixes: #16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
MylesBorins pushed a commit that referenced this pull request Nov 28, 2017
@jBarz @MylesBorins
test: skip test due to file size limit
9e74e54 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: #16273
Fixes: #16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
addaleax pushed a commit to ayojs/ayo that referenced this pull request Dec 7, 2017
@jBarz @addaleax
test: skip test due to file size limit
3158787 
The test requires a file size limit that is greater than the string
size limit. Some aix machines might not meet this criteria so in
which case we should skip the test.

PR-URL: nodejs/node#16273
Fixes: nodejs/node#16319
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: MichaëZasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Yuta Hiroto <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gibfahn gibfahn gibfahn approved these changes

@mhdawson mhdawson mhdawson approved these changes

@hiroppy hiroppy hiroppy approved these changes

@cjihrig cjihrig cjihrig approved these changes

@targos targos targos approved these changes

@addaleax addaleax addaleax approved these changes

@MylesBorins MylesBorins MylesBorins approved these changes

@jasnell jasnell jasnell approved these changes

Assignees
No one assigned
Labels
aix Issues and PRs related to the AIX platform. test Issues and PRs related to the tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
10 participants
@jBarz @MylesBorins @hiroppy @gibfahn @mhdawson @jasnell @addaleax @targos @cjihrig @nodejs-github-bot