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

Allow the dpm to query the server for local peer info #12920

Merged
merged 1 commit into from
Nov 19, 2024

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Nov 14, 2024

The local PMIx server may have info on the local peers for a job - so allow the client to request that info if it doesn't already possess it.

Also clean up some diagnostic output so it shows the PMIx attribute name (or just the provided string if it isn't an attribute name) to make it easier to correlate to the code.

The local PMIx server may have info on the local peers
for a job - so allow the client to request that info
if it doesn't already possess it.

Also clean up some diagnostic output so it shows the
PMIx attribute name (or just the provided string if it
isn't an attribute name) to make it easier to correlate
to the code.

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

rhc54 commented Nov 14, 2024

Not sure I grok the mpi4py failure - can't see what it has to do with this change. 🤷‍♂️ Have to leave that to you guys.

@rhc54
Copy link
Contributor Author

rhc54 commented Nov 19, 2024

Hey folks - I have no idea why mpi4py would be failing, but it doesn't seem possible that it stems from this change. Is there some way we can get this committed so it can go to the OMPI v5.0 branch?

@bosilca bosilca merged commit 594e29a into open-mpi:main Nov 19, 2024
14 of 15 checks passed
@rhc54 rhc54 deleted the topic/dpm branch December 2, 2024 20:21
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.

2 participants