Seemingly bug-free and free (as in freedom) version of Observium's ifAlias snmp extension. #70

Merged
merged 2 commits into from Jan 19, 2014

Projects

None yet

3 participants

@f0o
Member
f0o commented Jan 18, 2014

Related to #29

@f0o f0o referenced this pull request Jan 18, 2014
Closed

ifAlias not being polled right #29

@supertylerc
Contributor

@f0o -

This isn't a fitting PR in its current format. While the script looks good, it wouldn't actually be used. I believe ifAlias is grabbed via the ports polling script. Is there something wrong with it, and if so, can you point to what is broken?

If this script provides additional functionality on top of what is provided by the ports php poller, could you describe what the additional functionality is?

Thanks!

@f0o
Member
f0o commented Jan 18, 2014

This script is used for Interface Description Parsing see: http://observium.com/wiki/Interface_Description_Parsing
The name of the script is also taken from the observium wiki, as it should be a drop-in replacement.

However, the script from observium.com is buggy and does not work with dynamic interfaces that change their interface-IDs.
This one does work with dynamic interfaces and also works with snmpbulkwalk.

//Edit: See http://observium.com/wiki/IfAlias_descriptions_on_Debian/Ubuntu for their script & usage.

@laf
Member
laf commented Jan 18, 2014

It's a client side script rather than something LibreNMS uses directly.

@supertylerc
Contributor

@f0o + @laf

Thanks for the links--we'll also need to update documentation to indicate that snmpd.conf must be modified to make full use of the script. We'll also need to note that the script must be copied to an appropriate directory on all Linux servers that will make use of the script.

Could you please make these documentation modifications and add them to this PR? There's no particular reason to have separate PRs to update documentation for this since it's part of the feature.

Thanks!

@f0o f0o Added wiki page.
as discussed in #70
d1c7f00
@laf
Member
laf commented Jan 19, 2014

Great work @f0o :)

@supertylerc
Contributor

@f0o

Thanks for your work, sir!

@supertylerc supertylerc merged commit 34aed05 into librenms:master Jan 19, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment