From d545621e613f867ef84ba073588da79943f3bb61 Mon Sep 17 00:00:00 2001
From: Emilien Macchi <emilien@redhat.com>
Date: Tue, 14 Feb 2017 17:17:39 -0500
Subject: tuning: manage keystone resources only at step3

1. Manage Keystone resources only at step 3. Don't verify them
   at step 4 and 5, it's a huge loss of time.
2. Don't require Keystone resources for Gnocchi services, they are
   already ready at Step 5.

Related-Bug: #1664418
Change-Id: I9879718a1a86b862e5eb97e6f938533c96c9f5c8
---
 manifests/profile/base/gnocchi/metricd.pp |  2 --
 manifests/profile/base/gnocchi/statsd.pp  |  2 --
 manifests/profile/base/keystone.pp        | 18 +++++++-----------
 3 files changed, 7 insertions(+), 15 deletions(-)

diff --git a/manifests/profile/base/gnocchi/metricd.pp b/manifests/profile/base/gnocchi/metricd.pp
index e69bbd5..f6f80cd 100644
--- a/manifests/profile/base/gnocchi/metricd.pp
+++ b/manifests/profile/base/gnocchi/metricd.pp
@@ -30,7 +30,5 @@ class tripleo::profile::base::gnocchi::metricd (
 
   if $step >= 5 {
     include ::gnocchi::metricd
-    Keystone_endpoint<||> -> Service['gnocchi-metricd']
-    Keystone_user_role<||> -> Service['gnocchi-metricd']
   }
 }
diff --git a/manifests/profile/base/gnocchi/statsd.pp b/manifests/profile/base/gnocchi/statsd.pp
index 1fe4067..7c98a0a 100644
--- a/manifests/profile/base/gnocchi/statsd.pp
+++ b/manifests/profile/base/gnocchi/statsd.pp
@@ -30,7 +30,5 @@ class tripleo::profile::base::gnocchi::statsd (
 
   if $step >= 5 {
     include ::gnocchi::statsd
-    Keystone_endpoint<||> -> Service['gnocchi-statsd']
-    Keystone_user_role<||> -> Service['gnocchi-statsd']
   }
 }
diff --git a/manifests/profile/base/keystone.pp b/manifests/profile/base/keystone.pp
index 3ff8e63..999bcf6 100644
--- a/manifests/profile/base/keystone.pp
+++ b/manifests/profile/base/keystone.pp
@@ -157,22 +157,13 @@ class tripleo::profile::base::keystone (
       ssl_key_admin  => $tls_keyfile_admin,
     }
     include ::keystone::cors
-
-    if $manage_roles {
-      include ::keystone::roles::admin
-    }
-
-    if $manage_endpoint {
-      include ::keystone::endpoint
-    }
-
   }
 
   if $step >= 4 and $manage_db_purge {
     include ::keystone::cron::token_flush
   }
 
-  if $step >= 3 and $manage_domain {
+  if $step == 3 and $manage_domain {
     if hiera('heat_engine_enabled', false) {
       # create these seperate and don't use ::heat::keystone::domain since
       # that class writes out the configs
@@ -193,7 +184,12 @@ class tripleo::profile::base::keystone (
     }
   }
 
-  if $step >= 3 and $manage_endpoint{
+  if $step == 3 and $manage_roles {
+    include ::keystone::roles::admin
+  }
+
+  if $step == 3 and $manage_endpoint {
+    include ::keystone::endpoint
     if hiera('aodh_api_enabled', false) {
       include ::aodh::keystone::auth
     }
-- 
cgit 1.2.3-korg