openstack/puppet-tempest
Code Issues Proposed changes

Merge "Glance: Disable api v1 testing by default"

Browse Source
This commit is contained in:
Zuul 2022-01-18 17:48:55 +00:00 committed by Gerrit Code Review
commit 70b9dc8918
3 changed files with 8 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/init.pp
View File

@ -33,7 +33,7 @@
# [*image_name_alt*] # [*image_name_alt*]
# Defaults to undef # Defaults to undef
# [*glance_v1*] # [*glance_v1*]
# Defaults to true # Defaults to false
# [*glance_v2*] # [*glance_v2*]
# Defaults to true # Defaults to true
# [*configure_networks*] # [*configure_networks*]
@ -243,7 +243,7 @@ class tempest(
$configure_images = true, $configure_images = true,
$image_name = undef, $image_name = undef,
$image_name_alt = undef, $image_name_alt = undef,
$glance_v1 = true, $glance_v1 = false,
$glance_v2 = true, $glance_v2 = true,
# Neutron network config # Neutron network config

5
releasenotes/notes/disable-glance-api-v1-010c242db47cfabb.yaml Normal file
View File

@ -0,0 +1,5 @@
---
fixes:
- |
The ``[image-feature-enabled] api_v1`` parameter is now set to ``False`` by
default.

2
spec/classes/tempest_init_spec.rb
View File

@ -235,7 +235,7 @@ describe 'tempest' do
is_expected.to contain_tempest_config('identity/disable_ssl_certificate_validation').with(:value => nil) is_expected.to contain_tempest_config('identity/disable_ssl_certificate_validation').with(:value => nil)
is_expected.to contain_tempest_config('identity-feature-enabled/api_v2').with(:value => '<SERVICE DEFAULT>') is_expected.to contain_tempest_config('identity-feature-enabled/api_v2').with(:value => '<SERVICE DEFAULT>')
is_expected.to contain_tempest_config('identity-feature-enabled/api_v3').with(:value => true) is_expected.to contain_tempest_config('identity-feature-enabled/api_v3').with(:value => true)
is_expected.to contain_tempest_config('image-feature-enabled/api_v1').with(:value => true) is_expected.to contain_tempest_config('image-feature-enabled/api_v1').with(:value => false)
is_expected.to contain_tempest_config('image-feature-enabled/api_v2').with(:value => true) is_expected.to contain_tempest_config('image-feature-enabled/api_v2').with(:value => true)
is_expected.to contain_tempest_config('l2gw/l2gw_switch').with(:value => nil) is_expected.to contain_tempest_config('l2gw/l2gw_switch').with(:value => nil)
is_expected.to contain_tempest_config('network-feature-enabled/api_extensions').with(:value => nil) is_expected.to contain_tempest_config('network-feature-enabled/api_extensions').with(:value => nil)