Merge "Update default value of attach_encrypted_volume"

This commit is contained in:
Zuul
2025-08-15 17:00:28 +00:00
committed by Gerrit Code Review
3 changed files with 10 additions and 3 deletions

View File

@@ -113,7 +113,7 @@
# [*ssh_key_type*]
# Defaults to $facts['os_service_default']
# [*attach_encrypted_volume*]
# Defaults to false
# Defaults to $facts['os_service_default']
# [*resize_available*]
# Defaults to $facts['os_service_default']
# [*vnc_console*]
@@ -440,7 +440,7 @@ class tempest(
$volume_build_timeout = $facts['os_service_default'],
$object_storage_build_timeout = $facts['os_service_default'],
# testing features that are supported
$attach_encrypted_volume = false,
$attach_encrypted_volume = $facts['os_service_default'],
$resize_available = $facts['os_service_default'],
$vnc_console = $facts['os_service_default'],
$use_dynamic_credentials = $facts['os_service_default'],

View File

@@ -0,0 +1,7 @@
---
upgrade:
- |
The ``tempest::attach_encrypted_volume`` parameter now defaults to
the ``os_service_default`` fact, which means the corresponding option is
cleared by default. Set this parameter in case the previous default value
(`false`) is preferred.

View File

@@ -215,7 +215,7 @@ describe 'tempest' do
is_expected.to contain_tempest_config('volume/build_interval').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('volume/build_timeout').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('object-storage/build_timeout').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('compute-feature-enabled/attach_encrypted_volume').with(:value => false)
is_expected.to contain_tempest_config('compute-feature-enabled/attach_encrypted_volume').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('compute-feature-enabled/resize').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('compute-feature-enabled/vnc_console').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('validation/image_ssh_user').with(:value => nil)