Expose configure_service

This parameter has been implemented in the underlying defined resource
type but was not configurable.

Change-Id: I9439d4b37cda8be922a4a1b8a6cc3acc7078ec6d

manifests/keystone/auth.pp

@@ -60,6 +60,10 @@
 #   (Optional) Should the admin role be configured for the service user?
 #   Defaults to true.
 #
+# [*configure_service*]
+#   (Optional) Should the service be configurd?
+#   Defaults to True
+#
 # [*service_name*]
 #   (Optional) Name of the service.
 #   Defaults to the value of 'ironic'.
@@ -110,6 +114,7 @@ class ironic::keystone::auth (
   Boolean $configure_endpoint             = true,
   Boolean $configure_user                 = true,
   Boolean $configure_user_role            = true,
+  Boolean $configure_service              = true,
   String[1] $service_name                 = 'ironic',
   String[1] $service_type                 = 'baremetal',
   String[1] $service_description          = 'Ironic Bare Metal Provisioning Service',
@@ -127,6 +132,7 @@ class ironic::keystone::auth (
     configure_user      => $configure_user,
     configure_user_role => $configure_user_role,
     configure_endpoint  => $configure_endpoint,
+    configure_service   => $configure_service,
     service_name        => $service_name,
     service_type        => $service_type,
     auth_name           => $auth_name,

manifests/keystone/auth_inspector.pp

@@ -56,6 +56,10 @@
 #   (Optional) Should the admin role be configured for the service user?
 #   Defaults to 'true'.
 #
+# [*configure_service*]
+#   (Optional) Should the service be configurd?
+#   Defaults to True
+#
 # [*service_name*]
 #   (Optional) Name of the service.
 #   Defaults to the value of auth_name, but must differ from the value.
@@ -101,6 +105,7 @@ class ironic::keystone::auth_inspector (
   Boolean $configure_endpoint             = true,
   Boolean $configure_user                 = true,
   Boolean $configure_user_role            = true,
+  Boolean $configure_service              = true,
   Optional[String[1]] $service_name       = undef,
   String[1] $service_type                 = 'baremetal-introspection',
   String[1] $service_description          = 'Bare Metal Introspection Service',
@@ -120,6 +125,7 @@ class ironic::keystone::auth_inspector (
     configure_user      => $configure_user,
     configure_user_role => $configure_user_role,
     configure_endpoint  => $configure_endpoint,
+    configure_service   => $configure_service,
     service_name        => $real_service_name,
     service_type        => $service_type,
     auth_name           => $auth_name,

releasenotes/notes/configure_service-d3637d5ac22e0496.yaml

@@ -0,0 +1,7 @@
+---
+features:
+  - |
+    The following two parameter have been added.
+
+    - ``ironic::keystone::auth::configure_service``
+    - ``ironic::keystone::auth_inspector::configure_service``

spec/classes/ironic_keystone_auth_inspector_spec.rb

@@ -32,6 +32,7 @@ describe 'ironic::keystone::auth_inspector' do
         :configure_user      => true,
         :configure_user_role => true,
         :configure_endpoint  => true,
+        :configure_service   => true,
         :service_name        => 'ironic-inspector',
         :service_type        => 'baremetal-introspection',
         :auth_name           => 'ironic-inspector',
@@ -61,6 +62,7 @@ describe 'ironic::keystone::auth_inspector' do
           :configure_endpoint  => false,
           :configure_user      => false,
           :configure_user_role => false,
+          :configure_service   => false,
           :service_description => 'Alternative Bare Metal Introspection Service',
           :service_name        => 'alt_service',
           :service_type        => 'alt_baremetal-introspection',
@@ -74,6 +76,7 @@ describe 'ironic::keystone::auth_inspector' do
         :configure_user      => false,
         :configure_user_role => false,
         :configure_endpoint  => false,
+        :configure_service   => false,
         :service_name        => 'alt_service',
         :service_type        => 'alt_baremetal-introspection',
         :auth_name           => 'alt_ironic-inspector',

spec/classes/ironic_keystone_auth_spec.rb

@@ -31,6 +31,7 @@ describe 'ironic::keystone::auth' do
         :configure_user      => true,
         :configure_user_role => true,
         :configure_endpoint  => true,
+        :configure_service   => true,
         :service_name        => 'ironic',
         :service_type        => 'baremetal',
         :service_description => 'Ironic Bare Metal Provisioning Service',
@@ -60,6 +61,7 @@ describe 'ironic::keystone::auth' do
           :configure_endpoint  => false,
           :configure_user      => false,
           :configure_user_role => false,
+          :configure_service   => false,
           :service_description => 'Alternative Ironic Bare Metal Provisioning Service',
           :service_name        => 'alt_service',
           :service_type        => 'alt_baremetal',
@@ -73,6 +75,7 @@ describe 'ironic::keystone::auth' do
         :configure_user      => false,
         :configure_user_role => false,
         :configure_endpoint  => false,
+        :configure_service   => false,
         :service_name        => 'alt_service',
         :service_type        => 'alt_baremetal',
         :service_description => 'Alternative Ironic Bare Metal Provisioning Service',