search

stanvit / nagios-monit-plugin

Nagios plugin for monit state checking
MIT License
12 stars 5 forks source link

Fix NameError: global name maintenance is not defined. #13

Closed darnuria closed 1 year ago

darnuria commented 1 year ago

Hello! Just upstreaming a Fix found by one of our ops @gandi thanks for the script!

Line 256 maintenance was checked but wasn't setted before.

osantiano commented 1 year ago

We had TB:

Traceback (most recent call last): File "/usr/lib/nagios/plugins/check_monit.py", line 370, in main() File "/usr/lib/nagios/plugins/check_monit.py", line 350, in main process_monit_response(get_status()) File "/usr/lib/nagios/plugins/check_monit.py", line 289, in process_monit_response process_service(service) File "/usr/lib/nagios/plugins/check_monit.py", line 256, in process_service if maintenance: NameError: global name 'maintenance' is not defined

stanvit commented 1 year ago

Thanks @darnuria , @Gandi and @osantiano for you fix, it's merged now

darnuria commented 1 year ago

yay thank you for the review merge and the project! :)