openstack/puppet-tempest
Code Issues Proposed changes

trove: add db_flavor_ref parameter

Browse Source 
db_flavor_ref is required to configure the Nova flavor that will be used
to run Trove tempest tests.

Change-Id: Id2d6b9ae571515d90f6232a13a9beecd357703cd
Related-bug: #1612398
This commit is contained in:
Emilien Macchi 2016-08-17 10:05:32 -04:00
parent 3d78d8f294
commit 2a3e609dfd
2 changed files with 7 additions and 0 deletions
manifests
init.pp
spec/classes
tempest_spec.rb

6
manifests/init.pp

@ -169,6 +169,9 @@
# [*tempest_roles*]
# Should be an array.
# Defaults to $::os_service_default
# [*db_flavor_ref*]
# Valid primary flavor to use in Trove tests.
# Defaults to $::os_service_default
#
# DEPREACTED PARAMETERS
# [*verbose*]
@ -261,6 +264,8 @@ class tempest(
$public_router_id = '',
# Sahara config
$sahara_plugins = undef,
# Trove config
$db_flavor_ref = $::os_service_default,
# Service configuration
$cinder_available = true,
$glance_available = true,
@ -445,6 +450,7 @@ class tempest(
'network/public_router_id': value => $public_router_id;
'dashboard/login_url': value => $login_url;
'dashboard/dashboard_url': value => $dashboard_url;
'database/db_flavor_ref': value => $db_flavor_ref;
'service_available/cinder': value => $cinder_available;
'service_available/glance': value => $glance_available;
'service_available/heat': value => $heat_available;

1
spec/classes/tempest_spec.rb

@ -218,6 +218,7 @@ describe 'tempest' do
is_expected.to contain_tempest_config('network/public_router_id').with(:value => '')
is_expected.to contain_tempest_config('dashboard/login_url').with(:value => nil)
is_expected.to contain_tempest_config('dashboard/dashboard_url').with(:value => nil)
is_expected.to contain_tempest_config('database/db_flavor_ref').with_value('<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('service_available/cinder').with(:value => true)
is_expected.to contain_tempest_config('service_available/glance').with(:value => true)
is_expected.to contain_tempest_config('service_available/heat').with(:value => false)