On Tue, Nov 23, 2010 at 12:36:07AM -0600, Luke Kanies wrote:
> If you invert it, it works better:
> 
> port { '22/udp': label => 'telnet' }
> 

Havent thought of that an it looks pretty good for me. The duplication
doesnt really bothers me, because I can easily write a define for that.

But what I dont like about that approach is, that while it is good for
adding entries its just near to useless when you're trying to detect
errors. Because I think you rareley want to change the
label of a port but maybe you want to check, if the specified port is
actually right. (At least I want to do that). So in the example abvoe:
If I already have

  telnet 20/udp # someone copied&pasted but didnt changed port

in my file, puppet will happily adding another telnet entry.

-Stefan

Attachment: pgp9h7lazTusS.pgp
Description: PGP signature

Reply via email to