On Wed, 28 Sep 2011 15:00:03 -0800, Erinn Looney-Triggs <erinn.looneytriggs@xxxxxxxxx> wrote: > Whilst trying to learn how to write func modules I stumbled across > nagios-check.py, which looks like it could use a little love. The > attached patch gets it to work, cleans up some formatting, changes the > name to nagios_check.py so you can import it easily to test, and allows > for the use of configuration files to modify the nagios plugins path. I > also changed the class name, there is already a newer Nagios class in > the nagios.py module, and though this may not matter, I believe it can > for the conf files that are created. Perhaps I don't understand the > whole structure of things, but it looks to me like this could lead to a > collision. > > -Erinn Good catch on the class names. I didn't even consider that when I wrote the class in nagios.py. I guess I always just assumed it was named NagiosCheck or something. -- Tim Bielawa Associate Software Engineer Production Control Team 919.754.4741 Cube / 919.332.6411 Cell IRC: tbielawa Raleigh, NC 1BA0 4FAB 4C13 FBA0 A036 4958 AD05 E75E 0333 AE37
Attachment:
pgpgL5i8Mpzg6.pgp
Description: PGP signature
_______________________________________________ Func-list mailing list Func-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/func-list
