Random test failures in test/parallel/test-dgram-send-callback-recursive.js #1675

Activity

1. Fishrock123 commented on May 11, 2015

   

   Fishrock123

   on May 11, 2015

   Contributor

   More actions

   x-post #1313 (comment)

   https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/657/nodes=osx1010/tapTestReport/test.tap-153/

   https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/657/nodes=win2012r2/tapTestReport/test.tap-153/

   https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/657/nodes=win2008r2/tapTestReport/test.tap-153/

   not ok 153 - test-dgram-send-callback-recursive.js
   # c:\workspace\iojs+any-pr+multi\nodes\win2012r2\test\parallel\test-dgram-send-callback-recursive.js:14
   # throw new Error('infinite loop detected');
   # ^
   # Error: infinite loop detected
   # at SendWrap.onsend [as callback] (c:\workspace\iojs+any-pr+multi\nodes\win2012r2\test\parallel\test-dgram-send-callback-recursive.js:14:11)
   # at Immediate._onImmediate (dgram.js:342:10)
   # at processImmediate [as _immediateCallback] (timers.js:368:17)
   
2. mscdex

   added

   dgramIssues and PRs related to UDP and the dgram module.Issues and PRs related to UDP and the dgram module.

   testIssues and PRs related to Node.js core tests and test infrastructure.Issues and PRs related to Node.js core tests and test infrastructure.

   on May 11, 2015
3. yosuke-furukawa commented on May 12, 2015

   

   yosuke-furukawa

   on May 12, 2015

   Member

   More actions

   OK, I will investigate it.
4. yosuke-furukawa commented on May 12, 2015

   

   yosuke-furukawa

   on May 12, 2015

   Member

   More actions

   OK I found the reason.
   I just mistook the condition.
   When io.js CPU load is heavy, the loop counter goes 10 higher like 11, but This counter limit is 10.
   I should fix the condition. I will make pull request ASAP .

   Thank you for notification.
5. Fishrock123

   added a commit that references this issue on May 14, 2015

   test: fix infinite loop detection

   ...

   4e2f999
6. Fishrock123 commented on May 14, 2015

   

   Fishrock123

   on May 14, 2015

   Contributor

   More actions

   Should be fixed in 4e2f999

   CI: https://jenkins-iojs.nodesource.com/view/iojs/job/iojs+any-pr+multi/676/
7. evanlucas commented on May 14, 2015

   

   evanlucas

   on May 14, 2015

   ContributorAuthor

   More actions

   Looks good to me :]
8. evanlucas

   closed this as completedon May 14, 2015
9. Fishrock123

   added a commit that references this issue on May 19, 2015

   test: fix infinite loop detection

   ...

   e7427ae