openstack/puppet-watcher
Code Issues Proposed changes

Remove logic dependent on the removed parameter

Browse Source 
The watcher::api::watcher_client_password parameter was removed a few
cycles ago. Actually the logic was just useless because the parameter
was not really used in the logic within the authtoken class.

Change-Id: I41fc37c2697fa132757bf1490c0a45d1bee82c2d
This commit is contained in:
Takashi Kajinami 2023-06-20 23:52:55 +09:00
parent 34cd75010d
commit 854d79fddb
1 changed files with 2 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
manifests/keystone/authtoken.pp
View File

@ -5,8 +5,7 @@
# === Parameters:
#
# [*password*]
# (Optional) Password to create for the service user
# Defaults to $facts['os_service_default']
# (Required) Password to create for the service user
#
# [*username*]
# (Optional) The name of the service user
@ -194,7 +193,7 @@
# Defaults to $facts['os_service_default'].
#
class watcher::keystone::authtoken (
$password = $facts['os_service_default'],
String[1] $password,
$username = 'watcher',
$auth_url = 'http://localhost:5000',
$project_name = 'services',
@ -235,12 +234,6 @@ class watcher::keystone::authtoken (
include watcher::deps
if is_service_default($password) and ! $::watcher::api::watcher_client_password {
fail('Please set password for watcher service user')
}
validate_legacy(String, 'validate_string', $password)
keystone::resource::authtoken { 'watcher_config':
password => $password,
username => $username,