From a29182c983e4bcd349ef9c069aba312575071132 Mon Sep 17 00:00:00 2001
From: =?utf8?q?Fran=C3=A7ois=20Charlier?= <fcharlier@ploup.net>
Date: Mon, 10 Dec 2012 15:58:20 +0100
Subject: [PATCH] Finish moving Keystone Auth from init.pp to api.pp

---
 manifests/api.pp  | 17 +++++++++++++----
 manifests/init.pp |  5 +----
 2 files changed, 14 insertions(+), 8 deletions(-)

diff --git a/manifests/api.pp b/manifests/api.pp
index 2db8dd9..1751a38 100644
--- a/manifests/api.pp
+++ b/manifests/api.pp
@@ -4,9 +4,12 @@ class ceilometer::api(
   $keystone_port = '35357',
   $keystone_protocol = 'http',
   $keystone_user = 'ceilometer',
-  $keystone_password,
+  $keystone_password = undef,
 ) {
 
+
+  validate_string($keystone_password)
+
   package { 'ceilometer-api':
     ensure => installed
   }
@@ -18,11 +21,17 @@ class ceilometer::api(
   }
 
   service { 'ceilometer-api':
-    name	=> $::ceilometer::params::api_service_name
-    enable      => $enabled,
+    name       => $::ceilometer::params::api_service_name,
+    enable     => $enabled,
     hasstatus  => true,
     hasrestart => true,
-    require => Package['ceilometer-api']
+    require    => Package['ceilometer-api']
+  }
+
+  ceilometer_setting {
+    'keystone_authtoken/auth_host': value => $keystone_host;
+    'keystone_authtoken/auth_port': value => $keystone_port;
+    'keystone_authtoken/protocol': value => $keystone_protocol;
   }
 
 }
diff --git a/manifests/init.pp b/manifests/init.pp
index c78e40f..7bd19cc 100644
--- a/manifests/init.pp
+++ b/manifests/init.pp
@@ -35,8 +35,8 @@ class ceilometer(
   }
 
   package { 'ceilometer-common':
-    name   => $::ceilometer::params::common_package_name,
     ensure => $package_ensure,
+    name   => $::ceilometer::params::common_package_name,
   }
 
   ceilometer_setting {
@@ -48,9 +48,6 @@ class ceilometer(
     'DEFAULT/debug': value => $debug;
     'DEFAULT/verbose': value => $verbose;
     'DEFAULT/database_connection': value => $database_connection;
-    'keystone_authtoken/auth_host': value => $keystone_host;
-    'keystone_authtoken/auth_port': value => $keystone_port;
-    'keystone_authtoken/protocol': value => $keystone_protocol;
   }
 
 
-- 
2.45.2