Skip to content

failed test: parallel/test-tls-inception.js #1012

Closed
Closed
failed test: parallel/test-tls-inception.js#1012
Labels
freebsdIssues and PRs related to the FreeBSD platform.testIssues and PRs related to Node.js core tests and test infrastructure.

Description

@jbergstroem
jbergstroem
opened on Mar 1, 2015

This test fails on FreeBSD 10.1-release:

=== release test-tls-inception ===                                 
Path: parallel/test-tls-inception
events.js:141
      throw er; // Unhandled 'error' event
            ^
Error: read ECONNRESET
    at exports._errnoException (util.js:734:11)
    at TLSWrap.onread (net.js:538:26)
Command: out/Release/iojs /usr/home/jbergstroem/io.js/test/parallel/test-tls-inception.js

git bisect tells me that b968623 is the offending commit (verified by running parent 4 times). Unfortunately, this seems to be race-y.

/cc @indutny

Activity

  1. brendanashworth
    added
    testIssues and PRs related to Node.js core tests and test infrastructure.
    freebsdIssues and PRs related to the FreeBSD platform.
    on Mar 1, 2015

  2. jbergstroem commented on Mar 2, 2015

    @jbergstroem
    jbergstroem
    on Mar 2, 2015
    MemberAuthor

    This test fails on SmartOS as well (upcoming build slave in the jenkins test cluster) with the same behaviour.

  3. indutny commented on Mar 2, 2015

    @indutny
    indutny
    on Mar 2, 2015
    Member

    Reproduced the problem, working on the isolating it.

  4. indutny commented on Mar 2, 2015

    @indutny
    indutny
    on Mar 2, 2015
    Member

    Seems to coming from this syscall:

    shutdown(0x10, 0x1, 0x0)         = -1 Err#54

  5. indutny commented on Mar 2, 2015

    @indutny
    indutny
    on Mar 2, 2015
    Member

    Nope, it isn't. Guess we are not closing the socket at the right time... or closing it way too early on other side. Will figure it out later, doesn't seem to be a C++ issue from the first sight, just a behavior change... :(

  6. indutny commented on Mar 2, 2015

    @indutny
    indutny
    on Mar 2, 2015
    Member

    Small log:

    (0x8030fcc80) StreamWrap()
(0x8030fce40) StreamWrap()
(0x8030fd8c0) StreamWrap()
(0x8030fda80) StreamWrap()
(0x8030fdc40) StreamWrap()
(0x8030fde70) TLSWrap() on 0x8030fdc40 StreamBase
(0x8030fe030) TLSWrap() on 0x8030fde70 StreamBase
(0x8030fe180) StreamWrap()
(0x8030fe3b0) TLSWrap() on 0x8030fe180 StreamBase
(0x8030fe180) OnReadCommon 258
(0x8030fe3b0) OnReadImpl 258
(0x8030fdc40) OnReadCommon 1168
(0x8030fde70) OnReadImpl 1168
(0x8030fe180) OnReadCommon 299
(0x8030fe3b0) OnReadImpl 299
(0x8030fe6c0) StreamWrap()
(0x8030fe500) StreamWrap()
(0x8030fe8f0) TLSWrap() on 0x8030fe500 StreamBase
(0x8030fdc40) OnReadCommon 314
(0x8030fde70) OnReadImpl 314
(0x8030fe180) OnReadCommon 341
(0x8030fe3b0) OnReadImpl 341
(0x8030fe3b0) OnReadSelf 258
(0x8030fe500) OnReadCommon 258
(0x8030fe8f0) OnReadImpl 258
(0x8030fe6c0) OnReadCommon 1168
(0x8030fdc40) OnReadCommon 1253
(0x8030fde70) OnReadImpl 1253
(0x8030fe030) OnReadImpl 1024
(0x8030fe030) OnReadImpl 144
(0x8030fe180) OnReadCommon 373
(0x8030fe3b0) OnReadImpl 373
(0x8030fe3b0) OnReadSelf 299
(0x8030fe500) OnReadCommon 299
(0x8030fe8f0) OnReadImpl 299
(0x8030fe500) DoShutdown
(0x8030fe6c0) OnReadCommon 484
(0x8030fe6c0) OnReadCommon -4095
(0x8030fe680) Close()
(0x8030fe180) DoShutdown
(0x8030fdc40) OnReadCommon 565
(0x8030fde70) OnReadImpl 565
(0x8030fe030) OnReadImpl 484
(0x8030fe030) OnReadSelf 5
(0x8030fe030) OnReadSelf -4095
(0x8030fdc00) Close()
(0x8030fd880) Close()
(0x8030fda40) Close()
(0x8030fe500) OnReadCommon -4095
(0x8030fe8f0) OnReadImpl -4095
(0x8030fe8f0) OnReadSelf -4095
(0x8030fe4c0) Close()
(0x8030fe180) OnReadCommon -54
(0x8030fe3b0) OnReadImpl -54
(0x8030fe3b0) OnReadSelf -54
(0x8030fe140) Close()

  7. indutny commented on Mar 3, 2015

    @indutny
    indutny
    on Mar 3, 2015
    Member

    dtruss output:

    https://gist.github.com/indutny/19b45650664938fd2dd9

  8. indutny
    added a commit that references this issue on Mar 3, 2015

    test: fix racey-ness in tls-inception

    0eabef3

  9. indutny commented on Mar 3, 2015

    @indutny
    indutny
    on Mar 3, 2015
    Member

    Should be fixed by #1040

  10. indutny
    added a commit that references this issue on Mar 3, 2015

    test: fix racey-ness in tls-inception

    e1bf670

  11. jbergstroem commented on Mar 4, 2015

    @jbergstroem
    jbergstroem
    on Mar 4, 2015
    MemberAuthor

    Confirmed fixed. Closing.

  12. jbergstroem
    closed this as completedon Mar 4, 2015

  13. nikolmarku1 commented on Oct 1, 2015

    @nikolmarku1
    nikolmarku1
    on Oct 1, 2015

    this is not fixed just seen in on V 1 - 4 - 13

  14. jbergstroem commented on Oct 1, 2015

    @jbergstroem
    jbergstroem
    on Oct 1, 2015
    MemberAuthor

    @nikolmarku1 on the what now? What node/iojs version on what os are you running?

  15. jongyoonlee commented on Oct 26, 2015

    @jongyoonlee
    jongyoonlee
    on Oct 26, 2015

    I'm also seeing a similar issue on v4.2.1 stack running on heroku.

    Error: read ECONNRESET
    at exports._errnoException (util.js:874:11)
    at TLSWrap.onread (net.js:544:26)
    at TLSWrap.<anonymous> (/app/node_modules/continuation-local-storage/node_modules/async-listener/glue.js:188:31)

  16. rfink commented on Jan 6, 2016

    @rfink
    rfink
    on Jan 6, 2016

    Seeing on AWS 4.2.1 - inconsistently

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    freebsdIssues and PRs related to the FreeBSD platform.testIssues and PRs related to Node.js core tests and test infrastructure.

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions