From d60cd845022679f70c382f355bd59f5007210e17 Mon Sep 17 00:00:00 2001
From: Juan Antonio Osorio Robles <jaosorior@redhat.com>
Date: Wed, 1 Nov 2017 15:15:34 +0200
Subject: [PATCH] Only enable keystone notifications if telemetry is enabled

This will get rid of the unnecessary notifications that were being
produced and help with the undercloud's memory consumption.

Change-Id: If4d97e5069ca678edbcd2270a75f6faf67364420
Closes-Bug: #1729293
---
 elements/puppet-stack-config/puppet-stack-config.pp | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/elements/puppet-stack-config/puppet-stack-config.pp b/elements/puppet-stack-config/puppet-stack-config.pp
index 8366b030f..9dcd76707 100644
--- a/elements/puppet-stack-config/puppet-stack-config.pp
+++ b/elements/puppet-stack-config/puppet-stack-config.pp
@@ -320,8 +320,15 @@ if hiera('tripleo::haproxy::service_certificate', undef) {
   $enable_proxy_headers_parsing = false
 }
 
+if str2bool(hiera('enable_telemetry', false)) {
+  $notification_topics = ['notifications']
+} else {
+  $notification_topics = []
+}
+
 class { '::keystone':
   enable_proxy_headers_parsing => $enable_proxy_headers_parsing,
+  notification_topics          => $notification_topics,
 }
 include ::keystone::wsgi::apache
 include ::keystone::cron::token_flush