Ticket 2355 - Add reservation flag to remove the reservation once no pending jobs for it remain
Summary: Add reservation flag to remove the reservation once no pending jobs for it re...
Status: RESOLVED FIXED
Alias: None
Product: Slurm
Classification: Unclassified
Component: slurmctld (show other tickets)
Version: 16.05.x
Hardware: Linux Linux
: 5 - Enhancement
Assignee: Moe Jette
QA Contact:
URL:
Depends on:
Blocks:
 
Reported: 2016-01-16 13:51 MST by Moe Jette
Modified: 2016-04-11 05:58 MDT (History)
2 users (show)

See Also:
Site: NERSC
Slinky Site: ---
Alineos Sites: ---
Atos/Eviden Sites: ---
Confidential Site: ---
Coreweave sites: ---
Cray Sites: ---
DS9 clusters: ---
Google sites: ---
HPCnow Sites: ---
HPE Sites: ---
IBM Sites: ---
NOAA SIte: ---
NoveTech Sites: ---
Nvidia HWinf-CS Sites: ---
OCF Sites: ---
Recursion Pharma Sites: ---
SFW Sites: ---
SNIC sites: ---
Tzag Elita Sites: ---
Linux Distro: ---
Machine Name:
CLE Version:
Version Fixed: 16.06.0-pre1
Target Release: ---
DevPrio: ---
Emory-Cloud Sites: ---


Attachments
Patch to add the feature to version 15.08 (4.85 KB, patch)
2016-01-17 14:50 MST, Moe Jette
Details | Diff

Note You need to log in before you can comment on or make changes to this ticket.
Description Moe Jette 2016-01-16 13:51:15 MST

    
Comment 1 Moe Jette 2016-01-17 14:50:13 MST
Created attachment 2623 [details]
Patch to add the feature to version 15.08

Since this is a new feature rather than a bug fix, I am only adding the feature to our version 16.05 distribution, but here is a patch which you can apply locally to version 15.08 if desired. Basically you add an option of "flag=purge_comp" when creating the reservation for it to get purged once the last job completes. You can either set the start time in the future or have at least 60 seconds to submit a job to it before it gets purged.
Comment 2 Moe Jette 2016-01-17 14:52:39 MST
Here is the commit to the version 16.05 branch:
https://github.com/SchedMD/slurm/commit/be6e1fb50cd3f28f826ff03546219c39fc27911b
Comment 3 Doug Jacobsen 2016-01-17 15:01:55 MST
Great!  I'll include it in my future builds.

Thanks, Moe.

Doug
On Jan 17, 2016 8:52 PM, <bugs@schedmd.com> wrote:

> Moe Jette <jette@schedmd.com> changed bug 2355
> <http://bugs.schedmd.com/show_bug.cgi?id=2355>
> What Removed Added
> Status UNCONFIRMED RESOLVED
> Version Fixed   16.06.0-pre1
> Resolution --- FIXED
>
> *Comment # 2 <http://bugs.schedmd.com/show_bug.cgi?id=2355#c2> on bug 2355
> <http://bugs.schedmd.com/show_bug.cgi?id=2355> from Moe Jette
> <jette@schedmd.com> *
>
> Here is the commit to the version 16.05 branch:https://github.com/SchedMD/slurm/commit/be6e1fb50cd3f28f826ff03546219c39fc27911b
>
> ------------------------------
> You are receiving this mail because:
>
>    - You are on the CC list for the bug.
>
>
Comment 4 Doug Jacobsen 2016-01-20 13:10:38 MST
Hi Moe,

I can confirm that this feature works in a patched 15.08.7 build on cori.

Thanks!
Doug

----
Doug Jacobsen, Ph.D.
NERSC Computer Systems Engineer
National Energy Research Scientific Computing Center <http://www.nersc.gov>
dmjacobsen@lbl.gov

------------- __o
---------- _ '\<,_
----------(_)/  (_)__________________________


On Sun, Jan 17, 2016 at 10:01 PM, Douglas Jacobsen <dmjacobsen@lbl.gov>
wrote:

> Great!  I'll include it in my future builds.
>
> Thanks, Moe.
>
> Doug
> On Jan 17, 2016 8:52 PM, <bugs@schedmd.com> wrote:
>
>> Moe Jette <jette@schedmd.com> changed bug 2355
>> <http://bugs.schedmd.com/show_bug.cgi?id=2355>
>> What Removed Added
>> Status UNCONFIRMED RESOLVED
>> Version Fixed   16.06.0-pre1
>> Resolution --- FIXED
>>
>> *Comment # 2 <http://bugs.schedmd.com/show_bug.cgi?id=2355#c2> on bug
>> 2355 <http://bugs.schedmd.com/show_bug.cgi?id=2355> from Moe Jette
>> <jette@schedmd.com> *
>>
>> Here is the commit to the version 16.05 branch:https://github.com/SchedMD/slurm/commit/be6e1fb50cd3f28f826ff03546219c39fc27911b
>>
>> ------------------------------
>> You are receiving this mail because:
>>
>>    - You are on the CC list for the bug.
>>
>>