Skip to content

Use config_for_cluster_tests in sync cluster#8180

Merged
fjetter merged 1 commit into
dask:mainfrom
crusaderky:sync_cluster_fixture
Sep 12, 2023
Merged

Use config_for_cluster_tests in sync cluster#8180
fjetter merged 1 commit into
dask:mainfrom
crusaderky:sync_cluster_fixture

Conversation

@crusaderky

@crusaderky crusaderky commented Sep 11, 2023

Copy link
Copy Markdown
Collaborator

Use test-specific config overrides in workers and nannies started by thecluster context manager, as well as in the a, b pytest fixtures (not to be confused with a, b of @gen_cluster)

@crusaderky crusaderky requested a review from fjetter as a code owner September 11, 2023 16:46
@crusaderky crusaderky changed the title Use config_for_cluster_tests in sync fixture Use config_for_cluster_tests in sync cluster Sep 11, 2023
@github-actions

Copy link
Copy Markdown
Contributor

Unit Test Results

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

       21 files  ±  0         21 suites  ±0   10h 24m 28s ⏱️ - 27m 8s
  3 816 tests +  2    3 706 ✔️ +  3     107 💤 ±0  3  - 1 
36 887 runs  +20  35 084 ✔️ +20  1 800 💤 +1  3  - 1 

For more details on these failures, see this check.

Results for commit 56b8f37. ± Comparison against base commit 7744d68.

@fjetter fjetter merged commit 9895194 into dask:main Sep 12, 2023
@crusaderky crusaderky deleted the sync_cluster_fixture branch September 12, 2023 12:11
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.

2 participants