From 552d55fe5d55f8c0a531ea8def21dc3a739e36a2 Mon Sep 17 00:00:00 2001
From: Emilien Macchi <emilien.macchi@enovance.com>
Date: Wed, 19 Nov 2014 23:35:11 -0500
Subject: [PATCH] Refactorise Keystone resources management

Refactorise the code of Keystone resources management with backward
compatibility since we don't modify the unit tests.

Change-Id: I8fc27ea6cc49a4c0c518f7314b3277d6980491b4
Implements: blueprint common-openstack-identity-resource
---
 manifests/keystone/auth.pp | 41 +++++++++++++-------------------------
 1 file changed, 14 insertions(+), 27 deletions(-)

diff --git a/manifests/keystone/auth.pp b/manifests/keystone/auth.pp
index 2bc061f42..7abdbc488 100644
--- a/manifests/keystone/auth.pp
+++ b/manifests/keystone/auth.pp
@@ -94,37 +94,24 @@ class neutron::keystone::auth (
 
   Keystone_endpoint["${region}/${real_service_name}"]  ~> Service <| name == 'neutron-server' |>
 
-  if $configure_user {
-    keystone_user { $auth_name:
-      ensure   => present,
-      password => $password,
-      email    => $email,
-      tenant   => $tenant,
-    }
-  }
-
   if $configure_user_role {
     Keystone_user_role["${auth_name}@${tenant}"] ~> Service <| name == 'neutron-server' |>
-
-    keystone_user_role { "${auth_name}@${tenant}":
-      ensure => present,
-      roles  => 'admin',
-    }
   }
 
-  keystone_service { $real_service_name:
-    ensure      => present,
-    type        => $service_type,
-    description => 'Neutron Networking Service',
+  keystone::resource::service_identity { $auth_name:
+    configure_user      => $configure_user,
+    configure_user_role => $configure_user_role,
+    configure_endpoint  => $configure_endpoint,
+    service_type        => $service_type,
+    service_description => 'Neutron Networking Service',
+    service_name        => $real_service_name,
+    region              => $region,
+    password            => $password,
+    email               => $email,
+    tenant              => $tenant,
+    public_url          => "${public_protocol}://${public_address}:${real_public_port}/",
+    admin_url           => "${admin_protocol}://${admin_address}:${port}/",
+    internal_url        => "${internal_protocol}://${internal_address}:${port}/",
   }
 
-  if $configure_endpoint {
-    keystone_endpoint { "${region}/${real_service_name}":
-      ensure       => present,
-      public_url   => "${public_protocol}://${public_address}:${real_public_port}/",
-      internal_url => "${internal_protocol}://${internal_address}:${port}/",
-      admin_url    => "${admin_protocol}://${admin_address}:${port}/",
-    }
-
-  }
 }