Skip to content

Conversation

@karasevb
Copy link
Member

Corresponds to #3978

Fixed #3977

@karasevb karasevb requested a review from artpol84 July 28, 2017 10:01
@karasevb karasevb added the bug label Jul 28, 2017
@karasevb karasevb added this to the v3.0.0 milestone Jul 28, 2017
@bwbarrett
Copy link
Member

Please don't file PRs against release branches until the corresponding PR has made its way through master. Then please file a PR against the release branch with a cherry-pick note in the commit message (git cherry-pick -x is your friend). You can remove the "pushed-back" label and we'll look at this PR when #3978 is in master and the commit has been updated appropriately.

@artpol84
Copy link
Contributor

@bwbarrett
Sorry for the mess, I didn't communicate properly the decisions we made on the dev meeting to @karasevb.

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.

When you cherry-pick this across, the changes should be good to go

@bwbarrett
Copy link
Member

@artpol84, no worries at all.

@karasevb, rhc merged your patch into master, so if you update this one with the cherry pick information, we should be good to go.

We've already cut 3.0.0rc2, but we'll make sure this is in 3.0.0 (ie, we'll get to do 3.0.0rc3).

@artpol84
Copy link
Contributor

@bwbarrett thank you, This will be done tonight.

This commit fixes a hang when using external PMIx v1 module

Signed-off-by: Boris Karasev <karasev.b@gmail.com>
(cherry picked from commit e20b581)
@karasevb karasevb force-pushed the v3.x_fix_hangs_pmix1 branch from 2ec32c4 to 082c39d Compare July 28, 2017 19:36
@artpol84
Copy link
Contributor

@bwbarrett should be ok now

@bwbarrett bwbarrett merged commit 936427a into open-mpi:v3.0.x Aug 1, 2017
@karasevb karasevb deleted the v3.x_fix_hangs_pmix1 branch March 21, 2018 02:34
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.

5 participants