Service dependencies broken in CVS HEAD?

[Ethan Galstad]

Thomas Guyot-Sionnest wrote:
>> -----Original Message-----
>> From: nagios-devel-bounces at lists.sourceforge.net 
>> [mailto:nagios-devel-bounces at lists.sourceforge.net] On Behalf 
>> Of Ethan Galstad
>> Sent: February 26, 2007 12:38
>> To: Nagios Developers List
>> Subject: Re: [Nagios-devel] Service dependencies broken in CVS HEAD?
>>
>>
>> You shouldn't need to apply your patch - I've already implemented 
>> similar functionality in the CVS code.  If you want same host 
>> dependencies, don't use the "dependent_host_name", 
>> "dependent_hostgroup", or "dependent_servicegroup" directives.
>>
>> define servicedependency{
>> 	host h1,h2
>> 	service_description	svc1
>> 	dependent_service_description	svc2,svc3
>> 	}
>>
>> I just posted an update to CVS with some code cleanups, but 
>> no change in 
>> functionality.  Should make it easier to understand the code in 
>> duplicate_objects().
> 
> Thanks; I'm glad to see this is finally going in (and as I suggested, by
> simply ommiting the dependent_ hosts). I just tested it and it works :)
> 
> I'm not sure where (you patch is big) but you definately fixed something. It
> works now with exactly the same config as with yesterday's checkout. I don't
> understand where you do it though, I'll have to look closer...
> 
> Thomas
> 
> 

Hmm, well perhaps the ugly variable names got me mixed up when I was 
writing the initial code.  I guess changing the vars to better suited 
names resulted in a bug fix. :-)


Ethan Galstad,
Nagios Developer
---
Email: nagios at nagios.org
Website: http://www.nagios.org

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys-and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV