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: Ib95a8e4ec5d5f3b2b3ad9a32cb2e93873b529b62
This commit is contained in:
parent
11e694b010
commit
fd3c9bf640
@ -4,14 +4,13 @@
|
||||
#
|
||||
# === Parameters
|
||||
#
|
||||
# [*password*]
|
||||
# (Required) Password to create for the service user
|
||||
#
|
||||
# [*username*]
|
||||
# (Optional) The name of the service user
|
||||
# Defaults to 'placement'
|
||||
#
|
||||
# [*password*]
|
||||
# (Optional) Password to create for the service user
|
||||
# Defaults to $facts['os_service_default'].
|
||||
#
|
||||
# [*auth_url*]
|
||||
# (Optional) The URL to use for authentication.
|
||||
# Defaults to 'http://localhost:5000'.
|
||||
@ -196,8 +195,8 @@
|
||||
# authtoken class. Values set here override the individual parameters above.
|
||||
#
|
||||
class placement::keystone::authtoken(
|
||||
String[1] $password,
|
||||
$username = 'placement',
|
||||
$password = $facts['os_service_default'],
|
||||
$auth_url = 'http://localhost:5000',
|
||||
$project_name = 'services',
|
||||
$user_domain_name = 'Default',
|
||||
@ -238,10 +237,6 @@ class placement::keystone::authtoken(
|
||||
|
||||
include placement::deps
|
||||
|
||||
if is_service_default($password) {
|
||||
fail('Please set password for Placement service user')
|
||||
}
|
||||
|
||||
keystone::resource::authtoken {
|
||||
'placement_config':
|
||||
* => $params;
|
||||
|
Loading…
Reference in New Issue
Block a user