-
Notifications
You must be signed in to change notification settings - Fork 5.6k
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
Add snmp input plugin #495
Conversation
With HTTP JSON or Elasticsearch, one can also process values nested in arrays.
@titilambert I don't quite understand how this is configured? Users can specify an OID file from snmptranslate, or they can input each OID into the config file? |
Address string | ||
Community string | ||
// SNMP version. Default 2c | ||
Version int |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
shouldn't this be a string?
While at it also add a missing dependency on lsof required by the netstat plugin. closes influxdata#512
…ition in post-install. closes influxdata#526 closes influxdata#525
How would I go forward with this to get a table with interfaces and their metrics with correlation? Would it be possible to choose one value over another, depending if it is present or not? Thinking IF-MIB 's ifXTable (counter64) vs interfaces (counter32) issue on some appliances even to this day. I have a lot of data I want to gather that is in the snmp table format so I think this is imporant. I dont neccesarily want all columns/rows from the table either, I want to have some way to choose which results to insert. Maybe both on a given alias/name or only these columns. Thoughts? |
Hello ! @TheFlyingCorpse I worked on an other SNMP project called SNMPbooster for Shinken (http://shinken-module-snmp-booster.readthedocs.org/en/latest/). I can try to adapt its concept to this plugin if you find it useful ... |
d7e1b44
to
2ca2af4
Compare
@sparrc I add some comments in sample config output. |
@titilambert - if you could provide examples for how you would do tables like interfaces/ifXtable that would be great and really what I would need to be able to start testing myself. |
@TheFlyingCorpse here some examples
if you want to get a entire table (with SNMP bulk)
Then set your hosts
The To get oid translated
where |
@TheFlyingCorpse about the ifXtable example, unfortunately, you have to add each OID manually (or script generated) :/
|
@TheFlyingCorpse about your first question, I'm agree with you to get something better with "metrics correlation". But, this feature need this plugin have to store some data between two checks... and I don't know how to do it. Maybe @sparrc can help me on this point ? |
@TheFlyingCorpse to be short, this is only the first version of this plugin, the next one will be much better ;) |
I'm not thinking correlation for past checks, thats up to the backend to do (counter32/64 etc). What I was hoping for was an example of how I could easily get ifXtable and select which columns (or rows by some string to an oid?), not adding all OIDs manually. What else are the SNMP MIB conversion process for? |
@TheFlyingCorpse SNMP conversion process, it just convert your mib files into a single file with lines like:
It use to send Can we discuss about your needs on IRC ? It could be useful to get some specs :) |
@titilambert Sure, I see I didnt catch the full example earlier. I'm on IRC under this nickname on freenode. |
@sparrc After a discussion with @TheFlyingCorpse you can merge it, if you think this is good. I will release a new version of this plugin with more features later |
- If we detect errors when gathering stat via socket, return those error so it canbe appear in Telegraf log - Improve fcgi client, also upgrade it to current version of Go at https://golang.org/src/net/http/fcgi/fcgi.go - Add test for unix socket and fcgi to remotely connect but only as an extra url field. - Allow customization of fpm status path - Document about using of `host` in case `unixsocket` that it isn't used - Documet upgrade for new data layout closes influxdata#499 closes influxdata#502 closes influxdata#538
Thanks @titilambert, can you change the PR to be pushing to |
Gather metric by parsing XMLoutput of `passenger-status` utility. More information of this utility: https://www.phusionpassenger.com/library/admin/apache/overall_status_report.html closes influxdata#522
With the advent of Kafka 0.9.0+ it is possible to set up TLS client certificate based authentication to limit access to Kafka. Four new configuration variables are specified for setting up the authentication. If they're not set the behavior stays the same as before the change. closes influxdata#541
This is a new PR of #453 rebased on 0.3.0