hautreux/slurm — comment on commit 632de91534a723f4d84d59ab3fc5676e88f0f944
File: src/slurmd/slurmstepd/mgr.c line 1261
By grondo:

Ah, ok. I would still have that as a separate commit because the intent
wasn't clear
(at least to me)

On Fri, Jan 13, 2012 at 10:47 PM, hautreux <
reply@reply.github.com
> wrote:

> It is not just a cleanup as the intent is to be sure that in case of an IO
> setup error, SLURM_SUCCESS is still returned (to prevent the node from
> being set to drain). I needed to ensure the return of rc instead of
> SLURM_ERROR in the fail* label and as a result had to do some modification
> to set rc to SLURM_ERROR where necessary.
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/hautreux/slurm/commit/632de91534a723f4d84d59ab3fc5676e88f0f944#commitcomment-866397
>