Skip to content

Add WorkerState.all_running_tasks#6690

Merged
crusaderky merged 4 commits intodask:mainfrom
hendrikmakait:all-running-tasks
Jul 8, 2022
Merged

Add WorkerState.all_running_tasks#6690
crusaderky merged 4 commits intodask:mainfrom
hendrikmakait:all-running-tasks

Conversation

@hendrikmakait
Copy link
Copy Markdown
Member

@hendrikmakait hendrikmakait commented Jul 7, 2022

Addresses #6676 (comment)

  • Tests added / passed
  • Passes pre-commit run --all-files

assert task in ws.all_running_tasks


@pytest.mark.xfail(reason="distributed#6565")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

XFAIL: #6565

assert task not in ws.all_running_tasks


@pytest.mark.xfail(reason="distributed#6565")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

XFAIL: #6565

assert task not in ws.all_running_tasks


@pytest.mark.xfail(reason="distributed#6565")
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

XFAIL: #6565

@hendrikmakait hendrikmakait self-assigned this Jul 7, 2022
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jul 7, 2022

Unit Test Results

See test report for an extended history of previous test failures. This is useful for diagnosing flaky tests.

       15 files  ±  0         15 suites  ±0   6h 21m 28s ⏱️ - 3m 10s
  2 931 tests +10    2 845 ✔️ +  5    84 💤 +  4  0  - 1  2 🔥 +2 
21 712 runs  +85  20 723 ✔️ +38  986 💤 +45  1 ±0  2 🔥 +2 

For more details on these errors, see this check.

Results for commit 44e54dc. ± Comparison against base commit bec621e.

♻️ This comment has been updated with latest results.

@crusaderky
Copy link
Copy Markdown
Collaborator

Could you review the tests now that #6687 has been merged?

hendrikmakait and others added 2 commits July 8, 2022 09:31
Co-authored-by: crusaderky <crusaderky@gmail.com>
@hendrikmakait hendrikmakait requested a review from crusaderky July 8, 2022 07:56
@crusaderky crusaderky merged commit ade4266 into dask:main Jul 8, 2022
@crusaderky
Copy link
Copy Markdown
Collaborator

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants