Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[dev.icinga.com #7351] snmp-load checkcommand has a wrong "-T" param value #2038

Closed
icinga-migration opened this issue Oct 7, 2014 · 3 comments
Labels
area/configuration DSL, parser, compiler, error handling bug Something isn't working
Milestone

Comments

@icinga-migration
Copy link

This issue has been migrated from Redmine: https://dev.icinga.com/issues/7351

Created by ccesario on 2014-10-07 14:07:59 +00:00

Assignee: ccesario
Status: Resolved (closed on 2014-10-11 16:30:03 +00:00)
Target Version: 2.2.0
Last Update: 2014-10-17 09:03:31 +00:00 (in Redmine)

Icinga Version: 2.1.1

Currently the snmp-load checkcommand had the default threshold values changed in issue #7275.

But this change does not work as expected because the plugin has "-T stand" param as default and "-T stand" support only one critical and one warning value according plugin documentation http://nagios.manubulon.com/snmp\_load.html

-T stand support windows and linux env.

To use 3 values in warning and crictical params is needed to use -T netsl or other. By example

#/usr/lib/nagios/plugins/check_snmp_load.pl -H 192.168.15.232 -L md5,des -l usermd5 -X p@ssword  -x p@ssword -T netsl  -w 8,3,3 -c 8,5,5
Load : 0.00 0.01 0.05 : OK

# /usr/lib/nagios/plugins/check_snmp_load.pl -H 192.168.15.232 -L md5,des -l usermd5 -X p@ssword  -x p@ssword -T stand   -w 8,3,3 -c 8,5,5
Multiple warning/critical levels not available for this check
Usage: /usr/lib/nagios/plugins/check_snmp_load.pl [-v] -H  -C  [-2] | (-l login -x passwd [-X pass -L ,])  [-p ] [-P 

Attachments

Changesets

2014-10-11 16:27:04 +00:00 by (unknown) 911863f

Plugin Check Commands: Fix snmp-load threshold values

Fix snmp-load threshold values to suppoprt "-T stand" param in check command.
To use 3 values, change the snmp_load_type var to "netsl".

fixes #7351

Signed-off-by: Michael Friedrich <michael.friedrich@gmail.com>
@icinga-migration
Copy link
Author

Updated by ccesario on 2014-10-07 14:27:47 +00:00

  • File added 0001-Fix-snmp-load-threshold-values.patch

To use 3 values on threshold, change the snmp_load_type var to "netsl".

It is attached the revert patch.

@icinga-migration
Copy link
Author

Updated by mfriedrich on 2014-10-11 16:27:46 +00:00

  • Status changed from New to Assigned
  • Assigned to set to ccesario
  • Target Version set to 2.2.0

@icinga-migration
Copy link
Author

Updated by Anonymous on 2014-10-11 16:30:03 +00:00

  • Status changed from Assigned to Resolved
  • Done % changed from 0 to 100

Applied in changeset 911863f.

@icinga-migration icinga-migration added bug Something isn't working area/configuration DSL, parser, compiler, error handling labels Jan 17, 2017
@icinga-migration icinga-migration added this to the 2.2.0 milestone Jan 17, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/configuration DSL, parser, compiler, error handling bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant