From: Mark T. Voelker <mvoelker@cisco.com>
Date: Sat, 15 Mar 2014 05:15:02 +0000 (-0400)
Subject: Fix ensure_packages error
X-Git-Tag: 4.0.0~9^2
X-Git-Url: https://review.fuel-infra.org/gitweb?a=commitdiff_plain;h=555df8b3a9b4ea12398bef21aa8d986fdbc2faa6;p=puppet-modules%2Fpuppet-ceilometer.git

Fix ensure_packages error

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
---

diff --git a/manifests/collector.pp b/manifests/collector.pp
index fe2bf1c..cc0661f 100644
--- a/manifests/collector.pp
+++ b/manifests/collector.pp
@@ -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'