Use package resource directly

... instead of using ensure_resource to define a package resource,
because it is not expected that ceilometer packages are managed outside
of puppet-ceilometer.

Change-Id: I541450707b64277b30d55f136d57b8cf4443cfd4

diff --git a/manifests/agent/notification.pp b/manifests/agent/notification.pp
index 0c2eaadbd13e50f11f4708251deec95ee5824460..cf9823f466b659a072303381f9b652f3fad4c931 100644 (file)
--- a/manifests/agent/notification.pp
+++ b/manifests/agent/notification.pp
@@ -117,12 +117,11 @@ class ceilometer::agent::notification (
     $workers_real = $workers
   }
 
-  ensure_resource('package', [$::ceilometer::params::agent_notification_package_name],
-    {
-      ensure => $package_ensure,
-      tag    => ['openstack', 'ceilometer-package']
-    }
-  )
+  package { 'ceilometer-notification':
+    ensure => $package_ensure,
+    name   => $::ceilometer::params::agent_notification_package_name,
+    tag    => ['openstack', 'ceilometer-package']
+  }
 
   if $manage_service {
     if $enabled {

diff --git a/spec/classes/ceilometer_agent_notification_spec.rb b/spec/classes/ceilometer_agent_notification_spec.rb
index 39bbbacc79fe9161100b0e6bae5259de09659b8e..633060d24f2c6ea06d82e64762db19b649b632f7 100644 (file)
--- a/spec/classes/ceilometer_agent_notification_spec.rb
+++ b/spec/classes/ceilometer_agent_notification_spec.rb
@@ -37,8 +37,9 @@ describe 'ceilometer::agent::notification' do
     it { is_expected.to contain_class('ceilometer::params') }
 
     it 'installs ceilometer agent notification package' do
-      is_expected.to contain_package(platform_params[:agent_notification_package_name]).with(
+      is_expected.to contain_package('ceilometer-notification').with(
         :ensure => 'present',
+        :name   => platform_params[:agent_notification_package_name],
         :tag    => ['openstack', 'ceilometer-package'],
       )
     end