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

Investigate flaky test-fs-watch-encoding #7243

Closed
joaocgreis opened this issue Jun 9, 2016 · 10 comments
Closed

Investigate flaky test-fs-watch-encoding #7243

joaocgreis opened this issue Jun 9, 2016 · 10 comments
Labels
fs Issues and PRs related to the fs subsystem / file system. macos Issues and PRs related to the macOS platform / OSX. test Issues and PRs related to the tests.

Comments

@joaocgreis
Copy link
Member

joaocgreis commented Jun 9, 2016

  • Version: unrelated change rebased on master
  • Platform: osx1010
  • Subsystem: test

https://ci.nodejs.org/job/node-test-commit-osx/3726/nodes=osx1010/tapTestReport/test.tap-598/

not ok 598 parallel/test-fs-watch-encoding
# TIMEOUT

Test source: https://github.com/nodejs/node/blob/master/test/parallel/test-fs-watch-encoding.js

cc @nodejs/testing

@joaocgreis joaocgreis added fs Issues and PRs related to the fs subsystem / file system. test Issues and PRs related to the tests. macos Issues and PRs related to the macOS platform / OSX. labels Jun 9, 2016
@santigimeno
Copy link
Member

It's the same issue as: #4629.

@Trott
Copy link
Member

Trott commented Jun 16, 2016

Stress test results are surprising. Maybe I'm doing something wrong?: https://ci.nodejs.org/job/node-stress-single-test/755/nodes=osx1010/console

@Trott
Copy link
Member

Trott commented Jun 16, 2016

OK, looks like the test just isn't calling common.refreshTmpDir(). That's the issue with the stress test results. But it's still flaky and needs a timer on OS X too. PR coming in a few minutes....

@Trott
Copy link
Member

Trott commented Jun 16, 2016

Stress test with tmp dir being refreshed: https://ci.nodejs.org/job/node-stress-single-test/756/nodes=osx1010/console

@Trott
Copy link
Member

Trott commented Jun 16, 2016

Hmmm, no failure in that stress test with the refreshTmpDir() added. That surprises me. Let me try upping the number of runs from 999 to 9999. If it still doesn't fail, maybe declare it fixed, at least until we see it again: https://ci.nodejs.org/job/node-stress-single-test/757/nodes=osx1010/console

@santigimeno
Copy link
Member

FWIW I can consistently reproduce the issue locally even after adding refreshTmpDir(). OS X version 10.11.5

@Trott
Copy link
Member

Trott commented Jun 17, 2016

@santigimeno How are you running it? (Via tools.py or directly with node? Current master or v6.2.1 or something else? Anything else that might be relevant...)

I'm on 10.10.5 and unable to reproduce, but maybe I'm just not running it enough times or something?

@santigimeno
Copy link
Member

@Trott Current master. I can easily reproduce it using this script nodejs/testing#3 (comment). Also, running the test suite non-stop for a while reproduces the issue after a while (1 hour or so)

@Trott
Copy link
Member

Trott commented Jul 2, 2016

Is this still a problem now that #7356 landed 9 days ago?

@Trott
Copy link
Member

Trott commented Jul 4, 2016

I'm unable to replicate this problem now since 5267f29 landed. If I revert 5267f29, then I can easily replicate this problem.

So...I'm going to close this. Of course, feel free to re-open and/or comment if I am in error.

@Trott Trott closed this as completed Jul 4, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fs Issues and PRs related to the fs subsystem / file system. macos Issues and PRs related to the macOS platform / OSX. test Issues and PRs related to the tests.
Projects
None yet
Development

No branches or pull requests

3 participants