openstack/puppet-mistral
Code Issues Proposed changes

Fix dependency to purge default vhost config

Browse Source 
It turned out defining dependency for openstacklib::wsgi::apache
doesn't properly enforce resource order and the default vhost file
is not purged properly.
This change adds the more explicit dependency to enforce the order
properly.

Change-Id: Id4cfa06790d47c30ea65d69b8143b004c9f0f12b
This commit is contained in:
Takashi Kajinami
2022-01-11 18:50:00 +09:00
parent 8f4e5c9d6d
commit 4b6198fae3
1 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/wsgi/apache.pp
View File

@@ -137,6 +137,8 @@ class mistral::wsgi::apache (
include mistral::deps include mistral::deps
include mistral::params include mistral::params
Anchor['mistral::install::end'] -> Class['apache']
::openstacklib::wsgi::apache { 'mistral_wsgi': ::openstacklib::wsgi::apache { 'mistral_wsgi':
bind_host => $bind_host, bind_host => $bind_host,
bind_port => $port, bind_port => $port,
@@ -166,6 +168,5 @@ class mistral::wsgi::apache (
error_log_file => $error_log_file, error_log_file => $error_log_file,
custom_wsgi_process_options => $custom_wsgi_process_options, custom_wsgi_process_options => $custom_wsgi_process_options,
vhost_custom_fragment => $vhost_custom_fragment, vhost_custom_fragment => $vhost_custom_fragment,
require => Anchor['mistral::install::end'],
} }
} }