openstack/puppet-tempest
Code Issues Proposed changes

Deprecate support for Panko

Browse Source 
... because Panko has been retired.

Change-Id: I0895495a4f01a5caf4e99d4c46dcab9656746b6a
This commit is contained in:
Takashi Kajinami 2021-06-02 16:47:17 +09:00
parent 59731cdc96
commit d273aa3c56
3 changed files with 13 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
manifests/init.pp
View File

@ -127,8 +127,6 @@
# Defaults to false
# [*gnocchi_available*]
# Defaults to false
# [*panko_available*]
# Defaults to false
# [*designate_available*]
# Defaults to false
# [*horizon_available*]
@ -226,6 +224,8 @@
# Defaults to undef
# [*img_dir*]
# Defaults to undef
# [*panko_available*]
# Defaults to undef
#
class tempest(
$package_ensure = 'present',
@ -322,7 +322,6 @@ class tempest(
$ceilometer_available = false,
$aodh_available = false,
$gnocchi_available = false,
$panko_available = false,
$designate_available = false,
$horizon_available = true,
$neutron_available = false,
@ -368,6 +367,7 @@ class tempest(
$neutron_fwaas_available = undef,
$congress_available = undef,
$img_dir = undef,
$panko_available = false,
) {
if !is_service_default($tempest_roles) and !empty($tempest_roles){
@ -428,6 +428,10 @@ class tempest(
}
}
if $panko_available != undef {
warning('The panko_available parameter has been deprecated and has no effect')
}
include tempest::params
include openstacklib::openstackclient
@ -576,7 +580,6 @@ class tempest(
'service_available/barbican': value => $barbican_available;
'service_available/bgpvpn': value => $neutron_bgpvpn_available;
'service_available/gnocchi': value => $gnocchi_available;
'service_available/panko': value => $panko_available;
'service_available/designate': value => $designate_available;
'service_available/horizon': value => $horizon_available;
'service_available/l2gw': value => $neutron_l2gw_available;
@ -615,7 +618,7 @@ class tempest(
}
if $manage_tests_packages {
if ($aodh_available or $ceilometer_available or $panko_available or $gnocchi_available) and $::tempest::params::python_telemetry_tests {
if ($aodh_available or $ceilometer_available or $gnocchi_available) and $::tempest::params::python_telemetry_tests {
package { 'python-telemetry-tests':
ensure => present,
name => $::tempest::params::python_telemetry_tests,

5
releasenotes/notes/retire-panko-c37290df663d870d.yaml Normal file
View File

@ -0,0 +1,5 @@
---
deprecations:
- |
The ``tempest::panko_available`` parameter has been deprecated and has
no effect.

1
spec/classes/tempest_init_spec.rb
View File

@ -249,7 +249,6 @@ describe 'tempest' do
is_expected.to contain_tempest_config('service_available/aodh_plugin').with(:value => false)
is_expected.to contain_tempest_config('service_available/bgpvpn').with(:value => false)
is_expected.to contain_tempest_config('service_available/gnocchi').with(:value => false)
is_expected.to contain_tempest_config('service_available/panko').with(:value => false)
is_expected.to contain_tempest_config('service_available/horizon').with(:value => true)
is_expected.to contain_tempest_config('service_available/l2gw').with(:value => false)
is_expected.to contain_tempest_config('service_available/neutron').with(:value => true)