Stop replacing the default apache config file

The apache class by default purge all default vhost config files, and
the other modules expect the default files are purged by that behavior.

This change removes the logic to replace the default vhost config file,
which is implemented only in a few modules, to make implementation
of all modules more consistent.

Change-Id: I0160bd0b7b8d5495031838119d401e44b3ddf7fa
This commit is contained in:
Takashi Kajinami 2022-01-04 21:16:25 +09:00
parent d6bbb2c583
commit f6c1a66e9d
3 changed files with 7 additions and 15 deletions

View File

@ -19,7 +19,6 @@ class barbican::params {
$keystone_listener_service_name = 'openstack-barbican-keystone-listener'
$barbican_wsgi_script_path = '/var/www/cgi-bin/barbican'
$barbican_wsgi_script_source = '/usr/bin/barbican-wsgi-api'
$httpd_config_file = '/etc/httpd/conf.d/barbican-api.conf'
}
'Debian': {
$api_service_name = 'barbican-api'
@ -30,7 +29,6 @@ class barbican::params {
$keystone_listener_service_name = 'barbican-keystone-listener'
$barbican_wsgi_script_path = '/usr/lib/cgi-bin/barbican'
$barbican_wsgi_script_source = '/usr/bin/barbican-wsgi-api'
$httpd_config_file = '/etc/apache2/conf-available/barbican-api.conf'
}
default: {
fail("Unsupported osfamily: ${::osfamily} operating system")

View File

@ -138,19 +138,6 @@ class barbican::wsgi::apache (
include barbican::deps
include barbican::params
file { $::barbican::params::httpd_config_file:
ensure => present,
content => "#
# This file has been cleaned by Puppet.
#
# OpenStack Barbican configuration has been moved to:
# - ${priority}-barbican_wsgi_main.conf
#",
}
Anchor['barbican::install::end'] -> File[$::barbican::params::httpd_config_file]
File[$::barbican::params::httpd_config_file] ~> Service['httpd']
openstacklib::wsgi::apache { 'barbican_wsgi_main':
bind_host => $bind_host,
bind_port => $public_port,
@ -180,5 +167,6 @@ class barbican::wsgi::apache (
access_log_format => $access_log_format,
error_log_file => $error_log_file,
custom_wsgi_process_options => $custom_wsgi_process_options,
require => Anchor['barbican::install::end']
}
}

View File

@ -0,0 +1,6 @@
---
upgrade:
- |
The default vhost config file is no longer managed. Make sure
the parameters of the ``apache`` class are properly defined and
the default vhost file is deleted.