openstack
/
puppet-placement
Code Issues Proposed changes

Fix dependency to purge default vhost config 

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: I43599cd9ad217b78da3d7c86ed106f6fc57c58f2
This commit is contained in:
Takashi Kajinami 2022-01-11 18:13:42 +09:00
parent 8e82e50107
commit 441093739f
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
manifests/wsgi/apache.pp
View File

@ -127,12 +127,15 @@ class placement::wsgi::apache (
}
$ssl_real = pick($ssl, true)
include placement::deps
include placement::params
if $ensure_package != undef {
warning('The placement::wsgi::apache::ensure_package parameter is deprecated and has no effect')
}
Anchor['placement::install::end'] -> Class['apache']
::openstacklib::wsgi::apache { 'placement_wsgi':
bind_host => $bind_host,
bind_port => $api_port,
@ -161,7 +164,6 @@ class placement::wsgi::apache (
access_log_file => $access_log_file,
access_log_format => $access_log_format,
error_log_file => $error_log_file,
require => Anchor['placement::install::end'],
}
}