prun: propagate app exit code to caller? about prrte HOT 18 CLOSED

Could you please give the referenced PR a try and see if it works for you?

from prrte.

It's still returning 0 no matter what the app does. Looks like it's actually saying "I started the app OK".

from prrte.

Think I chased it down - can you try again?

from prrte.

Thanks. Almost, but not quite :) If I exit with some +ve value, I do get that value back, but exit(0) leads to prun exiting with status "1".

from prrte.

Grrr...ok, try this one (had wrong default value)

from prrte.

Looks like that might be it. Thanks! But will test a bit more. Update: looks good.

from prrte.

Please reopen (or open a new one) if you find a continuing problem

from prrte.

Unwanted behaviour has reappeared. Exit status from prun is always "1" (where tested) when end-user app does "exit(anything)".

from prrte.

Rats - okay, will fix. Also need to take that fix upstream so we don't keep getting overwritten on sync.

from prrte.

Odd - I'm not replicating this. Can you perhaps check to ensure that you are at the current head? If so, then is there anything about your configure or execution that is unusual (e.g., parameters)?

from prrte.

(In real code, PMIx is master @ 58b2593abdfd8959d1eba0b4fd56758b9bca3afa)

PRRTE is master @ fa56e0b

MWE:

int main() { return 23; }

running bare:

$ ./a.out; echo $?
23

running through prun/prte:

$ prte ...
DVM ready


$ prun -n 1 ./a.out; echo $?
1

from prrte.

I'm not seeing it - using your provided test case:

$ prun -n 1 ./foo
$ echo $?
23

Our hashes look the same, so I'm confused. Can you check to ensure that you are getting the prte and prun you expected? Also, note that you had to rerun autogen/configure when updating due to a removed component in PRRTE plus configure updates.

from prrte.

Always do autogen each install. Am rm-rf'ing source and installed directories, re-cloning from github in case of bit-rot.

from prrte.

OK, re-clone anew, git HEAD is same hash: autogen, configure, make install, still seeing exit = 1. Correct (only) prte/prun are first on PATH.

from prrte.

Huh - I re-cloned and was finally able to replicate it. Will have to see what is perhaps lingering on my local repo.

from prrte.

Okay, issue was somehow something to do with the branch of PMIx master I'm using. Once that branch passes CI, I'll commit it to PMIx master - should do the trick.

from prrte.

You might try refreshing pmix master checkout now and see if it works. I'm rebuilding myself just to be sure

from prrte.

Looks good, thanks!

from prrte.