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

v5: Fix singleton spawn #10716

Merged
merged 3 commits into from
Aug 29, 2022
Merged

v5: Fix singleton spawn #10716

merged 3 commits into from
Aug 29, 2022

Conversation

jjhursey
Copy link
Member

No description provided.

jjhursey and others added 3 commits August 25, 2022 10:30
 * See openpmix/openpmix#2705

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 245424c)
 * Fixes open-mpi#10590
 * Singletons will not have a PMIx value for `PMIX_LOCAL_PEERS`
   so make that optional instead of required.
 * `&` is being confused as an application argument in `prte`
   instead of the background character
   * Replace with `--daemonize` which is probably better anyway

Signed-off-by: Joshua Hursey <[email protected]>
(cherry picked from commit 16a1fa6)
When doing a comm_spawn as a singleton, you need to instruct
the "prte" DVM to forward any IO from the child job to the
singleton "parent" so it can be properly output. Otherwise,
the IO from the child job will simply be lost.

Signed-off-by: Ralph Castain <[email protected]>
(cherry picked from commit 48dbd1d)
@awlauria
Copy link
Contributor

bot:aws:retest

@awlauria awlauria merged commit 2ade147 into open-mpi:v5.0.x Aug 29, 2022
@jjhursey jjhursey deleted the v5-fix-spawn branch August 29, 2022 14:15
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.

3 participants