-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
CI: limit pytest version on 3.6 #27416
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
Conversation
Any idea what the root cause is? |
we had set this previously on some of the 3.6 builds, no idea really why its actually failing. |
@TomAugspurger https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=14549 is failing the numpydev build even with an older pytest; so this is issue is solved for the 3.6 builds, but any idea about this ? |
The primary difference is that the numpydev build uses
are being elevated to errors, causing some internal purest error? I'm not sure where they're coming from, but perhaps adding a |
Not at a computer to check but are those warnings coming from the IO Excel tests? Think I’ve seen them before |
Probably not excel, unless they open files before being skipped. That job has pretty minimal dependencies. I'm not sure if the INTERTNALERROR from pytest is printed inline with the test output, or whether it's buffered till the end :/
|
Not sure this is related but comparing the environments between the broken builds and the previous working ones there's a diff in bzip2 version, with it previously being 1.0.7 on the working ones and 1.0.8 on the broken ones. Didn't see another difference save pandas |
Sorry can ignore previous comment for now; I thought still looking at Travis failures but those are just timeouts now due to verbosity. Looking at np37 too to see if there's anything that sticks out |
this works for me on a local build. (on mac), so must be a different package picked up on linux. |
The only diff in deps https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=14539 and https://dev.azure.com/pandas-dev/pandas/_build/results?buildId=14544 is pytest-azure-pipelines. Going to try pinning that to the old value of 0.8.0. |
Oh, sorry two different. The failing build also has pytest-nunit (maybe a pytest-azure-pipelines dep?) |
thanks @TomAugspurger |
closes #27406