Reference variable from template

asked 2013-05-24 11:13:47 -0500

mcvicch
11 ●1 ●1 ●3

I want to create a params.pp class that'll house all my common variables used across various modules but when I reference them in a template, they just show up as "undefined."

Here's my common variables file located in: /etc/puppetlabs/puppet/modules/project/manifests/params.pp:

class project::params {
   $app_cluster = 'app'
}

Here's my manifest file located in: /etc/puppetlabs/puppet/modules/system/manifests/scripts.pp:

class system::scripts {
  file { '/h/scripts':
    ensure    => directory,
    source    => 'puppet:///modules/system/scripts',
    mode      => '750',
    recurse   => true,
  }

  file { '/h/scripts/system.properties':
    ensure    => present,
    content   => template('system/scripts/constructSystemProperties.erb'),
    mode ...

(more)

edit retag flag offensive close merge delete

add a comment

3 Answers

Sort by » oldest newest most voted

answered 2013-05-28 15:26:02 -0500

Stefan

2604 ●9 ●38 ●62 https://github.com/sts...

Personally I'd try to avoid using lookupvar and not cross reference variables from other modules in templates. Instead I'd make the variable local first and use that one in a template. Because when someone else is reading your class he'll normally expect that every information the class depends on is found inside the class (and no dependency is hidden away in templates)

class system::scripts(
  $app_cluster = $project::params::app_cluster
) {
  file { '/h/scripts/system.properties':
    ensure    => present,
    content   => template('system/scripts/constructSystemProperties.erb'),
    mode      => '750',
  }
}

So if the app_cluster parameter is not overwritten, it will use the value ... (more)

edit flag offensive delete link

add a comment

answered 2013-05-24 17:58:05 -0500

Julien R.

472 ●1 ●9 ●17 http://www.meetup.com/...

Try to make sure project::params is loaded before system::scripts with require

ie :

class system::scripts {

  require project::params 

  file { '/h/scripts':
    ensure    => directory,
    source    => 'puppet:///modules/system/scripts',
    mode      => '750',
    recurse   => true,
  }
  [...]