-
Notifications
You must be signed in to change notification settings - Fork 6.4k
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
Datalabeling tests are broken #2643
Comments
@kurtisvg Do you want to take this or should I? I'm happy to investigate but I saw that you'd done the change referenced in the commit and I was auto-assigned. |
Happy to let you take over if you are willing - I was only attempting to update the snippets to use the test endpoint, which is how I discovered the tests were broken. |
I'll take a look, currently waiting on a few other work items. |
Anecdotally the tests are passing under nox in Python 2.7 and when directly invoked from Python 3.7 with pytest. |
As mentioned in the bug this does seem to be specific to running the tests in parallel. |
Repro in #2662 |
That branch got really messy, it appears to be a race condition in the teardown. |
Progress in #2665 I have serialized the tests in that branch to try and determine the ordering of the build-up / tear down / timing that is causing the problem. Still failing though. |
I was unable to resolve this, assigning to @kurtisvg |
@kurtisvg PTAL |
Closing in favor of buildcop issues. |
Data labeling test are broken - likely a race condition when using a single project.
See #2642
The text was updated successfully, but these errors were encountered: