openstack/puppet-magnum
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: I8a9149ec7c43437f2392bcce39967e73c118dd34
This commit is contained in:
Takashi Kajinami 2024-10-12 12:15:13 +09:00
parent 77506a7dea
commit 9e88273614
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

@ -43,6 +43,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 'container-infra'. # Defaults to 'container-infra'.
@ -84,6 +88,7 @@ class magnum::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_description = 'magnum Container Service', String[1] $service_description = 'magnum Container Service',
Optional[String[1]] $service_name = undef, Optional[String[1]] $service_name = undef,
String[1] $service_type = 'container-infra', String[1] $service_type = 'container-infra',
@ -103,6 +108,7 @@ class magnum::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 => $real_service_name, service_name => $real_service_name,
service_type => $service_type, service_type => $service_type,
service_description => $service_description, service_description => $service_description,

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

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

3
spec/classes/magnum_keystone_auth_spec.rb
View File

@ -15,6 +15,7 @@ describe 'magnum::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 => 'magnum', :service_name => 'magnum',
:service_type => 'container-infra', :service_type => 'container-infra',
:service_description => 'magnum Container Service', :service_description => 'magnum Container Service',
@ -44,6 +45,7 @@ describe 'magnum::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 magnum Container Service', :service_description => 'Alternative magnum Container Service',
:service_name => 'alt_service', :service_name => 'alt_service',
:service_type => 'alt_container-infra', :service_type => 'alt_container-infra',
@ -57,6 +59,7 @@ describe 'magnum::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_container-infra', :service_type => 'alt_container-infra',
:service_description => 'Alternative magnum Container Service', :service_description => 'Alternative magnum Container Service',