dpeters / puppet-opsview

Puppet types/providers to support Opsview resources
12 stars 10 forks source link

Added SNMP config into puppet module #11

Closed duncs closed 11 years ago

duncs commented 11 years ago

Add in SNMP v1/2c/3 config detail into puppet module

dpeters commented 11 years ago

Thanks again, I'll get this merged in. I definitely agree about ideally deferring the opsview reload until all resources have been applied.

For the SNMP additions to the JSON, do you know which release of opsview they were introduced in?

duncs commented 11 years ago

On 20 Nov 2012, at 18:53, Devon Peters notifications@github.com wrote:

Thanks again, I'll get this merged in. I definitely agree about ideally deferring the opsview reload until all resources have been applied.

For the SNMP additions to the JSON, do you know which release of opsview they were introduced in?

— Reply to this email directly or view it on GitHub.

As far as I can remember most of the options have been in there since the beginning.

snmp_port was added over 2 years ago (commit 5204 - https://secure.opsview.com/wsvn/wsvn/opsview/trunk/opsview-core/share/xml/opsview.rng?op=revision&rev=5204&peg=5204) and thats the last relevant entry I can see atm.

If you need more info, let me know and I'll track it down.

Duncs

dpeters commented 11 years ago

Ok, great. I just wanted to be sure to add a note if those features were only supported in some later version.

duncs commented 11 years ago

On 20 Nov 2012, at 21:44, Devon Peters notifications@github.com wrote:

Ok, great. I just wanted to be sure to add a note if those features were only supported in some later version.

I have picked up on one error I'm not sure how to sort yet (since my Puppet/Ruby foo is not great). The module cannot currently be used against an Opsview Core system as the module assumes use of distributed monitoring (i.e. 'monitored by') which core doesn't accept.

This should probably be put onto the TODO list to fix

Duncs