Index: src/slurmctld/job_mgr.c
===================================================================
--- src/slurmctld/job_mgr.c     (revision 22720)
+++ src/slurmctld/job_mgr.c     (revision 22721)
@@ -6306,6 +6306,10 @@
                                } else
                                        job_ptr->state_reason = WAIT_HELD_USER;
                                xfree(job_ptr->state_desc);
+                       } else if ((job_ptr->state_reason == WAIT_HELD) ||
+                                  (job_ptr->state_reason == WAIT_HELD_USER)) {
+                               job_ptr->state_reason = WAIT_NO_REASON;
+                               xfree(job_ptr->state_desc);
                        }
                } else if ((job_ptr->priority == 0) &&
                           (job_ptr->state_reason == WAIT_HELD_USER)) {

________________________________________
From: [email protected] [[email protected]] On Behalf 
Of Bjørn-Helge Mevik [[email protected]]
Sent: Wednesday, March 16, 2011 2:38 PM
To: [email protected]
Subject: Re: [slurm-dev] Possible bug: scontrol hold not possible when grpcpus 
is set

"Jette, Moe" <[email protected]> writes:

> The reason field clearing problem was discovered and fixed last week.
> It will be in v2.2.4.

That's great!

> Let me know if you want a patch now; it's just a couple of lines.

It would be nice with a patch now, since we already have the other patch
to apply on the production cluster.

--
Cheers,
Bjørn-Helge Mevik, dr. scient,
Research Computing Services, University of Oslo


Reply via email to