openstack
/
puppet-sahara
Code Issues Proposed changes

authtoken: Make password required 

The password parameter is not really optional. This makes it
a required parameter to give more sensible validation error.

Change-Id: I366ee11502901d749e8439f37d780eae19501d95
This commit is contained in:
Takashi Kajinami 2023-07-12 22:06:58 +09:00
parent e2b1d99501
commit 71b0fca354
1 changed files with 4 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
manifests/keystone/authtoken.pp
View File

@ -5,14 +5,13 @@
# #
# === Parameters # === Parameters
# #
# [*password*]
# (Required) Password to create for the service user
#
# [*username*] # [*username*]
# (Optional) The name of the service user # (Optional) The name of the service user
# Defaults to 'sahara' # Defaults to 'sahara'
# #
# [*password*]
# (Optional) Password to create for the service user
# Defaults to $facts['os_service_default']
#
# [*auth_url*] # [*auth_url*]
# (Optional) The URL to use for authentication. # (Optional) The URL to use for authentication.
# Defaults to 'http:://127.0.0.1:5000' # Defaults to 'http:://127.0.0.1:5000'
@ -193,8 +192,8 @@
# Defaults to $facts['os_service_default']. # Defaults to $facts['os_service_default'].
# #
class sahara::keystone::authtoken( class sahara::keystone::authtoken(
String[1] $password,
$username = 'sahara', $username = 'sahara',
$password = $facts['os_service_default'],
$auth_url = 'http://127.0.0.1:5000/', $auth_url = 'http://127.0.0.1:5000/',
$project_name = 'services', $project_name = 'services',
$user_domain_name = 'Default', $user_domain_name = 'Default',