On Wednesday, May 15, 2013 10:45:19 AM UTC-5, Dick Davies wrote:
On 15 May 2013 15:24, jcbollinger <John.Bo...@stjude.org <javascript:>>wrote:
On Wednesday, May 15, 2013 6:29:14 AM UTC-5, Dick Davies wrote:

At the minute we've set an empty string, but then templates etc. need to
be littered with

if ($url_from_hiera != "") {

which feels like the sort of 'magic variable' that will bite future me
in the ass in around 6 months.
So, what do you envision hiera returning instead? undef? How would that
make things better for you? Wouldn't you still need similar conditional
logic in all the same places?
I can get a bit closer to the ideal if I set the value as : false
- then the templates look like this (the $yum_proxy_url gets unpacked out
of hiera
up in the module):

<%- if yum_proxy_url != false %>
# use proxy
proxy=<%=yum_proxy_url %>
<% end -%>

which is a bit more readable. Not really clear why the : != false part is
required, but
feels cleaner to me.
In other words, yes, you do need all the same logic in all the same
places. In that case, your only real option is to choose the special value
you like best. Personally, I find the empty string very appropriate, and
whereas there could conceivably be a machine named "false", but there
cannot be one named "". Still, it's a matter of taste.

And no, annoyingly


breaks yum :)


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

Search Discussions

Discussion Posts


Follow ups

Related Discussions

Discussion Navigation
viewthread | post
posts ‹ prev | 4 of 5 | next ›
Discussion Overview
grouppuppet-users @
postedMay 15, '13 at 11:29a
activeMay 16, '13 at 6:53a

2 users in discussion

Dick Davies: 3 posts Jcbollinger: 2 posts



site design / logo © 2022 Grokbase