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

Fix the tree-spawn-with-rollup #4628

Merged
merged 1 commit into from
Dec 18, 2017
Merged

Fix the tree-spawn-with-rollup #4628

merged 1 commit into from
Dec 18, 2017

Conversation

rhc54
Copy link
Contributor

@rhc54 rhc54 commented Dec 16, 2017

Somehow, the code for passing a daemon's parent was accidentally removed, thus breaking the tree-spawn callback sequence and causing all daemons to phone directly home. Note that this is noticeably slower than no-tree-spawn for small clusters where directly ssh launch of the child daemons from the HNP doesn't overload the available file descriptors.

Signed-off-by: Ralph Castain rhc@open-mpi.org

Somehow, the code for passing a daemon's parent was accidentally removed, thus breaking the tree-spawn callback sequence and causing all daemons to phone directly home. Note that this is noticeably slower than no-tree-spawn for small clusters where directly ssh launch of the child daemons from the HNP doesn't overload the available file descriptors.

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
@rhc54
Copy link
Contributor Author

rhc54 commented Dec 16, 2017

@jjhursey @gpaulsen The IBM Jenkins is out of disk again!

@rhc54
Copy link
Contributor Author

rhc54 commented Dec 17, 2017

bot:ibm:retest

@open-mpi open-mpi deleted a comment from ibm-ompi Dec 17, 2017
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 17, 2017
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 17, 2017
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 18, 2017
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 18, 2017
@open-mpi open-mpi deleted a comment from ibm-ompi Dec 18, 2017
@rhc54
Copy link
Contributor Author

rhc54 commented Dec 18, 2017

bot:ibm:retest

@rhc54 rhc54 merged commit a863c26 into open-mpi:master Dec 18, 2017
@rhc54 rhc54 deleted the topic/treespawn branch December 18, 2017 14:48
@jjhursey
Copy link
Member

@rhc54 Will we need this in the release branches?

@rhc54
Copy link
Contributor Author

rhc54 commented Dec 19, 2017

Looks like it needs to go to v3.0.x and v3.1.x

@rhc54
Copy link
Contributor Author

rhc54 commented Dec 19, 2017

Should add #4637 when it is complete

@rhc54
Copy link
Contributor Author

rhc54 commented Dec 19, 2017

Also need #4627 and its follow-on

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