]> review.fuel-infra Code Review - puppet-modules/puppet-ceilometer.git/commitdiff
Move service polling credentials to service_credentials
authorMathieu Gagné <mgagne@iweb.com>
Tue, 4 Mar 2014 21:57:42 +0000 (16:57 -0500)
committerMathieu Gagné <mgagne@iweb.com>
Tue, 4 Mar 2014 21:57:42 +0000 (16:57 -0500)
Change-Id: I74a523bcc1acd18532bb9353b70da63c70e8e325
Closes-bug: #1287939

manifests/agent/auth.pp
spec/classes/ceilometer_agent_auth_spec.rb

index 04579c2cc7f60b94fdd7724017093b29bad223c8..9eec3d330504dc570dc357eb0c8bf459308f35d0 100644 (file)
@@ -40,22 +40,22 @@ class ceilometer::agent::auth (
 ) {
 
   if ! $auth_cacert {
-    ceilometer_config { 'DEFAULT/os_cacert': ensure => absent }
+    ceilometer_config { 'service_credentials/os_cacert': ensure => absent }
   } else {
-    ceilometer_config { 'DEFAULT/os_cacert': value => $auth_cacert }
+    ceilometer_config { 'service_credentials/os_cacert': value => $auth_cacert }
   }
 
   ceilometer_config {
-    'DEFAULT/os_auth_url'         : value => $auth_url;
-    'DEFAULT/os_auth_region'      : value => $auth_region;
-    'DEFAULT/os_username'         : value => $auth_user;
-    'DEFAULT/os_password'         : value => $auth_password;
-    'DEFAULT/os_tenant_name'      : value => $auth_tenant_name;
+    'service_credentials/os_auth_url'    : value => $auth_url;
+    'service_credentials/os_auth_region' : value => $auth_region;
+    'service_credentials/os_username'    : value => $auth_user;
+    'service_credentials/os_password'    : value => $auth_password;
+    'service_credentials/os_tenant_name' : value => $auth_tenant_name;
   }
 
   if ($auth_tenant_id != '') {
     ceilometer_config {
-      'DEFAULT/os_tenant_id'        : value => $auth_tenant_id;
+      'service_credentials/os_tenant_id' : value => $auth_tenant_id;
     }
   }
 
index bf8feb7cb7ccec70bae8451ccdbd213d98925935..ec3b5823b38128b3a926da2b91b5feb85bf83653 100644 (file)
@@ -19,19 +19,19 @@ describe 'ceilometer::agent::auth' do
   shared_examples_for 'ceilometer-agent-auth' do
 
     it 'configures authentication' do
-      should contain_ceilometer_config('DEFAULT/os_auth_url').with_value('http://localhost:5000/v2.0')
-      should contain_ceilometer_config('DEFAULT/os_auth_region').with_value('RegionOne')
-      should contain_ceilometer_config('DEFAULT/os_username').with_value('ceilometer')
-      should contain_ceilometer_config('DEFAULT/os_password').with_value('password')
-      should contain_ceilometer_config('DEFAULT/os_tenant_name').with_value('services')
-      should contain_ceilometer_config('DEFAULT/os_cacert').with(:ensure => 'absent')
+      should contain_ceilometer_config('service_credentials/os_auth_url').with_value('http://localhost:5000/v2.0')
+      should contain_ceilometer_config('service_credentials/os_auth_region').with_value('RegionOne')
+      should contain_ceilometer_config('service_credentials/os_username').with_value('ceilometer')
+      should contain_ceilometer_config('service_credentials/os_password').with_value('password')
+      should contain_ceilometer_config('service_credentials/os_tenant_name').with_value('services')
+      should contain_ceilometer_config('service_credentials/os_cacert').with(:ensure => 'absent')
     end
 
     context 'when overriding parameters' do
       before do
         params.merge!(:auth_cacert => '/tmp/dummy.pem')
       end
-      it { should contain_ceilometer_config('DEFAULT/os_cacert').with_value(params[:auth_cacert]) }
+      it { should contain_ceilometer_config('service_credentials/os_cacert').with_value(params[:auth_cacert]) }
     end
 
   end