[Nagiosplug-devel] Nagios::Plugin thresholds

Vonnahme, Nathan nathan.vonnahme at bannerhealth.com
Fri Sep 8 22:43:43 CEST 2006


 

> -----Original Message-----
> From: nagiosplug-devel-bounces at lists.sourceforge.net 
> [mailto:nagiosplug-devel-bounces at lists.sourceforge.net] On 
> Behalf Of Gavin Carr
> Sent: Thursday, September 07, 2006 3:20 PM

> More on check_threshold, I think we should make 'warning' and 
> 'critical'
> default to $plugin->opt('warning') and 
> $plugin->opt('critical') if those
> are defined, as that's likely to be far and away the common use case.
> That would mean your example above just becomes:
> 
>     my $return_status = $plugin->check_threshold( check => $value );
>     $plugin->nagios_exit( $return_status, $message );
> 
> Maybe we should allow:
> 
>     my $return_status = $plugin->check_threshold( $value );
> 
> for that case as well?
> 
> So does this mean you're putting your hand up to implement 
> this Nathan? ;-)
>   

I'd be happy to take a crack at it after November 1, but I won't be able
to until then.




> > On a side note, Gavin, is there any problem with 
> standardizing on one
> > spelling of license/licence in Getopt.pm ?
> 
> Er, well I speak the Queen's English, in which licence is the 
> noun, and 
> license is the verb. I like helping American's understand 
> that there's is
> not the only brand on the planet, but I'm not _too_ much of a 
> linguistic
> Nazi. ;-)
> 
> So no, if you want to Americanise it I don't really care.



Ah.  I had never noticed that verb/noun distinction!  I thought it was
just half American spelling and half Queen's and was going to
s/licen[cs]/licens/gi to minimize future confusion.

But this paragraph is still inconsistent, I think:


=item license

License text, included in the longer --help output (see below for an 
example). By default, this is set to the standard nagios plugins
GPL licence text:

  This nagios plugin is free software, and comes with ABSOLUTELY NO
WARRANTY. 
  It may be used, redistributed and/or modified under the terms of the
GNU 
  General Public Licence (see
http://www.fsf.org/licensing/licenses/gpl.txt).

and there's this line which looked like a mistake to me (though I think
it works correctly on closer inspection):

    license => { default => $DEFAULT{licence} },


I think the noun is meant in all those cases, so we could just
s/licen[cs]/licenc/gi except for the GPL URLs, right?




More information about the Devel mailing list