openstack
/
puppet-rally
Code Issues Proposed changes

Migrate parameters to the [openstack] section (2) 

The [user_context] parameters and the [cleanup] parameters were moved
to the [openstack] section by [1]. This change updates the parameter
sections accordingly.

[1] 4f955b08ec34b5144594bc52b086b24941d51323

Change-Id: I7306bf71b02a2d6e783334b7854515725cc05f0e
This commit is contained in:
Takashi Kajinami 2022-04-14 22:37:08 +09:00
parent 859482b893
commit 9ddb9b608d
2 changed files with 19 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
manifests/settings.pp
View File

@ -51,9 +51,16 @@ class rally::settings (
include rally::settings::watcher
rally_config {
'cleanup/resource_deletion_timeout': value => $resource_deletion_timeout;
'users_context/project_domain': value => $project_domain;
'users_context/resource_management_workers': value => $resource_management_workers;
'users_context/user_domain': value => $user_domain;
'openstack/resource_deletion_timeout': value => $resource_deletion_timeout;
'openstack/project_domain': value => $project_domain;
'openstack/users_context_resource_management_workers': value => $resource_management_workers;
'openstack/user_domain': value => $user_domain;
}
rally_config {
'cleanup/resource_deletion_timeout': ensure => absent;
'users_context/project_domain': ensure => absent;
'users_context/resource_management_workers': ensure => absent;
'users_context/user_domain': ensure => absent;
}
}

16
spec/classes/rally_settings_spec.rb
View File

@ -19,20 +19,20 @@ describe 'rally::settings' do
shared_examples_for 'with default parameters' do
it 'configures rally settings with default parameters' do
is_expected.to contain_rally_config('cleanup/resource_deletion_timeout').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('users_context/project_domain').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('users_context/resource_management_workers').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('users_context/user_domain').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('openstack/resource_deletion_timeout').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('openstack/project_domain').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('openstack/users_context_resource_management_workers').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_rally_config('openstack/user_domain').with(:value => '<SERVICE DEFAULT>')
end
end
shared_examples_for 'with all parameters' do
before { params.merge!( rally_settings_params ) }
it 'configures rally settings with all parameters' do
is_expected.to contain_rally_config('cleanup/resource_deletion_timeout').with(:value => 600)
is_expected.to contain_rally_config('users_context/project_domain').with(:value => 'default')
is_expected.to contain_rally_config('users_context/resource_management_workers').with(:value => 30)
is_expected.to contain_rally_config('users_context/user_domain').with(:value => 'default')
is_expected.to contain_rally_config('openstack/resource_deletion_timeout').with(:value => 600)
is_expected.to contain_rally_config('openstack/project_domain').with(:value => 'default')
is_expected.to contain_rally_config('openstack/users_context_resource_management_workers').with(:value => 30)
is_expected.to contain_rally_config('openstack/user_domain').with(:value => 'default')
end
end