Skip to content

Enable MPI pool on all CIs - #1224

Draft
rasolca wants to merge 1 commit into
masterfrom
ci/pool
Draft

rasolca wants to merge 1 commit into
masterfrom
ci/pool

Conversation

@rasolca

@rasolca rasolca commented Dec 2, 2024

Copy link
Copy Markdown
Collaborator

No description provided.

@rasolca
rasolca requested review from biddisco and msimberg December 2, 2024 15:53
@rasolca

rasolca commented Dec 2, 2024

Copy link
Copy Markdown
Collaborator Author

cscs-ci run

@msimberg

msimberg commented Dec 2, 2024

Copy link
Copy Markdown
Collaborator

Any particular reason? Debugging? Consistency? I have nothing much against this, but just looking to understand the reasoning (as I think it can be good to test with it both on and off).

@rasolca

rasolca commented Dec 3, 2024

Copy link
Copy Markdown
Collaborator Author

Noticed during rebase of #1192 that pool was enabled only on CUDA pipelines with no clear reason.

IMO:

  • Default behavior should be with pool.
  • If we want to test without pool we can add or modify some pipeline (and should be tested on both mc and gpu)

@msimberg msimberg left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Already discussed separately with @rasolca, but making the request explicit: I'd like to keep at least one CI configuration with the pool disabled just as a sanity check, since the pool is off by default.

@msimberg
msimberg marked this pull request as draft March 24, 2025 14:47
@msimberg msimberg moved this from In Progress to Todo in DLA-F Planning Sep 11, 2025

This branch has not been deployed

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

Projects

Status: Todo

Development

Successfully merging this pull request may close these issues.

2 participants