Skip to content

test: failed on sequential/test-child-process-pass-fd #8219

Closed

Description

@yorkie
  • Version: master
  • Platform: smartos
  • Subsystem: test/ci, child_process

The CI in #8198 failed on sequential/test-child-process-pass-fd, which is added at db6253f.

By the way, that commit is to fix an issue on OSX, my question is why would we test it on smartos? is it exactly right behavior?

to mention the commit author @santigimeno

Activity

  1. added
    testIssues and PRs related to Node.js core tests and test infrastructure.
    on Aug 22, 2016
  2. added
    child_processIssues and PRs related to the child_process subsystem.
    on Aug 22, 2016
  3. santigimeno commented on Aug 22, 2016

    @santigimeno
    Member

    There's already an issue: #8209 and a PR: #8212 that tries to fix the issue.

    By the way, that commit is to fix an issue on OSX, my question is why would we test it on smartos? is it exactly right behavior?

    That commit fixed an issue on OS X by modifying when the passed descriptors are closed for every platform, so I think it makes sense to test it everywhere.

  4. added
    duplicateIssues and PRs that are duplicates of other issues or PRs.
    on Aug 22, 2016
  5. addaleax commented on Aug 22, 2016

    @addaleax
    Member

    I guess this can be closed as a duplicate then?

  6. yorkie commented on Aug 22, 2016

    @yorkie
    ContributorAuthor
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

    child_processIssues and PRs related to the child_process subsystem.duplicateIssues and PRs that are duplicates of other issues or PRs.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