Merge pull request #681 from CIRB/master

fix missing ensure on concat::fragment resources
xbezdick · Mar 19, 2014 · 208005d · 208005d
2 parents 1421244 + 98f21a0
commit 208005d
Show file tree

Hide file tree

Showing 5 changed files with 7 additions and 0 deletions.
diff --git a/manifests/balancer.pp b/manifests/balancer.pp
@@ -55,6 +55,7 @@
   }
 
   concat::fragment { "00-${name}-header":
+    ensure  => present,
     target  => $target,
     order   => '01',
     content => "<Proxy balancer://${name}>\n",
@@ -67,12 +68,14 @@
   # concat fragments. We don't have to do anything about them.
 
   concat::fragment { "01-${name}-proxyset":
+    ensure  => present,
     target  => $target,
     order   => '19',
     content => inline_template("<% proxy_set.each do |key, value| %> Proxyset <%= key %>=<%= value %>\n<% end %>"),
   }
 
   concat::fragment { "01-${name}-footer":
+    ensure  => present,
     target  => $target,
     order   => '20',
     content => "</Proxy>\n",

diff --git a/manifests/balancermember.pp b/manifests/balancermember.pp
@@ -46,6 +46,7 @@
 ) {
 
   concat::fragment { "BalancerMember ${url}":
+    ensure  => present,
     target  => "${::apache::params::confd_dir}/balancer_${balancer_cluster}.conf",
     content => inline_template(" BalancerMember ${url} <%= @options.join ' ' %>\n"),
   }

diff --git a/manifests/init.pp b/manifests/init.pp
@@ -212,6 +212,7 @@
     require => Package['httpd'],
   }
   concat::fragment { 'Apache ports header':
+    ensure  => present,
     target  => $ports_file,
     content => template('apache/ports_header.erb')
   }

diff --git a/manifests/listen.pp b/manifests/listen.pp
@@ -3,6 +3,7 @@
 
   # Template uses: $listen_addr_port
   concat::fragment { "Listen ${listen_addr_port}":
+    ensure  => present,
     target  => $::apache::ports_file,
     content => template('apache/listen.erb'),
   }

diff --git a/manifests/namevirtualhost.pp b/manifests/namevirtualhost.pp
@@ -3,6 +3,7 @@
 
   # Template uses: $addr_port
   concat::fragment { "NameVirtualHost ${addr_port}":
+    ensure  => present,
     target  => $::apache::ports_file,
     content => template('apache/namevirtualhost.erb'),
   }