Re: [Nagios-devel] Fix for extraneous alerts + two development pleas
Posted: Wed Jan 12, 2011 8:32 am
Hi,
Ton Voon wrote:
> Hi!
>
> A customer of ours raised an issue which has kept me busy for the last
> few days. The problem is that if a host is shutdown, there are an
> excessive number of log entries for the host's services in nagios.log
> (though not for the first service that invoked a host check). More
> details about the problem and the fix are here: http://bit.ly/gZUzlR
>
> I have added test cases to catch this particular problem - details on
> running tests are here: http://wiki.nagios.org/index.php/Nagios ... es#Testing
>
> You can see the effect by reverting the fix to base/checks.c and
> running t-tap/ test_checks - this will fail for two attributes that
> were not being set correctly.
Thanks for the effort, much appreciated.
> I have two pleas:
> * Please add/request test cases for any new changes (where
> possible). This way we can build up a whole set of cases to measure
> impacts in future (I think handle_async_service_check_result() could
> do with a refactor but I daren't touch it at the moment)
Yeah that would be great - also regarding automated tests in tinderbox
or hudson (which is being used in opsview too as I have read)
> * Please can we move off CVS - I cannot link to a specific change
> which makes it hard to communicate about specific changes
+1 as usual.
I'm currently using various git mirrors to source patches from, like
Andreas' http://git.op5.org/git/?p=nagios.git;a=summary (your recent
commit is missing over there btw).
Kind regards,
Michael
> Kudos to Andreas for the test-stubs.c file in t-tap/.
>
> Ton
>
>
> ------------------------------------------------------------------------------
> Protect Your Site and Customers from Malware Attacks
> Learn about various malware tactics and how to avoid them. Understand
> malware threats, the impact they can have on your business, and how you
> can protect your company and customers by using code signing.
> http://p.sf.net/sfu/oracle-sfdevnl
> _______________________________________________
> Nagios-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/lis ... gios-devel
>
--
DI (FH) Michael Friedrich
Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria
email: [email protected]
phone: +43 1 4277 14359
fax: +43 1 4277 14338
web: http://www.univie.ac.at/zid
http://www.aco.net
Icinga Core& IDOUtils Developer
http://www.icinga.org
This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]
Ton Voon wrote:
> Hi!
>
> A customer of ours raised an issue which has kept me busy for the last
> few days. The problem is that if a host is shutdown, there are an
> excessive number of log entries for the host's services in nagios.log
> (though not for the first service that invoked a host check). More
> details about the problem and the fix are here: http://bit.ly/gZUzlR
>
> I have added test cases to catch this particular problem - details on
> running tests are here: http://wiki.nagios.org/index.php/Nagios ... es#Testing
>
> You can see the effect by reverting the fix to base/checks.c and
> running t-tap/ test_checks - this will fail for two attributes that
> were not being set correctly.
Thanks for the effort, much appreciated.
> I have two pleas:
> * Please add/request test cases for any new changes (where
> possible). This way we can build up a whole set of cases to measure
> impacts in future (I think handle_async_service_check_result() could
> do with a refactor but I daren't touch it at the moment)
Yeah that would be great - also regarding automated tests in tinderbox
or hudson (which is being used in opsview too as I have read)
> * Please can we move off CVS - I cannot link to a specific change
> which makes it hard to communicate about specific changes
+1 as usual.
I'm currently using various git mirrors to source patches from, like
Andreas' http://git.op5.org/git/?p=nagios.git;a=summary (your recent
commit is missing over there btw).
Kind regards,
Michael
> Kudos to Andreas for the test-stubs.c file in t-tap/.
>
> Ton
>
>
> ------------------------------------------------------------------------------
> Protect Your Site and Customers from Malware Attacks
> Learn about various malware tactics and how to avoid them. Understand
> malware threats, the impact they can have on your business, and how you
> can protect your company and customers by using code signing.
> http://p.sf.net/sfu/oracle-sfdevnl
> _______________________________________________
> Nagios-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/lis ... gios-devel
>
--
DI (FH) Michael Friedrich
Vienna University Computer Center
Universitaetsstrasse 7 A-1010 Vienna, Austria
email: [email protected]
phone: +43 1 4277 14359
fax: +43 1 4277 14338
web: http://www.univie.ac.at/zid
http://www.aco.net
Icinga Core& IDOUtils Developer
http://www.icinga.org
This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]