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

Issue-372 Clarify PMIX_CMD_LINE in spawn case. #377

Merged
merged 1 commit into from
Mar 7, 2022

Conversation

dsolt
Copy link
Contributor

@dsolt dsolt commented Dec 24, 2021

Signed-of-by: [email protected]

rhc54
rhc54 previously approved these changes Dec 24, 2021
Copy link
Member

@rhc54 rhc54 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sounds right to me!

@dsolt dsolt force-pushed the issue_372 branch 2 times, most recently from 145069f to fb94787 Compare January 12, 2022 19:06
@jjhursey jjhursey added Clarification Eligible Eligible for consideration by ASC RFC Request for Comment labels Jan 25, 2022
@jjhursey
Copy link
Member

Please use emoji reactions ON THIS COMMENT to indicate your position on this proposal.

  • You do not need to vote on every proposal
  • If you have no opinion, don't vote - that is also useful data
  • If you've already commented on this issue, please still vote so
    we know your current thoughts
  • Not all proposals solve exactly the same problem, so we may end
    up accepting proposals that appear to have some overlap
    This is not a binding majority-rule vote, but it will be a very
    significant input into the corresponding ASC decision.

Here are the meanings for the emojis:

  • Hooray or Rocket: I support this so strongly that I
    want to be an advocate for it
  • Heart: I think this is an ideal solution
  • Thumbs up: I'd be happy with this solution
  • Confused: I'd rather we not do this, but I can tolerate it
  • Thumbs down: I'd be actively unhappy, and may even consider
    other technologies instead
    If you want to explain in more detail, feel free to add another
    comment, but please also vote on this comment.

@jjhursey
Copy link
Member

jjhursey commented Mar 7, 2022

PMIx ASC 1Q 2022

  • Passed the first vote: 11 yes / 0 no / 0 abstain
  • This is can be merged into main/v5 and v4 branches (I'll work on that)

@jjhursey jjhursey added First Vote Passed ASC first vote passed Errata labels Mar 7, 2022
jjhursey
jjhursey previously approved these changes Mar 7, 2022
@jjhursey
Copy link
Member

jjhursey commented Mar 7, 2022

Editorial: Rebased the reference branch without text changes to prepare to merge.

@jjhursey jjhursey merged commit 2791082 into pmix:master Mar 7, 2022
@jjhursey jjhursey mentioned this pull request Mar 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Clarification Eligible Eligible for consideration by ASC Errata First Vote Passed ASC first vote passed RFC Request for Comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

What should PMIX_CMD_LINE specify for a PMIx rank started with PMIx_spawn()
3 participants