openstack/puppet-tempest
Code Issues Proposed changes

Add support for [auth] admin_system

Browse Source 
... which is required to run tests with SRBAC enforced.

Change-Id: Ibf247abff387d8844f6ed1ce39d85edae0fe770f
This commit is contained in:
Takashi Kajinami
2021-10-20 01:33:18 +09:00
parent c18ef504ce
commit 489f2198a3
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

@@ -88,6 +88,8 @@
# Defaults to $::os_service_default
# [*admin_project_domain_name*]
# Defaults to $::os_service_default
# [*admin_system*]
# Defaults to $::os_service_default
# [*default_credentials_domain_name*]
# Defaults to $::os_service_default
# [*image_ref*]
@@ -288,6 +290,7 @@ class tempest(
$admin_domain_name = $::os_service_default,
$admin_user_domain_name = $::os_service_default,
$admin_project_domain_name = $::os_service_default,
$admin_system = $::os_service_default,
$default_credentials_domain_name = $::os_service_default,
# roles fo the users created by tempest
$tempest_roles = $::os_service_default,
@@ -502,6 +505,7 @@ class tempest(
'auth/admin_password': value => $admin_password, secret => true;
'auth/admin_project_name': value => $admin_project_name;
'auth/admin_username': value => $admin_username;
'auth/admin_system': value => $admin_system;
'auth/tempest_roles': value => $tempest_roles_real;
'auth/use_dynamic_credentials': value => $use_dynamic_credentials;
'compute/change_password_available': value => $change_password_available;

4
releasenotes/notes/admin_system-6845c3b434b96b0a.yaml Normal file
View File

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

1
spec/classes/tempest_init_spec.rb
View File

@@ -202,6 +202,7 @@ describe 'tempest' do
is_expected.to contain_tempest_config('auth/admin_password').with_secret( true )
is_expected.to contain_tempest_config('auth/admin_project_name').with(:value => nil)
is_expected.to contain_tempest_config('auth/admin_username').with(:value => nil)
is_expected.to contain_tempest_config('auth/admin_system').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('auth/tempest_roles').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('auth/use_dynamic_credentials').with(:value => nil)
is_expected.to contain_tempest_config('dns/nameservers').with(:value => nil)