vault: Drop redundant hard-coded default of use_ssl

... because the option defaults to false in Barbican itself.

Change-Id: I328421404969571111a24d9c8a42c3da884a512b
This commit is contained in:
Takashi Kajinami 2023-12-15 16:50:11 +09:00
parent 19f9032983
commit 01ececb278
2 changed files with 3 additions and 3 deletions

View File

@ -26,7 +26,7 @@
# #
# [*use_ssl*] # [*use_ssl*]
# (optional) Enable or disable SSL # (optional) Enable or disable SSL
# Defaults to false # Defaults to $facts['os_service_default']
# #
# [*ssl_ca_crt_file*] # [*ssl_ca_crt_file*]
# (optional) Set the ssl CA cert file # (optional) Set the ssl CA cert file
@ -42,7 +42,7 @@ class barbican::plugins::vault (
$approle_role_id = $facts['os_service_default'], $approle_role_id = $facts['os_service_default'],
$approle_secret_id = $facts['os_service_default'], $approle_secret_id = $facts['os_service_default'],
$kv_mountpoint = $facts['os_service_default'], $kv_mountpoint = $facts['os_service_default'],
$use_ssl = false, $use_ssl = $facts['os_service_default'],
$ssl_ca_crt_file = $facts['os_service_default'], $ssl_ca_crt_file = $facts['os_service_default'],
$global_default = false, $global_default = false,
) { ) {

View File

@ -91,7 +91,7 @@ describe 'barbican::plugins::vault' do
is_expected.to contain_barbican_config('vault_plugin/kv_mountpoint') \ is_expected.to contain_barbican_config('vault_plugin/kv_mountpoint') \
.with_value('<SERVICE DEFAULT>') .with_value('<SERVICE DEFAULT>')
is_expected.to contain_barbican_config('vault_plugin/use_ssl') \ is_expected.to contain_barbican_config('vault_plugin/use_ssl') \
.with_value('false') .with_value('<SERVICE DEFAULT>')
is_expected.to contain_barbican_config('vault_plugin/ssl_ca_crt_file') \ is_expected.to contain_barbican_config('vault_plugin/ssl_ca_crt_file') \
.with_value('<SERVICE DEFAULT>') .with_value('<SERVICE DEFAULT>')
is_expected.to contain_barbican_config( is_expected.to contain_barbican_config(