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
Xfail failing test #6211
Xfail failing test #6211
Conversation
for more information, see https://pre-commit.ci
makes sense, feel free to merge as soon as the CI is green. That way, we can avoid having the CI fail in other PRs (this is not just a upstream-dev issue anymore) |
Can I suggest you xfail in xarray/xarray/tests/test_variable.py Line 2105 in 5470d93
(sorry, could have done this myself...) |
No prob, you do lots! |
We have another couple of failures though. I'll xfail them but please could someone who knows the dask interface better check them out and assess if we should raise them on the dask repo? |
for more information, see https://pre-commit.ci
I haven't seen them fail before - were they just flaky? |
More than a one-off — I got them multiple times. Though they look like tests that are liable to be flaky... |
RTD is being slow, but I'll merge regardless so we can return to green |
@@ -184,6 +184,7 @@ def test_dask_distributed_cfgrib_integration_test(loop) -> None: | |||
assert_allclose(actual, expected) | |||
|
|||
|
|||
@pytest.mark.xfail(reason="https://github.com/pydata/xarray/pull/6211") | |||
@gen_cluster(client=True) | |||
async def test_async(c, s, a, b) -> None: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@crusaderky can you take a look if you have the time?
They are both tests that fail on the cleanup of |
As per #6209 (comment)