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

MAINT enable verbose pytest on Azure to debug #MercuryRetrograde stalled builds #22309

Closed
wants to merge 2 commits into from

Conversation

ogrisel
Copy link
Member

@ogrisel ogrisel commented Jan 27, 2022

Trying to understand why some builds stall on Azure Pipelines despite the merge of #22302 that disabled pytest-xdist.

Edit: example stalled test after the merge of #22302:

Stalled builds from this PR with verbosity enabled:

@ogrisel ogrisel changed the title MAINT enable verbose pytest on Azure to debug stalled builds MAINT enable verbose pytest on Azure to debug #MercuryRetrograde stalled builds Jan 27, 2022
@glemaitre
Copy link
Member

glemaitre commented Jan 27, 2022

I just saw that coverage has been updated from 6.2 to 6.3 on January 25.
It could be worth launching a build downgrading the version and with xdist just to check that this is not linked to this.

@ogrisel
Copy link
Member Author

ogrisel commented Jan 27, 2022

I just saw that coverage has been updated from 6.2 to 6.3 on January 25.
It could be worth to launch a build downgrading

Please feel free to do that in a parallel PR to avoid cancelling the current running jobs.

@glemaitre
Copy link
Member

It seems that this is not a particular test.

@glemaitre
Copy link
Member

I see the light...

nedbat/coveragepy#1310

@ogrisel
Copy link
Member Author

ogrisel commented Jan 27, 2022

Ok so based on the linked discussion, coverage 6.3 is no longer multiprocessing fork-safe. And indeed we have a few tests that try to use multiprocessing fork instead of loky and the verbose output of this PR seems to point to those tests (see the summary in the description).

I think this is a regression in coverage. +1 for pinning it for now. Let's see if #22311 passes.

@ogrisel
Copy link
Member Author

ogrisel commented Jan 27, 2022

Closing in favor of #22311.

@ogrisel ogrisel closed this Jan 27, 2022
@ogrisel ogrisel deleted the verbose-pytest branch January 27, 2022 09:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants