Skip to content

Conversation

@janjust
Copy link
Contributor

@janjust janjust commented May 5, 2025

Update pmix and prrte pointers
pmix: bc3680a1 (v5.0.8)
prrte: c4f6f78d (v3.0.10)

bot:notacherrypick

@github-actions github-actions bot added this to the v5.0.8 milestone May 5, 2025
@janjust janjust requested review from jsquyres and removed request for jsquyres May 8, 2025 16:54
Signed-off-by: Tomislav Janjusic <[email protected]>
@janjust
Copy link
Contributor Author

janjust commented May 8, 2025

bot:retest

@janjust janjust requested a review from bosilca May 8, 2025 21:27
@janjust janjust merged commit 97c6e41 into open-mpi:v5.0.x May 9, 2025
15 checks passed
@shijin-aws
Copy link
Contributor

@janjust I have seen this PR caused mpirun --version output got directed to stderr instead of stdout. Do you have any insights how this can happen?

@rhc54
Copy link
Contributor

rhc54 commented May 10, 2025

Nothing changed in the output code - are you saying you just saw this, or are you saying it changed? FWIW: the code has always output to stderr in the PRRTE v3 series

@rhc54
Copy link
Contributor

rhc54 commented May 10, 2025

Actually, I take that back - there is a change in the current v5.0 branch head that switches to stderr from stdout. I'll put it back as there is no value in doing the switch.

@rhc54
Copy link
Contributor

rhc54 commented May 10, 2025

Fixed in the official PMIx v5.0.8 release (https://github.com/openpmix/openpmix/releases/tag/v5.0.8)

@shijin-aws
Copy link
Contributor

shijin-aws commented May 12, 2025

Nothing changed in the output code - are you saying you just saw this, or are you saying it changed? FWIW: the code has always output to stderr in the PRRTE v3 series

Yes, I just saw this, it used to be in stdout but switched to stderr.

Fixed in the official PMIx v5.0.8 release (https://github.com/openpmix/openpmix/releases/tag/v5.0.8)

Thanks! I will wait for this new pmix ingestion to ompi @janjust will this happen for this ompi5 release?

@janjust
Copy link
Contributor Author

janjust commented May 12, 2025

@shijin-aws I'll update the ptrs to the official release today - with a short interval between rc2 (today) and release probably friday

@shijin-aws
Copy link
Contributor

@shijin-aws I'll update the ptrs to the official release today - with a short interval between rc2 (today) and release probably friday

Thanks a lot

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.

4 participants