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

Conversation

@rhc54
Copy link

@rhc54 rhc54 commented Mar 2, 2016

The link from a PMIx comm failure back to the ORTE errmgr had been broken. Reconnect it so that the app's errmgr can abort upon loss of connection to the local daemon

@jsquyres The problem was that the call to PMIx_Register_errhandler wasn't implemented, and so we never connected to the PMIx errhandler to the ORTE errmgr even though we were registering it. This implements the registration code so the connection gets made.

Fixes open-mpi/ompi#1418

…The link from a PMIx comm failure back to the ORTE errmgr had been broken. Reconnect it so that the app's errmgr can abort upon loss of connection to the local daemon
@mellanox-github
Copy link

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

@jsquyres
Copy link
Member

jsquyres commented Mar 3, 2016

@gpaulsen Github pro tip: it is usually better to comment in the "Files changed" view on the PR itself.

@jsquyres
Copy link
Member

jsquyres commented Mar 3, 2016

@gpaulsen Gave this a 👍

hppritcha added a commit that referenced this pull request Mar 6, 2016
Repair the application suicide code for 2.0 - requires custom patch.
@hppritcha hppritcha merged commit ec2b0cf into open-mpi:v2.x Mar 6, 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.

5 participants