-
Notifications
You must be signed in to change notification settings - Fork 257
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
Fix599 #605
Fix599 #605
Conversation
…age_watchers" any more
I'm seeing a lot more test failures with this change. I just ran 4 times with this code and got 30 or 33 errors each time. Then I switched to the prior version and all tests failed. Switching back to this resulted in 30 errors and 1 failure again. I'll paste my results. |
test_add_watcher (circus.tests.test_arbiter.TestArbiter) ... ok ERROR: test_watchdog_discovery_not_found (circus.tests.test_plugin_watchdog.TestPluginWatchDog)Traceback (most recent call last): FAIL: test_watchdog_discovery_found (circus.tests.test_plugin_watchdog.TestPluginWatchDog)Traceback (most recent call last): |
Every error is "Address already in use" except for test_watchdog_discovery_found. |
Is it Py3 specific (no errors at all here (osx with Py2.7) ? |
I'll check. |
Same result with Py2.7. This is running what is on master right now. |
Got the 33 errors and 1 failure this time. Seems to be pretty reliably either 30 or 33 errors and 1 failure every time. |
Just got a clean run with Py2.6 |
Py2.6 ran clean twice. Switched back to Py2.7 and failed again. Have not gotten any clean runs with Py2.7 or Py3.3. |
And same failure on Py3.2 |
Can you skip test_watchdog_discovery_found and test another time ? |
Did that and got failures from test_plugin_resource_watcher. Commented out TestResourceWatcher and all ran without error. |
That is on 3.3 and 2.7 |
Yes ! I reproduce your issue with test_watchdog_discovery_found on Py3.3 (only). Working on it. |
In TestResourceWatcher the problem is test_resource_watcher_min_cpu. If I comment out that test, everything works. Even with test_watchdog_discovery_found turned back on! Strange. |
I am also on OS X, BTW. 10.9. |
@scottkmaxwell all fixed for me (Py33 and Py27) with #607, can you test ? |
Looks good on Py2.7. Will try on Py3.3 as well. |
Py3.3 looks good too! |
Great ! travis report was good too: https://travis-ci.org/mozilla-services/circus/builds/13059740 Just pushed another commit on this branch to fix latest errors ? |
Fix #599 by bypassing manage_watchers periodic callback by default in tests
The bypass is made by check_delay < 0
Note: only one test requires the periodic callback (check_delay > 0 forced on this one)