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

Investigate flaky test-http-server-consumed-timeout on FreeBSD #7643

Closed
Trott opened this issue Jul 10, 2016 · 1 comment
Closed

Investigate flaky test-http-server-consumed-timeout on FreeBSD #7643

Trott opened this issue Jul 10, 2016 · 1 comment
Labels
freebsd Issues and PRs related to the FreeBSD platform. http Issues or PRs related to the http subsystem. test Issues and PRs related to the tests.

Comments

@Trott
Copy link
Member

Trott commented Jul 10, 2016

https://ci.nodejs.org/job/node-test-commit-freebsd/3211/nodes=freebsd10-64/console

not ok 512 parallel/test-http-server-consumed-timeout
# 
# assert.js:89
#   throw new assert.AssertionError({
#   ^
# AssertionError: Should not happen
#     at IncomingMessage.req.setTimeout (/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-http-server-consumed-timeout.js:14:5)
#     at emitOne (events.js:96:13)
#     at IncomingMessage.emit (events.js:188:7)
#     at Socket.<anonymous> (_http_server.js:307:50)
#     at emitNone (events.js:86:13)
#     at Socket.emit (events.js:185:7)
#     at Socket._onTimeout (net.js:333:8)
#     at tryOnTimeout (timers.js:224:11)
#     at Timer.listOnTimeout (timers.js:198:5)
  ---
  duration_ms: 0.650
@Trott Trott added http Issues or PRs related to the http subsystem. test Issues and PRs related to the tests. freebsd Issues and PRs related to the FreeBSD platform. labels Jul 10, 2016
@Trott
Copy link
Member Author

Trott commented Jul 13, 2016

Another one: https://ci.nodejs.org/job/node-test-commit-freebsd/3273/nodes=freebsd10-64/console

not ok 512 parallel/test-http-server-consumed-timeout
# 
# assert.js:89
#   throw new assert.AssertionError({
#   ^
# AssertionError: Should not happen
#     at IncomingMessage.req.setTimeout (/usr/home/iojs/build/workspace/node-test-commit-freebsd/nodes/freebsd10-64/test/parallel/test-http-server-consumed-timeout.js:14:5)
#     at emitOne (events.js:96:13)
#     at IncomingMessage.emit (events.js:188:7)
#     at Socket.<anonymous> (_http_server.js:307:50)
#     at emitNone (events.js:86:13)
#     at Socket.emit (events.js:185:7)
#     at Socket._onTimeout (net.js:333:8)
#     at tryOnTimeout (timers.js:224:11)
#     at Timer.listOnTimeout (timers.js:198:5)
  ---
  duration_ms: 0.597

@Trott Trott mentioned this issue Jul 13, 2016
Closed
3 tasks
Trott added a commit to Trott/io.js that referenced this issue Jul 13, 2016
@Trott
test: fix flaky test-http-server-consumed-timeout
7127acf 
Using identical timeout values appears to have eliminated the flakiness
in the test.

Fixes: nodejs#7643
@Trott Trott mentioned this issue Jul 13, 2016
Closed
3 tasks
@Trott Trott closed this as completed in 7cb70e9 Jul 17, 2016
evanlucas pushed a commit that referenced this issue Jul 19, 2016
@Trott @evanlucas
test: fix flaky test-http-server-consumed-timeout
aa045cd 
Using identical timeout values appears to have eliminated the flakiness
in the test.

Fixes: #7643
PR-URL: #7717
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
evanlucas pushed a commit that referenced this issue Jul 20, 2016
@Trott @evanlucas
test: fix flaky test-http-server-consumed-timeout
39f5d9c 
Using identical timeout values appears to have eliminated the flakiness
in the test.

Fixes: #7643
PR-URL: #7717
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
MylesBorins pushed a commit that referenced this issue Oct 10, 2016
@Trott
test: fix flaky test-http-server-consumed-timeout
f8b32d9 
Using identical timeout values appears to have eliminated the flakiness
in the test.

Fixes: #7643
PR-URL: #7717
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
rvagg pushed a commit that referenced this issue Oct 18, 2016
@Trott @rvagg
test: fix flaky test-http-server-consumed-timeout
b22838c 
Using identical timeout values appears to have eliminated the flakiness
in the test.

Fixes: #7643
PR-URL: #7717
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
MylesBorins pushed a commit that referenced this issue Oct 26, 2016
@Trott @MylesBorins
test: fix flaky test-http-server-consumed-timeout
924ea0a 
Using identical timeout values appears to have eliminated the flakiness
in the test.

Fixes: #7643
PR-URL: #7717
Reviewed-By: Fedor Indutny <fedor.indutny@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
freebsd Issues and PRs related to the FreeBSD platform. http Issues or PRs related to the http subsystem. test Issues and PRs related to the tests.
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

test: fix flaky test-http-server-consumed-timeout Trott/io.js
1 participant
@Trott