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.

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

test: mark test-inspector flaky on windows #8835

Closed
wants to merge 1 commit into from

Conversation

Trott
Copy link
Member

@Trott Trott commented Sep 28, 2016

Checklist
  • make -j8 test (UNIX), or vcbuild test nosign (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

test

Description of change

@Trott Trott added the test Issues and PRs related to the tests. label Sep 28, 2016
@Trott
Copy link
Member Author

Trott commented Sep 28, 2016

@eugeneo
Copy link
Contributor

eugeneo commented Sep 28, 2016

LGTM, I'm planning to investigate the flakiness soon.

@mscdex mscdex added the inspector Issues and PRs related to the V8 inspector protocol label Sep 28, 2016
@MylesBorins MylesBorins mentioned this pull request Sep 28, 2016
2 tasks
@MylesBorins
Copy link
Contributor

MylesBorins commented Sep 28, 2016

LGTM

@Trott
Copy link
Member Author

Trott commented Sep 28, 2016

SmartOS build failure, but otherwise, CI is green. Windows TAP output has it marked as flaky.

@MylesBorins
Copy link
Contributor

SmartOS failure is infra related. I'm going to land this so we can move forward with the inspector backport

@MylesBorins
Copy link
Contributor

landed in 33dd4ec

MylesBorins pushed a commit that referenced this pull request Sep 29, 2016
PR-URL: #8835
Reviewed-By: Eugene Ostroukhov <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
MylesBorins pushed a commit to MylesBorins/node that referenced this pull request Sep 29, 2016
PR-URL: nodejs#8835
Reviewed-By: Eugene Ostroukhov <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
jasnell pushed a commit that referenced this pull request Sep 29, 2016
PR-URL: #8835
Reviewed-By: Eugene Ostroukhov <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
MylesBorins pushed a commit that referenced this pull request Sep 30, 2016
PR-URL: #8835
Reviewed-By: Eugene Ostroukhov <[email protected]>
Reviewed-By: Myles Borins <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@Trott Trott deleted the inspector-status branch January 13, 2022 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
inspector Issues and PRs related to the V8 inspector protocol test Issues and PRs related to the tests.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants