LGTM, thanks
On Mon, Dec 23, 2013 at 6:23 PM, Klaus Aehlig <[email protected]> wrote: > When cleaning up finished jobs, remove the inotify > attached to them, if any. > > Signed-off-by: Klaus Aehlig <[email protected]> > --- > src/Ganeti/JQScheduler.hs | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/Ganeti/JQScheduler.hs b/src/Ganeti/JQScheduler.hs > index b1030aa..aa8da86 100644 > --- a/src/Ganeti/JQScheduler.hs > +++ b/src/Ganeti/JQScheduler.hs > @@ -156,6 +156,7 @@ cleanupFinishedJobs qstate = do > jlist = commaJoin $ map showJob finished > unless (null finished) > . logInfo $ "Finished jobs: " ++ jlist > + mapM_ (maybe (return ()) killINotify . jINotify) finished > > -- | Decide on which jobs to schedule next for execution. This is the > -- pure function doing the scheduling. > -- > 1.8.5.1 > > -- -- Helga Velroyen | Software Engineer | [email protected] | Google Germany GmbH Dienerstr. 12 80331 München Registergericht und -nummer: Hamburg, HRB 86891 Sitz der Gesellschaft: Hamburg Geschäftsführer: Graham Law, Christine Elizabeth Flores
