Merge "Gnocchi: Drop redundant integration_enable"

This commit is contained in:
Zuul 2023-08-25 06:36:33 +00:00 committed by Gerrit Code Review
commit 900cd09480
3 changed files with 7 additions and 21 deletions

View File

@ -84,8 +84,7 @@ class { 'openstack_integration::provision':
image_format => 'raw',
}
class { 'openstack_integration::gnocchi':
backend => 'ceph',
integration_enable => true,
backend => 'ceph',
}
class { 'openstack_integration::tempest':

View File

@ -88,8 +88,7 @@ class { 'openstack_integration::ceilometer':
}
include openstack_integration::aodh
class { 'openstack_integration::gnocchi':
integration_enable => true,
backend => 'swift',
backend => 'swift',
}
class { 'openstack_integration::tempest':

View File

@ -1,17 +1,11 @@
# Configure the Gnocchi service
#
# [*backend*]
# (optional) Backend storage to be used. This is overridden by 'file' when
# integration_enable is false.
# Defaults to 'ceph'.
#
# [*integration_enable*]
# (optional) Boolean to run integration tests.
# Defaults to true.
# (optional) Backend storage to be used.
# Defaults to 'file'.
#
class openstack_integration::gnocchi (
$backend = 'ceph',
$integration_enable = true,
$backend = 'file',
){
include openstack_integration::config
@ -25,12 +19,6 @@ class openstack_integration::gnocchi (
Exec['update-ca-certificates'] ~> Service['httpd']
}
if ! $integration_enable {
$backend_real = 'file'
} else {
$backend_real = $backend
}
class { 'gnocchi::logging':
debug => true,
}
@ -60,7 +48,7 @@ class openstack_integration::gnocchi (
# cycle, caused by multiple usage of httpd. In the mean time
# skip initializing storage for swift, because the current
# implementation only validates connection to swift.
$db_sync_extra_opts = $backend_real ? {
$db_sync_extra_opts = $backend ? {
'swift' => '--skip-storage',
default => undef,
}
@ -111,7 +99,7 @@ class openstack_integration::gnocchi (
}
class { 'gnocchi::storage': }
case $backend_real {
case $backend {
'ceph': {
class { 'gnocchi::storage::ceph':
ceph_username => 'openstack',