chroot selector items need to be delimited about puppet-bind

[mazzystr]

sorry for the terrible formatting!

[thias]

Could you maybe use "diff -u" (unified) and put that between ``` lines?

[mazzystr]

sure, np. give me a sec.

[mazzystr]

--- /etc/puppet/modules/bind/manifests/init.pp 2014-03-18 11:27:42.807512784 -0400
+++ /etc/puppet/modules/bind/manifests/init.pp-old 2014-03-18 13:38:35.511510396 -0400
@@ -23,8 +23,8 @@

Main package and service

$packagenamesuffix = $chroot ? {

• 'true' => '-chroot',
• 'false' => '',

• true => '-chroot',
• false => '',
  }
  class { 'bind::package':
  packagenameprefix => $packagenameprefix,
  @@ -37,8 +37,8 @@

We want a nice log file which the package doesn't provide a location for

$bindlogdir = $chroot ? {

• 'true' => '/var/named/chroot/var/log/named',
• 'false' => '/var/log/named',

• true => '/var/named/chroot/var/log/named',
• false => '/var/log/named',
  }
  file { $bindlogdir:
  require => Class['bind::package'],

[thias]

What version of my module are you using? I see this in the current code :

  # Main package and service
  $packagenamesuffix = $chroot ? {
    true  => '-chroot',
    false => '',
  }

And :

  # We want a nice log file which the package doesn't provide a location for
  $bindlogdir = $chroot ? {
    true  => '/var/named/chroot/var/log/named',
    false => '/var/log/named',
  }

And the module is working fine for me...

[mazzystr]

puppet --version

2.7.23

cat bind/Modulefile

name 'thias-bind'
version '0.5.0'

[thias]

And are you passing the $chroot parameter value using hiera? If you are, then that must be the problem, as there was a bug in older hiera versions where booleans were incorrectly passed as strings.

[thias]

No reply, so closing (I hope the boolean/string issue was the cause!).