openstack/puppet-tempest
Code Issues Proposed changes

Support [service-clients] http_timeout

Browse Source 
We have occasionally seen tempest failures caused by http timeout in
CI. This introduces the parameter to tune the timeout option to lower
down the failure late (hopefully).

Change-Id: I3b82cc711b0baa18e4836d4f9dbd11c29bc5cff4
This commit is contained in:
Takashi Kajinami
2023-04-12 12:16:50 +09:00
parent 3b634a51c2
commit df2d370a52
3 changed files with 9 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/init.pp
View File

@@ -58,6 +58,8 @@
# Defaults to false # Defaults to false
# [*logging_context_format_string*] # [*logging_context_format_string*]
# Defaults to $facts['os_service_default'] # Defaults to $facts['os_service_default']
# [*http_timeout*]
# Defaults to $facts['os_service_default']
# [*username*] # [*username*]
# Defaults to undef # Defaults to undef
# [*password*] # [*password*]
@@ -334,6 +336,7 @@ class tempest(
$use_stderr = true, $use_stderr = true,
$use_syslog = false, $use_syslog = false,
$logging_context_format_string = $facts['os_service_default'], $logging_context_format_string = $facts['os_service_default'],
$http_timeout = $facts['os_service_default'],
$attach_encrypted_volume = false, $attach_encrypted_volume = false,
# non admin user # non admin user
$username = undef, $username = undef,
@@ -572,6 +575,7 @@ class tempest(
tempest_config { tempest_config {
'service-clients/http_timeout': value => $http_timeout;
'auth/admin_domain_name': value => $admin_domain_name; 'auth/admin_domain_name': value => $admin_domain_name;
'auth/admin_project_domain_name': value => $admin_project_domain_name; 'auth/admin_project_domain_name': value => $admin_project_domain_name;
'auth/admin_user_domain_name': value => $admin_user_domain_name; 'auth/admin_user_domain_name': value => $admin_user_domain_name;

4
releasenotes/notes/http_timeout-a0d7c667102fa106.yaml Normal file
View File

@@ -0,0 +1,4 @@
---
features:
- |
The new ``tempest::http_timeout`` parameter has been added.

1
spec/classes/tempest_init_spec.rb
View File

@@ -198,6 +198,7 @@ describe 'tempest' do
it { should contain_class('openstacklib::openstackclient') } it { should contain_class('openstacklib::openstackclient') }
it 'configure tempest config' do it 'configure tempest config' do
is_expected.to contain_tempest_config('service-clients/http_timeout').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('auth/admin_domain_name').with(:value => '<SERVICE DEFAULT>') is_expected.to contain_tempest_config('auth/admin_domain_name').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('auth/admin_project_domain_name').with(:value => '<SERVICE DEFAULT>') is_expected.to contain_tempest_config('auth/admin_project_domain_name').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('auth/admin_user_domain_name').with(:value => '<SERVICE DEFAULT>') is_expected.to contain_tempest_config('auth/admin_user_domain_name').with(:value => '<SERVICE DEFAULT>')