Skip to content

Conversation

@artpol84
Copy link
Contributor

send_error() is only packing status and peer info in the reply.
While remote counterpart in pmix_server_dmdx_resp() expects
the "hotel room number" to proceed correctly.

Signed-off-by: Artem Polyakov artpol84@gmail.com

@artpol84
Copy link
Contributor Author

Address #3941

@artpol84 artpol84 requested a review from rhc54 July 20, 2017 16:48
@artpol84 artpol84 self-assigned this Jul 20, 2017
@artpol84 artpol84 added the bug label Jul 20, 2017
`send_error()` is only packing status and peer info in the reply.
While remote counterpart in `pmix_server_dmdx_resp()` expects
the "hotel room number" to proceed correctly.

Signed-off-by: Artem Polyakov <artpol84@gmail.com>
@artpol84 artpol84 force-pushed the dmdx_err_resp/master branch from d6c45b6 to 79c10c8 Compare July 20, 2017 16:51
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.

Looks good - thanks!

@rhc54 rhc54 merged commit d34dbf1 into open-mpi:master Jul 20, 2017
@rhc54 rhc54 deleted the dmdx_err_resp/master branch July 20, 2017 17:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants