sensu-plugins / sensu-plugins-haproxy

This plugin provides native HAProxy instrumentation for monitoring and metrics collection, including: service health and backend server metrics.
http://sensu-plugins.io
MIT License
11 stars 37 forks source link

check-haproxy.rb checking max instead of limit #39

Closed jindraj closed 4 years ago

jindraj commented 5 years ago

Shouldn't be here comparison to limit instead of max? Because otherwise it triggers CRIT everytime you hit new max which can be still way below limit.

https://github.com/sensu-plugins/sensu-plugins-haproxy/blob/master/bin/check-haproxy.rb#L164-L165

https://github.com/sensu-plugins/sensu-plugins-haproxy/blob/master/bin/check-haproxy.rb#L171-L172