Skip to content

Conversation

@jjhursey
Copy link
Member

@jjhursey jjhursey commented Feb 6, 2018

Signed-off-by: Ralph Castain <rhc@open-mpi.org>
(cherry picked from commit ce901ba)
@jjhursey jjhursey added the bug label Feb 6, 2018
@jjhursey jjhursey added this to the v3.1.0 milestone Feb 6, 2018
@jjhursey jjhursey requested a review from bosilca February 6, 2018 15:41
@jsquyres
Copy link
Member

jsquyres commented Feb 6, 2018

@jjhursey could you add commit b643852 to this PR?

@jjhursey
Copy link
Member Author

jjhursey commented Feb 6, 2018

It didn't apply cleanly, so I'll have to investigate a bit. I'll try to get it done today.

@bosilca
Copy link
Member

bosilca commented Feb 6, 2018

Outside the errors in the CI things looks good. I confirm that the original patch works in master.

@jjhursey
Copy link
Member Author

jjhursey commented Feb 7, 2018

@bosilca I tested the current v3.1.x HEAD without this patch and it seems to be working correctly.

Can you double check the commands below that I am trying to reproduce the same problem you are encountering.

[jjhursey@node03] orte-dvm --host node03:2,node04:2,node05:2 --system-server &
[1] 104250
DVM ready

[jjhursey@node03] prun -n 1 -H node05 ./bogus
[node03:104285] Job failed to spawn: The specified executable could not be executed
[jjhursey@node03] prun -npernode 1 -H node05 hostname
node05
[jjhursey@node03] prun -npernode 1 -H node04,node05 hostname
node05
node04
[jjhursey@node03] prun -n 1 -H node05 hostname
node05
[jjhursey@node03] prun -npernode 1 -H node04,node05 ./bogus
[node03:104351] Job failed to spawn: The specified executable could not be executed
[jjhursey@node03] prun -npernode 1 -H node04,node05 bogus
[node03:104357] Job failed to spawn: The specified executable could not be found
[jjhursey@node03] prun -npernode 1 -H node05 bogus
[node03:106523] Job failed to spawn: The specified executable could not be found
[jjhursey@node03]

@bosilca
Copy link
Member

bosilca commented Feb 8, 2018

I confirm this is not necessary on the current 3.1. Thanks @jjhursey for investigating.

@jjhursey
Copy link
Member Author

jjhursey commented Feb 8, 2018

Ok. I'll close this PR then, and the one for v3.0.x. Thank for the confirmation.

@jjhursey jjhursey closed this Feb 8, 2018
@jjhursey jjhursey deleted the fix/v3.1.x/badexe branch February 8, 2018 16:58
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.

3 participants