Skip to content
This repository has been archived by the owner on Sep 30, 2022. It is now read-only.

fix spawn on heterogeneous clusters #977

Merged
merged 2 commits into from
Mar 8, 2016

Conversation

ggouaillardet
Copy link
Contributor

fix spawn in heterogeneous clusters by retrieving the remote arch

and accept a proc from a different job

(cherry picked from commit open-mpi/opmi@a4aa4c95715ea7ec7922b7c3f3550aebd1dad3d0)
also remove code duplication by using ompi_proc_complete_init_single()

Thanks Siegmar Gross for reporting this issue, and Ralph for the guidance.

(cherry picked from commit open-mpi/ompi@308bbcb)
@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1359/ for details.

@lanl-ompi
Copy link
Contributor

Test FAILed.

2 similar comments
@lanl-ompi
Copy link
Contributor

Test FAILed.

@lanl-ompi
Copy link
Contributor

Test FAILed.

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1360/ for details.

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1361/ for details.

@mellanox-github
Copy link

Test FAILed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1362/ for details.

@ggouaillardet
Copy link
Contributor Author

:bot:assign: @rhc54
:bot:milestone:v2.0.0
:bot🏷️bug

@ompiteam-bot ompiteam-bot added this to the v2.0.0 milestone Feb 22, 2016
@jsquyres
Copy link
Member

jsquyres commented Mar 2, 2016

bot:retest

@jsquyres
Copy link
Member

jsquyres commented Mar 2, 2016

@ggouaillardet Can you supply a proper title and description for this PR?

@ggouaillardet ggouaillardet changed the title Topic/v2.x/hetero spawn fix spawn on heterogeneous clusters Mar 2, 2016
@ggouaillardet
Copy link
Contributor Author

done

@mellanox-github
Copy link

Test PASSed.
See http://bgate.mellanox.com/jenkins/job/gh-ompi-release-pr/1392/ for details.

@jsquyres
Copy link
Member

jsquyres commented Mar 7, 2016

@rhc54 Can you have a look at this one, too?

@rhc54 rhc54 added the reviewed label Mar 8, 2016
@rhc54
Copy link

rhc54 commented Mar 8, 2016

@jsquyres Looks fine to me

jsquyres added a commit that referenced this pull request Mar 8, 2016
@jsquyres jsquyres merged commit bd3d6ff into open-mpi:v2.x Mar 8, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants