Skip to content

Conversation

@mmarchini
Copy link
Contributor

test-trace-events-fs-sync has been failing ocasinally on FreeBSD. While
we don't have a fix, it should be marked as flaky.

Ref: #21038

Checklist

test-trace-events-fs-sync has been failing ocasinally on FreeBSD. While
we don't have a fix, it should be marked as flaky.

Ref: nodejs#21038
@nodejs-github-bot nodejs-github-bot added the test Issues and PRs related to the tests. label May 30, 2018
@mmarchini mmarchini added the flaky-test Issues and PRs related to the tests with unstable failures on the CI. label May 30, 2018
@joyeecheung
Copy link
Member

Should we fast-track this?

Copy link
Contributor

@apapirovski apapirovski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1 to fast-tracking

@apapirovski apapirovski added the fast-track PRs that do not need to wait for 48 hours to land. label May 31, 2018
@lance
Copy link
Member

lance commented Jun 4, 2018

@lance
Copy link
Member

lance commented Jun 4, 2018

Landed in 5aa47bd

@lance lance closed this Jun 4, 2018
lance pushed a commit that referenced this pull request Jun 4, 2018
test-trace-events-fs-sync has been failing ocasinally on FreeBSD. While
we don't have a fix, it should be marked as flaky.

Ref: #21038

PR-URL: #21039
Refs: #21038
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Lance Ball <lball@redhat.com>
MylesBorins pushed a commit that referenced this pull request Jun 6, 2018
test-trace-events-fs-sync has been failing ocasinally on FreeBSD. While
we don't have a fix, it should be marked as flaky.

Ref: #21038

PR-URL: #21039
Refs: #21038
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Anatoli Papirovski <apapirovski@mac.com>
Reviewed-By: Lance Ball <lball@redhat.com>
@MylesBorins MylesBorins mentioned this pull request Jun 6, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fast-track PRs that do not need to wait for 48 hours to land. flaky-test Issues and PRs related to the tests with unstable failures on the CI. test Issues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants