-
-
Notifications
You must be signed in to change notification settings - Fork 273
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 SNMPv3 support #665
Add SNMPv3 support #665
Conversation
Re-running checks... |
Tried this out but was getting a lot of errors from the data types (using PE 2016.4.3 which includes Puppet 4.7). I've made my own fork with the changes (minus the Puppet4 data types) here which is passing tests for me. |
What were the errors? I don't think it's realistic to just remove all of the Puppet 4 types but it might be a simple fix. An example of how you're calling the module/define would be good too. |
Hi @bodgit , thanks for the PR. Can you please rebase it against latest master? |
* Refactor SNMP plugin to use Puppet 4 data types. * Rename `invertmatch` and `instanceprefix` parameters in `collectd::plugin::snmp::data` defined type to match the hash keys passed in the `data` parameter in `collectd::plugin::snmp` class.
b3deef3
to
d063661
Compare
Done. Tests all seem to pass. |
$data = {}, | ||
$hosts = {}, | ||
$interval = undef, | ||
Enum['present', 'absent'] $ensure = 'present', |
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.
datatypes \o/
Thanks @bodgit. |
This PR adds SNMPv3 support to the SNMP plugin which should fix #236, I also took the opportunity to refactor the plugin to use Puppet 4.x data types.
The only breaking change AFAICT is that I renamed the
invertmatch
andinstanceprefix
parameters toinvert_match
andinstance_prefix
respectively in thecollectd::plugin::snmp::data
defined type to match the hash keys passed in thedata
parameter incollectd::plugin::snmp
class given everything else matched up.To keep my employers legal team happy, I have to include the following: