From d12931f1f03958d7731e747fb005c77487bed5b5 Mon Sep 17 00:00:00 2001
From: Takashi Kajinami <kajinamit@oss.nttdata.com>
Date: Sat, 12 Oct 2024 01:27:11 +0900
Subject: [PATCH] Expose configure_service

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

Change-Id: I26fdedb6ec221295aa5d78e300997bb8f7055220
---
 manifests/keystone/auth.pp                                 | 6 ++++++
 releasenotes/notes/configure_service-ac32ead265b7da2e.yaml | 5 +++++
 spec/classes/placement_keystone_auth_spec.rb               | 3 +++
 3 files changed, 14 insertions(+)
 create mode 100644 releasenotes/notes/configure_service-ac32ead265b7da2e.yaml

diff --git a/manifests/keystone/auth.pp b/manifests/keystone/auth.pp
index 69f1ef1..5a7b5d4 100644
--- a/manifests/keystone/auth.pp
+++ b/manifests/keystone/auth.pp
@@ -47,6 +47,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_type*]
 #   (Optional) Type of service.
 #   Defaults to 'placement'.
@@ -89,6 +93,7 @@ class placement::keystone::auth (
   Boolean $configure_endpoint             = true,
   Boolean $configure_user                 = true,
   Boolean $configure_user_role            = true,
+  Boolean $configure_service              = true,
   String[1] $service_name                 = 'placement',
   String[1] $service_description          = 'OpenStack Placement Service',
   String[1] $service_type                 = 'placement',
@@ -107,6 +112,7 @@ class placement::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,
     service_description => $service_description,
diff --git a/releasenotes/notes/configure_service-ac32ead265b7da2e.yaml b/releasenotes/notes/configure_service-ac32ead265b7da2e.yaml
new file mode 100644
index 0000000..535f807
--- /dev/null
+++ b/releasenotes/notes/configure_service-ac32ead265b7da2e.yaml
@@ -0,0 +1,5 @@
+---
+features:
+  - |
+    The new ``placement::keystone::auth::configure_service`` parameter has been
+    added.
diff --git a/spec/classes/placement_keystone_auth_spec.rb b/spec/classes/placement_keystone_auth_spec.rb
index 80edc98..37d3507 100644
--- a/spec/classes/placement_keystone_auth_spec.rb
+++ b/spec/classes/placement_keystone_auth_spec.rb
@@ -15,6 +15,7 @@ describe 'placement::keystone::auth' do
         :configure_user      => true,
         :configure_user_role => true,
         :configure_endpoint  => true,
+        :configure_service   => true,
         :service_name        => 'placement',
         :service_type        => 'placement',
         :service_description => 'OpenStack Placement Service',
@@ -44,6 +45,7 @@ describe 'placement::keystone::auth' do
           :configure_endpoint  => false,
           :configure_user      => false,
           :configure_user_role => false,
+          :configure_service   => false,
           :service_description => 'Alternative OpenStack Placement Service',
           :service_name        => 'alt_service',
           :service_type        => 'alt_placement',
@@ -57,6 +59,7 @@ describe 'placement::keystone::auth' do
         :configure_user      => false,
         :configure_user_role => false,
         :configure_endpoint  => false,
+        :configure_service   => false,
         :service_name        => 'alt_service',
         :service_type        => 'alt_placement',
         :service_description => 'Alternative OpenStack Placement Service',