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

Fix conversion of OMPI MCA params from legacy ORTE #1840

Merged
merged 2 commits into from
Nov 16, 2023

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Nov 16, 2023

Convert OMPI MCA params from legacy orte

Pickup and convert all "OMPI_MCA_orte_xxx" params
to their PRRTE equivalent. Note that some don't
have a PRRTE equivalent, so those will silently
be ignored as it would be too big a job to try
and detect and warn for them.

Signed-off-by: Ralph Castain [email protected]

Fix display of DVM allocation

Delay processing of PRRTE params until after the
selected schizo component gets a chance to translate
params (e.g., from OMPI to PRRTE). Ensure we detect
the "--display alloc" request and honor it for the
DVM itself.

Signed-off-by: Ralph Castain [email protected]

Pickup and convert all "OMPI_MCA_orte_xxx" params
to their PRRTE equivalent. Note that some don't
have a PRRTE equivalent, so those will silently
be ignored as it would be too big a job to try
and detect and warn for them.

Signed-off-by: Ralph Castain <[email protected]>
Delay processing of PRRTE params until after the
selected schizo component gets a chance to translate
params (e.g., from OMPI to PRRTE). Ensure we detect
the "--display alloc" request and honor it for the
DVM itself.

Signed-off-by: Ralph Castain <[email protected]>
@jsquyres
Copy link
Contributor

Thanks @rhc54!

FYI @mdosanjh

@rhc54 rhc54 merged commit 0bae0b4 into openpmix:master Nov 16, 2023
11 checks passed
@rhc54 rhc54 deleted the topic/convert branch November 16, 2023 19:38
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