Skip to content

Add nagios_host parameter "address6"#530

Closed
ssm wants to merge 1 commit intopuppetlabs:masterfrom
ssm:nagios_host_ipaddress6
Closed

Add nagios_host parameter "address6"#530
ssm wants to merge 1 commit intopuppetlabs:masterfrom
ssm:nagios_host_ipaddress6

Conversation

@ssm
Copy link
Contributor

@ssm ssm commented Feb 28, 2012

The "address6" parameter is used for multi-protocol (ipv4 and ipv6)
monitoring.

 The "address6" parameter is used for multi-protocol (ipv4 and ipv6)
 monitoring.

 - supported in nagios via community patch in nagios exchange
   (http://exchange.nagios.org/directory/Patches/Nagios/IPv6-address-in-host-definition-patch/details)

 - supported in icinga
@ssm
Copy link
Contributor Author

ssm commented Feb 28, 2012

There does not seem to be any parameter specific tests for the nagios types, so I'm not sure where to start on a test.

Also, the icinga documentation for "address6" in the "host" object is at http://docs.icinga.org/1.6/en/objectdefinitions.html#id1330888

@cprice404
Copy link

This looks worthwhile, and fairly safe... but I'm not entirely familiar with how the nagios support works.

@cprice404
Copy link

Reviewed this a bit further; there are a couple of issues to consider here:

  1. As mentioned in the original pull request description, the 'address6' parameter isn't supported by out-of-the-box nagios, so we'd definitely need tests to ensure that this doesn't break anything for compatibility.

  2. Nagios has been moved to a module for Telly, so this will need to be re-based against the module repo.

I've created a ticket here:

http://projects.puppetlabs.com/issues/13674

To make sure that we consider this further. Closing this pull request since the file that the patch is against is no longer in the Telly codebase.

ssm--thanks for the submission and for bringing this to our attention, it's definitely something we should be considering.

@cprice404 cprice404 closed this Apr 6, 2012
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