Merge "Add manage_package parameter"

manifests/init.pp

@@ -3,6 +3,10 @@
 #
 # == Parameters
 #
+# [*manage_package*]
+#   (Optional) Manage package resources.
+#   Defaults to true.
+#
 # [*package_ensure*]
 #   (Optional) Desired ensure state of packages.
 #   accepts latest or specific versions.
@@ -368,6 +372,7 @@
 # Copyright 2012 Puppetlabs Inc, unless otherwise noted.
 #
 class keystone(
+  Boolean $manage_package                         = true,
   $package_ensure                                 = 'present',
   $catalog_driver                                 = $facts['os_service_default'],
   $token_provider                                 = 'fernet',
@@ -466,12 +471,14 @@ class keystone(
   include keystone::db
   include keystone::params
 
-  package { 'keystone':
-    ensure => $package_ensure,
-    name   => $::keystone::params::package_name,
-    tag    => ['openstack', 'keystone-package'],
+  if $manage_package {
+    package { 'keystone':
+      ensure => $package_ensure,
+      name   => $::keystone::params::package_name,
+      tag    => ['openstack', 'keystone-package'],
+    }
+    include openstacklib::openstackclient
   }
-  include openstacklib::openstackclient
 
   resources { 'keystone_config':
     purge  => $purge_config,

releasenotes/notes/keystone-manage-package-6770a8a31ead11a4.yaml

@@ -0,0 +1,4 @@
+---
+features:
+  - |
+    Added ``manage_package`` parameter to the ``keystone`` class.

spec/classes/keystone_init_spec.rb

@@ -239,6 +239,15 @@ describe 'keystone' do
       it { is_expected.to_not contain_service('keystone') }
     end
 
+    context 'with disabled package managing' do
+      let :params do
+        { :manage_package => false }
+      end
+
+      it { is_expected.to_not contain_package('keystone') }
+      it { is_expected.to_not contain_class('openstacklib::openstackclient') }
+    end
+
     context 'when sync_db is set to false' do
       let :params do
         {