flaky test-net-throttle on daily master

[sam-github]

Failed last three nightlies:

• https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/13869/
• https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/13904/
• https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/13850/

Error Message
fail (1)
Stacktrace
build big string
server started on port 38697
assert.js:103
  throw new AssertionError(obj);
  ^

AssertionError [ERR_ASSERTION]: Expected values to be strictly equal:

true !== false

    at Server.<anonymous> (/home/iojs/build/workspace/node-test-commit-custom-suites-freestyle/test/pummel/test-net-throttle.js:38:10)
    at Server.emit (events.js:315:20)
    at TCP.onconnection (net.js:1537:8) {
  generatedMessage: true,
  code: 'ERR_ASSERTION',
  actual: true,
  expected: false,
  operator: 'strictEqual'
}

[sam-github]

We were almost green last night... albeit by diligent skipping of tests.

[Trott]

First failed daily run was for f8d5474 and last successful daily run was for 24a4e61. That's a span of only 6 commits. If this reproduces locally, a bisect should find the problematic commit pretty quickly. And if it doesn't reproduce locally, using CI to find the problematic commit should be feasible.

[Trott]

Starting the bisect on CI since that will probably finish before the local compilation I'm doing for this will finish.

https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/14177/ will be f8d5474 and should fail.

https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/14178/ will be 24a4e61 and should pass.

https://ci.nodejs.org/job/node-test-commit-custom-suites-freestyle/14179/ will be 658cae0, which is between the two.

[Trott]

The test passes locally for me, and all three CI jobs failed, so that suggests it may have been a change on the CI host and not a change in the code...

[Trott]

It looks to me like the test makes assumptions about how much data it will take before the kernel buffer fills and data gets queued in memory, and that this assumption may no longer be true for the CI host. Is that your assessment too, @sam-github? Or should all systems queue data with this test?

@nodejs/build Any chance anyone beefed up test-rackspace-ubuntu1604-x64-1 around two weeks ago?

[Trott]

#33329 seems to fix it.