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

Rudy/snmp table #1005

Merged
merged 20 commits into from
Jul 8, 2014
Merged

Rudy/snmp table #1005

merged 20 commits into from
Jul 8, 2014

Conversation

bunelr
Copy link
Contributor

@bunelr bunelr commented Jun 26, 2014

Update to the snmp checks, modifying the way we query things.

Switching from doing snmpgets on the value specified in the configuration to
doing snmpwalks on domain specified by the user and gathering information from the results.

For completely specified objects, this doesn't change anything.
It allows for more powerful query: can query table whose size is not known at configuration time, can query object being in a table with multiple indexes, gather information for tags from the indexes or use information contained in other columns of the row as tags for the results we want.

I will also update the documentation in the documentation website with detailed examples.

@remh
Copy link

remh commented Jul 7, 2014

👍

interface_type = tag.split(":")[1]
try:
float(interface_type)
except:
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You should avoid bare except.

remh pushed a commit that referenced this pull request Jul 8, 2014
@remh remh merged commit 9f1374a into master Jul 8, 2014
@bunelr bunelr deleted the rudy/snmp-table branch July 21, 2014 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants