Skip to content

pmix/pmix112: Update to PMIx 1.2.1 release #2958

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Feb 21, 2017
Merged

pmix/pmix112: Update to PMIx 1.2.1 release #2958

merged 1 commit into from
Feb 21, 2017

Conversation

jjhursey
Copy link
Member

@jjhursey jjhursey commented Feb 10, 2017 •
edited
Loading

@jjhursey jjhursey added this to the v2.1.0 milestone Feb 10, 2017
@jjhursey jjhursey requested a review from rhc54 February 10, 2017 23:21
@hppritcha
Copy link
Member

let's try again with Mellanox bot:

bot:mellanox:retest

rhc54
rhc54 approved these changes Feb 14, 2017
View reviewed changes
Copy link
Contributor

@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.

Impossible for me to review this line for line, but it appears to be a valid PMIx v1.2.1 tarball

@jjhursey
Copy link
Member Author

@hjelmn This PR contains the latest PMIx if you want to give it a try at scale again.

@hppritcha hppritcha mentioned this pull request Feb 14, 2017
Merged
@hppritcha
Copy link
Member

@hjelmn have you had a chance to test this PR?

@jjhursey
Copy link
Member Author

@hjelmn Just checking in to see if you had a chance to test. I think your 👍 is the last item before we release PMIx 1.2.1.

@jjhursey jjhursey force-pushed the topic/v2.x-pmix-1.2.1 branch from fc83a91 to 4b67318 Compare February 17, 2017 20:50
@jjhursey
Copy link
Member Author

We are waiting on a small configure change, then we are planning to release PMIx v1.2.1 today.

Once that is officially released, I'll update this PR and we can merge it in. Once merged in then MTT can work on it for a few days before OMPI v2.1.0 release.

@jjhursey jjhursey force-pushed the topic/v2.x-pmix-1.2.1 branch from 4b67318 to 0933ed6 Compare February 21, 2017 17:40
@jjhursey
Copy link
Member Author

PMIx v1.2.1 is now officially released. I updated this PR.
I'll let @rhc54 re-review it. @hppritcha @jsquyres This should be ready to go once that is complete.

@jsquyres
Copy link
Member

@hppritcha Since this is the feature for v2.1.0, I'm going to merge this when CI finishes (even though you're unavailable).

@jsquyres jsquyres merged commit c24ba53 into open-mpi:v2.x Feb 21, 2017
@jjhursey jjhursey deleted the topic/v2.x-pmix-1.2.1 branch February 23, 2017 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rhc54 rhc54 rhc54 approved these changes

Assignees

@rhc54 rhc54

Labels
bug enhancement RM approved
Projects
None yet
Milestone
v2.1.0
Development

Successfully merging this pull request may close these issues.
4 participants
@jjhursey @hppritcha @jsquyres @rhc54