Re: [Nagios-devel] notification inheritance

[Guest]

--Apple-Mail-681-115535083
Content-Type: text/plain;
charset=US-ASCII;
format=flowed;
delsp=yes
Content-Transfer-Encoding: 7bit


On 24 May 2010, at 17:40, Gordon Messmer wrote:

> The documentation for object inheritance[1] indicates that where a
> notification_period is not available for a service, the
> notification_period from the associated host will be implicitly
> inherited.
>
> In my nagios configuration, I have a number of services which are
> applied to hostgroups. Rather than inheriting any notification
> period, each service has a "NULL" notification period which is
> treated as a '24x7' period. With the attached patch, nagios will
> check for a NULL period and use the period from the associated host
> as the documentation states will happen.
>
> I have no idea whether or not similar issues exist with the other
> implied inheritance values.

Hi Gordon,

Thanks for the patch. Could you include a change to the test
configurations in t/etc which demonstrates the problem.

When you've made a change to the configurations with the inheritance
problem, you should be able to run t/900-configparsing.t and get a
failure, but with your code patch, it should then work as expected.

More details: http://wiki.nagios.org/index.php/Nagios ... es#Testing

Re-post with your test changes and I'll apply to the stable branch.

Ton


--Apple-Mail-681-115535083
Content-Type: text/html;
charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

On 24 May 2010, =
at 17:40, Gordon Messmer wrote:The =
documentation for object inheritance[1] indicates that where a =
notification_period is not available for a service, the =
notification_period from the associated host will be implicitly =
inherited.In my nagios configuration, I have a number of =
services which are applied to hostgroups.  Rather than inheriting =
any notification period, each service has a "NULL" notification period =
which is treated as a '24x7' period.  With the attached patch, =
nagios will check for a NULL period and use the period from the =
associated host as the documentation states will happen.I have =
no idea whether or not similar issues exist with the other implied =
inheritance values.Hi =
Gordon,Thanks for the patch. Could you include a change to the =
test configurations in t/etc which demonstrates the problem.When =
you've made a change to the configurations with the inheritance problem, =
you should be able to run t/900-configparsing.t and get a failure, but =
with your code patch, it should then work as expected.More =
details: http://wiki.nagios.org/index.php/Nagios ... uidelines=
#TestingRe-post with your test changes and I'll apply to the =
stable branch.Ton=

--Apple-Mail-681-115535083--





This post was automatically imported from historical nagios-devel mailing list archives
Original poster: [email protected]