]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Fix ensure_packages error
authorMark T. Voelker <mvoelker@cisco.com>
Sat, 15 Mar 2014 05:15:02 +0000 (01:15 -0400)
committerMark T. Voelker <mvoelker@cisco.com>
Sat, 15 Mar 2014 05:17:55 +0000 (01:17 -0400)
The ceilometer::collector class calls ensure_packages()
on a single package name, but the package name is provided
as a string rather than a one-element array.  This causes an
error of the form:

Error: ensure_packages(): Requires array given (String) at
/usr/share/puppet/modules/ceilometer/manifests/collector.pp:16

This patch corrects the error by passing an array instead
of a string.

Change-Id: I96cedf7561af29d92924746881c52bdd1502365f
Closes-Bug: #1292775

manifests/collector.pp

index fe2bf1c14ac4ee69ca1fa60dc48ce01edb1d3955..cc0661f6951b4e9cc0603b933246e526058a6bae 100644 (file)
@@ -13,7 +13,7 @@ class ceilometer::collector (
   Ceilometer_config<||> ~> Service['ceilometer-collector']
 
   Package[$::ceilometer::params::collector_package_name] -> Service['ceilometer-collector']
-  ensure_packages($::ceilometer::params::collector_package_name)
+  ensure_packages([$::ceilometer::params::collector_package_name])
 
   if $enabled {
     $service_ensure = 'running'