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

Update submodule pointers - do not commit #10788

Closed
wants to merge 1 commit into from

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Sep 10, 2022

Just testing the configuration changes in PMIx/PRRTE

Signed-off-by: Ralph Castain rhc@pmix.org

Signed-off-by: Ralph Castain <rhc@pmix.org>
@ompiteam-bot
Copy link

Can one of the admins verify this patch?

@jsquyres
Copy link
Member

ok to test

@rhc54
Copy link
Contributor Author

rhc54 commented Sep 11, 2022

@janjust The Mellonox CI is failing on test of an MCA param --mca mca_base_env_list 'XXX_A=1;XXX_B=2;XXX_C;XXX_D;XXX_E'. I suspect this was relying on PMIx to pick it up, which it no longer does as that MCA param has nothing to do with PMIx.

Is this a behavior you wish to preserve? You might need to pick it up in the schizo/ompi component. Note that we do pick up the same behavior from the cmd line in PRRTE - just not as an MCA param.

@rhc54
Copy link
Contributor Author

rhc54 commented Sep 13, 2022

@jsquyres @gpaulsen Can you please bring this up on today's telecon? You folks need to decide what to do about Mellanox CI and the path forward so you can commit PRRTE/PMIx updates.

@gpaulsen
Copy link
Member

@rhc54 Yes, we're discussing and will discuss at today's Open MPI call.

@gpaulsen
Copy link
Member

Our consensus was that we'll implement something in schizo/ompi component for v5.
Shorter term, mellanox will remove the test(s) so that we can update our submodule pointers until we have the new schizo/ompi code, then they will re-enable the mca_base_env_list tests to test long term.

@rhc54
Copy link
Contributor Author

rhc54 commented Sep 13, 2022

Okay, thx! I'll close this as it appears everything else is fine.

@rhc54 rhc54 closed this Sep 13, 2022
@rhc54 rhc54 deleted the topic/test branch September 13, 2022 16:02
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.

5 participants