[dev.icinga.com #2411] address/address6 are not required #299
Comments
Updated by calestyo on 2012-03-07 00:58:43 +00:00 Oh and I just noted, that the BOTH of "contacts" and "contact_groups" may be undefined, too. |
Updated by Wolfgang on 2012-03-07 07:17:20 +00:00
Well, we can argue about this. Technically seen you are right, but: |
Updated by mfriedrich on 2012-03-07 09:16:08 +00:00 the core has a safety hook not to work with NULL values. nevertheless, the user must be made aware that not setting the address will trigger a gehthostbyname which requires DNS resolution and can be an unwanted dependency in case of failure. with my dns admin hat on, i would leave the address attribute mandatory. |
Updated by calestyo on 2012-03-07 20:18:36 +00:00 Well that this uses DNS then is explained already in the text, right? As well as the implications. I guess there are many setups where it's just "useless" to set the address[6] field. At the local institute for example, we run our own authoritative DNS servers. So I'd suggest... remove the red colour, keep the big fat warning, that DNS is then used and that this has implications. As it is now at least the text conflicts itself. The red colour says it's required, the text below says it's not. Cheers, |
Updated by mfriedrich on 2012-03-07 20:31:07 +00:00 sure, we run our own dns resolvers as well. but that does not imply that we want to query the local resolver => recursive resolver each time a check runs, and the cache cannot answer. the default examples on the most commands use if the core would have a bug, and not setting the address field to the hostname (which could be something else than an fqdn btw), this will lead into faulty condition some might not be able to debug. the past did show that is the most common mistake
you actually can NOT enfore the users to just use fqdn then. for my personal understanding, the address attribute is a must and i would make it happen in the core as well, if that would not break compatibility. so my vote is for leaving it to mandatory RED, and adding a note to the users, what could happen if they don't set it. and to add an addon which enforces the usage of the address attribute - checkmk. and that is good how it is, as it prevents live on execution dns resolve errors, warning your already on config creation if such an automated lookup will fail on generate, doing a rollback transaction. @wolfgang |
Updated by Wolfgang on 2012-03-08 20:33:54 +00:00
changed in master and r1.7 |
Updated by mfriedrich on 2012-03-16 14:21:29 +00:00
|
Updated by mfriedrich on 2012-05-05 10:12:00 +00:00
well if you insist on the change ... i would not do it. |
This issue has been migrated from Redmine: https://dev.icinga.com/issues/2411
Created by calestyo on 2012-03-07 00:53:39 +00:00
Assignee: Wolfgang
Status: Resolved (closed on 2012-05-05 10:12:00 +00:00)
Target Version: 1.7
Last Update: 2012-05-05 10:12:00 +00:00 (in Redmine)
Hi.
In http://docs.icinga.org/1.6/en/objectdefinitions.html#objectdefinitions-host the "address" directive is marked red as being required.
However, as mentioned later in the text it is not required.
Even if address6 isn't specified, then the value of "host_name" is simply taken as default.
So I guess, the red colour should be removed. :)
Cheers,
Chris.
The text was updated successfully, but these errors were encountered: