openstack/puppet-placement
Code Issues Proposed changes

Expose configure_service

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

Change-Id: I26fdedb6ec221295aa5d78e300997bb8f7055220
This commit is contained in:
Takashi Kajinami 2024-10-12 01:27:11 +09:00
parent f379485d75
commit d12931f1f0
3 changed files with 14 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
manifests/keystone/auth.pp
View File

@ -47,6 +47,10 @@
# (Optional) Should the admin role be configured for the service user? # (Optional) Should the admin role be configured for the service user?
# Defaults to 'true'. # Defaults to 'true'.
# #
# [*configure_service*]
# (Optional) Should the service be configurd?
# Defaults to True
#
# [*service_type*] # [*service_type*]
# (Optional) Type of service. # (Optional) Type of service.
# Defaults to 'placement'. # Defaults to 'placement'.
@ -89,6 +93,7 @@ class placement::keystone::auth (
Boolean $configure_endpoint = true, Boolean $configure_endpoint = true,
Boolean $configure_user = true, Boolean $configure_user = true,
Boolean $configure_user_role = true, Boolean $configure_user_role = true,
Boolean $configure_service = true,
String[1] $service_name = 'placement', String[1] $service_name = 'placement',
String[1] $service_description = 'OpenStack Placement Service', String[1] $service_description = 'OpenStack Placement Service',
String[1] $service_type = 'placement', String[1] $service_type = 'placement',
@ -107,6 +112,7 @@ class placement::keystone::auth (
configure_user => $configure_user, configure_user => $configure_user,
configure_user_role => $configure_user_role, configure_user_role => $configure_user_role,
configure_endpoint => $configure_endpoint, configure_endpoint => $configure_endpoint,
configure_service => $configure_service,
service_name => $service_name, service_name => $service_name,
service_type => $service_type, service_type => $service_type,
service_description => $service_description, service_description => $service_description,

5
releasenotes/notes/configure_service-ac32ead265b7da2e.yaml Normal file
View File

@ -0,0 +1,5 @@
---
features:
- |
The new ``placement::keystone::auth::configure_service`` parameter has been
added.

3
spec/classes/placement_keystone_auth_spec.rb
View File

@ -15,6 +15,7 @@ describe 'placement::keystone::auth' do
:configure_user => true, :configure_user => true,
:configure_user_role => true, :configure_user_role => true,
:configure_endpoint => true, :configure_endpoint => true,
:configure_service => true,
:service_name => 'placement', :service_name => 'placement',
:service_type => 'placement', :service_type => 'placement',
:service_description => 'OpenStack Placement Service', :service_description => 'OpenStack Placement Service',
@ -44,6 +45,7 @@ describe 'placement::keystone::auth' do
:configure_endpoint => false, :configure_endpoint => false,
:configure_user => false, :configure_user => false,
:configure_user_role => false, :configure_user_role => false,
:configure_service => false,
:service_description => 'Alternative OpenStack Placement Service', :service_description => 'Alternative OpenStack Placement Service',
:service_name => 'alt_service', :service_name => 'alt_service',
:service_type => 'alt_placement', :service_type => 'alt_placement',
@ -57,6 +59,7 @@ describe 'placement::keystone::auth' do
:configure_user => false, :configure_user => false,
:configure_user_role => false, :configure_user_role => false,
:configure_endpoint => false, :configure_endpoint => false,
:configure_service => false,
:service_name => 'alt_service', :service_name => 'alt_service',
:service_type => 'alt_placement', :service_type => 'alt_placement',
:service_description => 'Alternative OpenStack Placement Service', :service_description => 'Alternative OpenStack Placement Service',