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

Assign values to remaining constants #394

Merged
merged 1 commit into from
Aug 11, 2022
Merged

Conversation

raffenet
Copy link
Contributor

Finish assigning values to constants as defined in OpenPMIx. This is
basically a revert of
a256e08
and
296edb4
from #359.

Refs #358.

Signed-off-by: Ken Raffenetti [email protected]

@raffenet raffenet added the RFC Request for Comment label Feb 15, 2022
@raffenet raffenet added this to the PMIx v5 Standard milestone Feb 15, 2022
@raffenet raffenet force-pushed the rfc/constants2 branch 2 times, most recently from 8dd83b8 to 2dc74c4 Compare February 16, 2022 22:22
@jjhursey
Copy link
Member

FYI: I rebased the branch now that PR #359 is merged. It should pass CI now.

@jjhursey jjhursey mentioned this pull request Mar 10, 2022
12 tasks
@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 jjhursey linked an issue Apr 12, 2022 that may be closed by this pull request
@jjhursey jjhursey added the Eligible Eligible for consideration by ASC label Apr 12, 2022
@jjhursey
Copy link
Member

PMIx ASC 2Q 2022

  • Passed the first vote: 11 yes / 0 no / 0 abstain
  • This is eligible for second vote in the 3Q 2022 ASC quarterly meeting

@jjhursey jjhursey added the First Vote Passed ASC first vote passed label May 12, 2022
@jjhursey
Copy link
Member

PMIx ASC 3Q 2022

  • Passed the second vote: 8 yes / 0 no / 0 abstain
  • This will be included in the next major release.

@jjhursey jjhursey added the Accepted as Stable ASC second vote passed. Accepted as Stable! label Aug 11, 2022
Finish assigning values to constants as defined in OpenPMIx. This is
basically a revert of
pmix@a256e08
and
pmix@296edb4
from pmix#359.

Refs pmix#358.

Signed-off-by: Ken Raffenetti <[email protected]>
@jjhursey
Copy link
Member

Rebased branch. No changes made.

@jjhursey jjhursey merged commit 1ab719d into pmix:master Aug 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted as Stable ASC second vote passed. Accepted as Stable! Eligible Eligible for consideration by ASC First Vote Passed ASC first vote passed RFC Request for Comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Constant values need to be defined
2 participants