From 71b0fca354810715d8403a83ac668667a9e89832 Mon Sep 17 00:00:00 2001 From: Takashi Kajinami Date: Wed, 12 Jul 2023 22:06:58 +0900 Subject: [PATCH] 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 --- manifests/keystone/authtoken.pp | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/manifests/keystone/authtoken.pp b/manifests/keystone/authtoken.pp index de721610..a294124c 100644 --- a/manifests/keystone/authtoken.pp +++ b/manifests/keystone/authtoken.pp @@ -5,14 +5,13 @@ # # === Parameters # +# [*password*] +# (Required) Password to create for the service user +# # [*username*] # (Optional) The name of the service user # Defaults to 'sahara' # -# [*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:://127.0.0.1:5000' @@ -193,8 +192,8 @@ # Defaults to $facts['os_service_default']. # class sahara::keystone::authtoken( + String[1] $password, $username = 'sahara', - $password = $facts['os_service_default'], $auth_url = 'http://127.0.0.1:5000/', $project_name = 'services', $user_domain_name = 'Default',