From 93dc1078b4f911bf807ac583df926cdfc734e8f8 Mon Sep 17 00:00:00 2001
From: Dan Prince <dprince@redhat.com>
Date: Fri, 9 Dec 2016 09:37:41 +0100
Subject: Decouple swift-proxy from ceilometer packages

This patch updates the swift proxy so that it only depends
on ceilometer if the ceilometer_api_enabled all-nodes-data hiera
setting has been set.

Also removes a parameter dependency where the
tripleo::profile::base::swift::proxy class was referencing
a puppet-ceilometer value from hiera (which can
also cause ceilometer dependencies).

Depends-On: Ief5399d7ea4d26e96ce54903a69d660fa4fe3ce9

Change-Id: I8d9f69f5e9160543b372bd9886800f16f625fdc6
Closes-bug: #1648736
---
 manifests/profile/base/swift/proxy.pp | 23 +++++++++++++++--------
 1 file changed, 15 insertions(+), 8 deletions(-)

diff --git a/manifests/profile/base/swift/proxy.pp b/manifests/profile/base/swift/proxy.pp
index 974a725..7bbef1e 100644
--- a/manifests/profile/base/swift/proxy.pp
+++ b/manifests/profile/base/swift/proxy.pp
@@ -37,14 +37,19 @@
 #
 # [*rabbit_port*]
 #   IP port for rabbitmq service
-#   Defaults to hiera('swift::proxy::ceilometer::rabbit_port', 5672)
+#   Defaults to 5672
+#
+# [*ceilometer_enabled*]
+#   Whether the ceilometer pipeline is enabled.
+#   Defaults to true
 #
 class tripleo::profile::base::swift::proxy (
-  $step             = hiera('step'),
-  $memcache_servers = hiera('memcached_node_ips'),
-  $memcache_port    = 11211,
-  $rabbit_hosts     = hiera('rabbitmq_node_names', undef),
-  $rabbit_port      = hiera('swift::proxy::ceilometer::rabbit_port', 5672),
+  $step               = hiera('step'),
+  $memcache_servers   = hiera('memcached_node_ips'),
+  $memcache_port      = 11211,
+  $rabbit_hosts       = hiera('rabbitmq_node_names', undef),
+  $rabbit_port        = 5672,
+  $ceilometer_enabled = true,
 ) {
   if $step >= 4 {
     $swift_memcache_servers = suffix(any2array(normalize_ip_for_uri($memcache_servers)), ":${memcache_port}")
@@ -64,8 +69,10 @@ class tripleo::profile::base::swift::proxy (
     include ::swift::proxy::formpost
     include ::swift::proxy::bulk
     $swift_rabbit_hosts = suffix(any2array($rabbit_hosts), ":${rabbit_port}")
-    class { '::swift::proxy::ceilometer':
-      rabbit_hosts => $swift_rabbit_hosts,
+    if $ceilometer_enabled {
+      class { '::swift::proxy::ceilometer':
+        rabbit_hosts => $swift_rabbit_hosts,
+      }
     }
     include ::swift::proxy::versioned_writes
     include ::swift::proxy::slo
-- 
cgit 1.2.3-korg