]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Fix the ceilometer::client class
authorFrançois Charlier <francois.charlier@enovance.com>
Sat, 20 Apr 2013 02:54:31 +0000 (04:54 +0200)
committerFrançois Charlier <francois.charlier@enovance.com>
Sat, 20 Apr 2013 06:29:06 +0000 (08:29 +0200)
the client python library is python-ceilometerclient
• fix package name on Debian platforms
• fix Package resource name

manifests/client.pp
manifests/params.pp
spec/classes/ceilometer_client_spec.rb

index 83588a2e8ef100cf675441624e166fc29da2916e..7bb73351ee4cf83bdad5cd0b54c03e5be71b1c37 100644 (file)
@@ -10,7 +10,7 @@ class ceilometer::client (
 
   include ceilometer::params
 
-  package { 'python-ceilometer':
+  package { 'python-ceilometerclient':
     ensure => $ensure,
     name   => $::ceilometer::params::client_package_name,
   }
index e1c70ff4788ea1302695d0ba83e99369820b9da9..f2a2c1fc1d3c59e1b041c874e53117e7ac00c7e1 100644 (file)
@@ -27,7 +27,7 @@ class ceilometer::params {
       $api_package_name           = 'ceilometer-api'
       $collector_package_name     = 'ceilometer-collector'
       $common_package_name        = 'ceilometer-common'
-      $client_package_name        = 'python-ceilometer'
+      $client_package_name        = 'python-ceilometerclient'
       # service names
       $agent_central_service_name = 'ceilometer-agent-central'
       $agent_compute_service_name = 'ceilometer-agent-compute'
index 432f8c117259dcf18238e3ac7af6ff3b14ec9b61..0da9e1eca96b8b03110363f179635658aa514e93 100644 (file)
@@ -7,7 +7,7 @@ describe 'ceilometer::client' do
     it { should include_class('ceilometer::params') }
 
     it 'installs ceilometer client package' do
-      should contain_package('python-ceilometer').with(
+      should contain_package('python-ceilometerclient').with(
         :ensure => 'present',
         :name   => platform_params[:client_package_name]
       )
@@ -20,7 +20,7 @@ describe 'ceilometer::client' do
     end
 
     let :platform_params do
-      { :client_package_name => 'python-ceilometer' }
+      { :client_package_name => 'python-ceilometerclient' }
     end
 
     it_configures 'ceilometer client'