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 PMIx to v2.1.0 #4605

Closed
rhc54 opened this issue Dec 11, 2017 · 11 comments
Closed

Update PMIx to v2.1.0 #4605

rhc54 opened this issue Dec 11, 2017 · 11 comments

Comments

@rhc54
Copy link
Contributor

rhc54 commented Dec 11, 2017

Need to refresh PMIx in OMPI v3.1.0 prior to release - should only be an internal-to-PMIx update, but needed to ensure backward compatibility with earlier PMIx and OMPI releases.

Will try to complete ASAP.

@rhc54
Copy link
Contributor Author

rhc54 commented Dec 11, 2017

Waiting on openpmix/openpmix#586

@rhc54
Copy link
Contributor Author

rhc54 commented Jan 11, 2018

@bwbarrett @jsquyres I was going to provide a vanilla version of the official PMIx v2.1.0 release. However, it just hit me that the cleanup registration code won't be in that release - it involved adding attributes etc that moved it into PMIx v3.

Do you want the cleanup code here? It solves the problem of leftover files in /dev/shm, which is why I ask. Would mean taking the embedded pmix code currently in master.

@karasevb
Copy link
Member

openpmix/openpmix#586 was closed, and related PRs are merged openpmix/openpmix#613, openpmix/openpmix#614.

@karasevb
Copy link
Member

@rhc54 Do we have any other blockers to update PMIx?

@rhc54
Copy link
Contributor Author

rhc54 commented Jan 13, 2018

Not that I know of, but I do need to resolve the question I posted above.

@karasevb
Copy link
Member

@rhc54 can we sync PMIx v2.1 early than mentioned feature will be resolved? Since we faced the failure during testing with internal PMIx which was fixed here openpmix/openpmix#614.

@rhc54
Copy link
Contributor Author

rhc54 commented Jan 15, 2018

I'm afraid Monday is a national holiday over here, so I don't know who will be around. I suspect this will at least need to wait until Tues or Wed before anything could be committed, regardless.

@artpol84
Copy link
Contributor

@rhc54 Let's discuss this on Todays call.

@jladd-mlnx jladd-mlnx assigned karasevb and unassigned rhc54 Jan 23, 2018
@jladd-mlnx
Copy link
Member

@karasevb since Ralph is no longer working on the OMPI project, please do the merge.

@alinask @artpol84 FYI...

@karasevb
Copy link
Member

Done #4746

@bwbarrett
Copy link
Member

We've merged 2.1 (and now 2.1.1) into 3.1, so closing this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants