Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

test: mark microtask-queue-run(-domain) as flaky #25606

Closed
wants to merge 1 commit into from
Closed

test: mark microtask-queue-run(-domain) as flaky #25606

wants to merge 1 commit into from

Conversation

joaocgreis
Copy link
Member

test-microtask-queue-run and test-microtask-queue-run-domain appear to be flaky, failing very rarely. Testing 10k runs on the platforms I have available, I got failure rates as high as:

  • Linux x64: OK: 9964 NOT OK: 36
  • Windows 7 x64: OK: 9843 NOT OK: 157
  • SmartOS x86 (gcc 4.7): OK: 9904 NOT OK: 96

The results appear to be influenced by the machine load, I got much lower results on an otherwise idle machine.

Quick test:

time (OK=0; NOK=0; for i in `seq 10000`; do ./out/Release/node test/simple/test-microtask-queue-run.js && OK=$(($OK+1)) || NOK=$(($NOK+1)); echo "$i   OK: $OK   NOT OK: $NOK"; done)

Other tests from test-microtask-queue-* seem okay.

test-microtask-queue-run and test-microtask-queue-run-domain fail very
rarely at least on linux x64, windows 2012r2 x64 and smartos x86
@orangemocha
Copy link
Contributor

LGTM. Please file a P-1 issue to investigate these tests' flakiness and assign it to the next milestone v0.12.6.

@joaocgreis
Copy link
Member Author

Issue to investigate: #25607

/cc @joyent/node-collaborators

@misterdjules
Copy link

LGTM, thank you @joaocgreis 👍

joaocgreis added a commit that referenced this pull request Jul 2, 2015
test-microtask-queue-run and test-microtask-queue-run-domain fail very
rarely at least on linux x64, windows 2012r2 x64 and smartos x86

Reviewed-By: Alexis Campailla <alexis@janeasystems.com>
PR-URL: #25606
@joaocgreis
Copy link
Member Author

Landed in 8294f32

@joaocgreis joaocgreis closed this Jul 3, 2015
jBarz pushed a commit to ibmruntimes/node that referenced this pull request Nov 4, 2016
test-microtask-queue-run and test-microtask-queue-run-domain fail very
rarely at least on linux x64, windows 2012r2 x64 and smartos x86

Reviewed-By: Alexis Campailla <alexis@janeasystems.com>
PR-URL: nodejs#25606
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants