openstack/puppet-placement
Code Issues Proposed changes

Add validations for keystone resource parameters

Browse Source 
The underlying implementation already have type validations. Inherit
these and make a few adjustment so that validation failures are raised
from the external interface layer instead of the internal interface.

Change-Id: I9caa6e0247646c9a97c1d1a87a282503d2750241
This commit is contained in:
Takashi Kajinami 2024-09-16 23:46:00 +09:00
parent ec2b643479
commit 27c1ada3ae
1 changed files with 18 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

36
manifests/keystone/auth.pp
View File

@ -78,24 +78,24 @@
# Defaults to 'http://127.0.0.1:8778' # Defaults to 'http://127.0.0.1:8778'
# #
class placement::keystone::auth ( class placement::keystone::auth (
$password, String[1] $password,
$ensure = 'present', String[1] $ensure = 'present',
$auth_name = 'placement', String[1] $auth_name = 'placement',
$email = 'placement@localhost', String[1] $email = 'placement@localhost',
$tenant = 'services', String[1] $tenant = 'services',
$roles = ['admin'], Array[String[1]] $roles = ['admin'],
$system_scope = 'all', String[1] $system_scope = 'all',
$system_roles = [], Array[String[1]] $system_roles = [],
$configure_endpoint = true, Boolean $configure_endpoint = true,
$configure_user = true, Boolean $configure_user = true,
$configure_user_role = true, Boolean $configure_user_role = true,
$service_name = 'placement', String[1] $service_name = 'placement',
$service_description = 'OpenStack Placement Service', String[1] $service_description = 'OpenStack Placement Service',
$service_type = 'placement', String[1] $service_type = 'placement',
$region = 'RegionOne', String[1] $region = 'RegionOne',
$public_url = 'http://127.0.0.1:8778', Keystone::PublicEndpointUrl $public_url = 'http://127.0.0.1:8778',
$admin_url = 'http://127.0.0.1:8778', Keystone::EndpointUrl $admin_url = 'http://127.0.0.1:8778',
$internal_url = 'http://127.0.0.1:8778', Keystone::EndpointUrl $internal_url = 'http://127.0.0.1:8778',
) { ) {
include placement::deps include placement::deps