From 22723cce88a65a44afac3065ad800b4c0008f75d Mon Sep 17 00:00:00 2001
From: Emilien Macchi <emilien@redhat.com>
Date: Thu, 24 Sep 2015 09:35:59 -0400
Subject: [PATCH] Implement auth_endpoint_type parameter

Add auth_endpoint_type which is the type of endpoint in Identity service
catalog to use for communication with OpenStack services. Default to
undef.

Change-Id: Ia62186b3a4520e1f323a4fa96d8a925874b3604a
---
 manifests/agent/auth.pp                    | 24 ++++++++++++++++------
 spec/classes/ceilometer_agent_auth_spec.rb |  6 +++++-
 2 files changed, 23 insertions(+), 7 deletions(-)

diff --git a/manifests/agent/auth.pp b/manifests/agent/auth.pp
index 360989b..373a26e 100644
--- a/manifests/agent/auth.pp
+++ b/manifests/agent/auth.pp
@@ -29,14 +29,20 @@
 #  [*auth_cacert*]
 #    Certificate chain for SSL validation. Optional; Defaults to 'None'
 #
+#  [*auth_endpoint_type*]
+#    Type of endpoint in Identity service catalog to use for
+#    communication with OpenStack services.
+#    Optional. Defaults to undef.
+#
 class ceilometer::agent::auth (
   $auth_password,
-  $auth_url         = 'http://localhost:5000/v2.0',
-  $auth_region      = 'RegionOne',
-  $auth_user        = 'ceilometer',
-  $auth_tenant_name = 'services',
-  $auth_tenant_id   = undef,
-  $auth_cacert      = undef,
+  $auth_url           = 'http://localhost:5000/v2.0',
+  $auth_region        = 'RegionOne',
+  $auth_user          = 'ceilometer',
+  $auth_tenant_name   = 'services',
+  $auth_tenant_id     = undef,
+  $auth_cacert        = undef,
+  $auth_endpoint_type = undef,
 ) {
 
   if ! $auth_cacert {
@@ -59,4 +65,10 @@ class ceilometer::agent::auth (
     }
   }
 
+  if $auth_endpoint_type {
+    ceilometer_config {
+      'service_credentials/os_endpoint_type' : value => $auth_endpoint_type;
+    }
+  }
+
 }
diff --git a/spec/classes/ceilometer_agent_auth_spec.rb b/spec/classes/ceilometer_agent_auth_spec.rb
index 14b1904..7e7c172 100644
--- a/spec/classes/ceilometer_agent_auth_spec.rb
+++ b/spec/classes/ceilometer_agent_auth_spec.rb
@@ -29,9 +29,13 @@ describe 'ceilometer::agent::auth' do
 
     context 'when overriding parameters' do
       before do
-        params.merge!(:auth_cacert => '/tmp/dummy.pem')
+        params.merge!(
+          :auth_cacert        => '/tmp/dummy.pem',
+          :auth_endpoint_type => 'internalURL',
+        )
       end
       it { is_expected.to contain_ceilometer_config('service_credentials/os_cacert').with_value(params[:auth_cacert]) }
+      it { is_expected.to contain_ceilometer_config('service_credentials/os_endpoint_type').with_value(params[:auth_endpoint_type]) }
     end
 
   end
-- 
2.45.2