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

Revision section for v5 #412

Merged
merged 1 commit into from
Jul 18, 2022
Merged

Conversation

jjhursey
Copy link
Member

No description provided.

Signed-off-by: Joshua Hursey <[email protected]>
@jjhursey jjhursey added this to the PMIx v5 Standard milestone May 26, 2022
@jjhursey jjhursey requested review from raffenet and dsolt May 26, 2022 17:58
@jjhursey
Copy link
Member Author

  • In PR Issue 175 make query chap -rebased on master #407 we removed PMIX_QUERY_PARTIAL_SUCCESS
    %%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%%
    %% NOTE: This is not used anywhere. If this is supposed to be returned by
    %% the query API's, it was never indicated. They currently return
    %% PMIX_ERR_PARTIAL_SUCCESS
    %%\subsection{Query-specific constants}
    %%\label{api:struct:constants:query}
    %%\begin{constantdesc}
    %%%
    %%\declareconstitemvalue{PMIX_QUERY_PARTIAL_SUCCESS}{-104}
    %%Some, but not all, of the requested information was returned.
    %%%
    %%\end{constantdesc}

I wanted to capture that removal in the Revision History and noticed that we don't have a v5 section. The RMs will need to fill this out based on what has been merged to master and not to v4. This should get you all started with a framework.

@jjhursey
Copy link
Member Author

@raffenet @dsolt Can you give this a review so we can merge?

Copy link
Contributor

@raffenet raffenet left a comment

Choose a reason for hiding this comment

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

I suppose @dsolt and I should go about populating this too 😄.

@jjhursey jjhursey merged commit b121640 into pmix:master Jul 18, 2022
@jjhursey jjhursey deleted the revision-list-for-v5 branch July 18, 2022 22:00
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