On 01/17/2011 02:52 PM, Adrian Bridgett wrote:
> Hi Frank,

"Felix".

> then overridden here:
> 
> class puppetserver::config inherits puppet::config{

Oops - sorry, I completely missed that inherits clause in your previous
mail. Also your override syntax. *facepalm*

>> Also note that resource defaults become mute if you assign *any*
>> parameter value in an actual instance declaration. So
>>
>> File { notify => Service["foo"] }
>> file { "baz": notify +> Service["bar"] }
>>
>> doesn't mean "enhance the default by Service["bar"]", but instead it
>> means "replace the default by whatever was set *for this very file* plus
>> Service["bar"]".

I should have added "that's what I infer and what seems sort of logical
to me."

>> As an aside, I try and steer clear of plusignment whenever it's possible.
> 
> Ah, interesting - I hadn't appreciated that, it seems a bit of a shame
> in some ways.  That explains what I'm seeing.
> 
> One reason for the slight oddness here and setting defaults was to all
> a more standardised approach in how we write modules - to avoid
> missing notifies (or conversely too many).

I agree about exploiting resource defaults where possible, but
apparently they don't play nice with plusignment overrides, as you
noticed here.

It would be interesting to see whether repeating the notify parameter in
your file { } declaration (in redundance with the default you declare)
fixes your problem.

Cheers,
Felix

-- 
You received this message because you are subscribed to the Google Groups 
"Puppet Users" group.
To post to this group, send email to puppet-users@googlegroups.com.
To unsubscribe from this group, send email to 
puppet-users+unsubscr...@googlegroups.com.
For more options, visit this group at 
http://groups.google.com/group/puppet-users?hl=en.

Reply via email to