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 #11931] Adding option to access ifName for manubulon snmp-interface check command #4276

Closed
icinga-migration opened this issue Jun 12, 2016 · 7 comments

Comments

Projects
None yet
1 participant
@icinga-migration
Copy link
Member

commented Jun 12, 2016

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

Created by cyco on 2016-06-12 12:19:13 +00:00

Assignee: mfriedrich
Status: Resolved (closed on 2016-06-15 16:05:03 +00:00)
Target Version: 2.5.0
Last Update: 2016-06-16 08:11:42 +00:00 (in Redmine)

Backport?: Not yet backported
Include in Changelog: 1

Recent Net-SNMP's snmpd daemon has changed behavior returning detailed
information about the network interfaces in the IF-MIB::ifDescr MIB.

There is a patch out there that is working around the issue (see also
https://bugs.debian.org/812837) by providing the behaviour via '-N'.

This path provides a way to make use of this workaround.

#85

Attachments

Changesets

2016-06-15 16:02:36 +00:00 by (unknown) 5e3ea47

ITL: Add option for ifName to Manubulon CheckCommand 'snmp_interface'

Recent Net-SNMP's snmpd daemon has changed behavior returning detailed
information about the network interfaces in the IF-MIB::ifDescr MIB.

There is a patch out there that is working around the issue (see also
https://bugs.debian.org/812837) by providing the behaviour via '-N'.

This path provides a way to make use of this workaround.

fixes #11931

Signed-off-by: Michael Friedrich <michael.friedrich@netways.de>
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 12, 2016

Updated by cyco on 2016-06-12 12:29:07 +00:00

Added documentation update into PR.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 13, 2016

Updated by gbeutner on 2016-06-13 06:59:01 +00:00

  • Category changed from Configuration to ITL
  • Status changed from New to Assigned
  • Assigned to set to cyco
  • Target Version set to 307

Please squash those two commits. There should only be one commit for each independently usable feature.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 13, 2016

Updated by cyco on 2016-06-13 07:48:41 +00:00

  • File added manubulon_snmp-interface_ifName.patch

I've updated the PR. Find the new patch attached.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 13, 2016

Updated by mfriedrich on 2016-06-13 07:51:10 +00:00

The mentioned patch for check_snmp_int only exists in Debian packages and would fail everywhere else. We should take a note on that.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 14, 2016

Updated by gbeutner on 2016-06-14 06:23:17 +00:00

  • Assigned to changed from cyco to mfriedrich
@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 15, 2016

Updated by Anonymous on 2016-06-15 16:05:03 +00:00

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

Applied in changeset 5e3ea47.

@icinga-migration

This comment has been minimized.

Copy link
Member Author

commented Jun 16, 2016

Updated by gbeutner on 2016-06-16 08:11:42 +00:00

  • Target Version changed from 307 to 2.5.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.