From 3bda006e6e40a3581f4f0893600066312ab027e3 Mon Sep 17 00:00:00 2001
From: "Matthew J. Black" <mjblack@gmail.com>
Date: Wed, 11 Jan 2017 17:18:55 -0500
Subject: [PATCH] Allow python-memcache install from authtoken class

The python-memcache package is required if using memcached. By
default the package is not installed and the define has it set to
false. This change allows managing the python-memcache package
install from the authtoken class.

Change-Id: If8fea78f01e1bf54689f9ac9cb69ae4ca6be19dc
---
 manifests/keystone/authtoken.pp                             | 6 ++++++
 .../authtoken_manage_memcache_package-56ab9dcc6745492c.yaml | 3 +++
 spec/classes/ceilometer_keystone_authtoken_spec.rb          | 5 +++++
 3 files changed, 14 insertions(+)
 create mode 100644 releasenotes/notes/authtoken_manage_memcache_package-56ab9dcc6745492c.yaml

diff --git a/manifests/keystone/authtoken.pp b/manifests/keystone/authtoken.pp
index f982378b..3acbf64c 100644
--- a/manifests/keystone/authtoken.pp
+++ b/manifests/keystone/authtoken.pp
@@ -164,6 +164,10 @@
 #   caching. If left undefined, tokens will instead be cached in-process.
 #   Defaults to $::os_service_default.
 #
+# [*manage_memcache_package*]
+#  (Optional) Whether to install the python-memcache package.
+#  Defaults to false.
+#
 # [*region_name*]
 #   (Optional) The region in which the identity server can be found.
 #   Defaults to $::os_service_default.
@@ -219,6 +223,7 @@ class ceilometer::keystone::authtoken(
   $memcache_security_strategy     = $::os_service_default,
   $memcache_use_advanced_pool     = $::os_service_default,
   $memcached_servers              = $::os_service_default,
+  $manage_memcache_package        = false,
   $region_name                    = $::os_service_default,
   $revocation_cache_time          = $::os_service_default,
   $token_cache_time               = $::os_service_default,
@@ -268,6 +273,7 @@ class ceilometer::keystone::authtoken(
     memcache_use_advanced_pool     => $memcache_use_advanced_pool,
     memcache_pool_unused_timeout   => $memcache_pool_unused_timeout,
     memcached_servers              => $memcached_servers,
+    manage_memcache_package        => $manage_memcache_package,
     region_name                    => $region_name,
     revocation_cache_time          => $revocation_cache_time,
     token_cache_time               => $token_cache_time,
diff --git a/releasenotes/notes/authtoken_manage_memcache_package-56ab9dcc6745492c.yaml b/releasenotes/notes/authtoken_manage_memcache_package-56ab9dcc6745492c.yaml
new file mode 100644
index 00000000..09c6d5a7
--- /dev/null
+++ b/releasenotes/notes/authtoken_manage_memcache_package-56ab9dcc6745492c.yaml
@@ -0,0 +1,3 @@
+---
+features:
+  - Passes parameter to keystone authtoken define to manage python-memcache package install.
diff --git a/spec/classes/ceilometer_keystone_authtoken_spec.rb b/spec/classes/ceilometer_keystone_authtoken_spec.rb
index 0c203f8b..4bea6a1b 100644
--- a/spec/classes/ceilometer_keystone_authtoken_spec.rb
+++ b/spec/classes/ceilometer_keystone_authtoken_spec.rb
@@ -82,6 +82,7 @@ describe 'ceilometer::keystone::authtoken' do
           :memcache_security_strategy           => 'ENCRYPT',
           :memcache_use_advanced_pool           => true,
           :memcached_servers                    => ['memcached01:11211','memcached02:11211'],
+          :manage_memcache_package              => true,
           :region_name                          => 'region2',
           :revocation_cache_time                => '11',
           :token_cache_time                     => '301',
@@ -124,6 +125,10 @@ describe 'ceilometer::keystone::authtoken' do
         is_expected.to contain_ceilometer_config('keystone_authtoken/revocation_cache_time').with_value(params[:revocation_cache_time])
         is_expected.to contain_ceilometer_config('keystone_authtoken/token_cache_time').with_value(params[:token_cache_time])
       end
+
+      it 'installs python memcache package' do
+        is_expected.to contain_package('python-memcache')
+      end
     end
   end