Re: [Mon-devel] _last_checked?

[David N. <vit...@cm...>]

--On Tuesday, July 06, 2004 8:55 AM -0500 Tim Klein 
<tkp...@ti...> wrote:

> Subroutine proc_cleanup() contains a reference to _last_checked .
> Shouldn't that be _last_check instead?  I can't find any other
> reference to _last_checked in the mon code.

At first glance, that certainly looks like a bug.

However its not completely clear if the typo should be fixed or the
line deleted.

run_monitor() sets _last_check and _next_check
proc_cleanup() sets _last_failure or _last_success

_last_check is never used internal to Mon, except to set _next_check

So, changing proc_cleanup() to set _next_check when a monitor script exits 
will only result in the timestamp passed to Mon clients being when the 
script exited instead of when it was started.  But when it exited is 
actually already available as either _last_success or _last_failure.  But 
having _last_check be when the script was started might be interesting for 
purposes of calculating how long mon scripts are taking to run.

I'd lean towards deleting the line from proc_cleanup.

Jim?

-David

David Nolan                    <*>                    vit...@cm...
curses: May you be forced to grep the termcap of an unclean yacc while
      a herd of rogue emacs fsck your troff and vgrind your pathalias!