openstack/puppet-placement
Code Issues Proposed changes

Merge "Centralize user/group definition"

Browse Source
This commit is contained in:
Zuul 2024-10-21 15:42:00 +00:00 committed by Gerrit Code Review
commit 2014fe5bcb
3 changed files with 5 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/db/sync.pp
View File

@ -12,11 +12,12 @@ class placement::db::sync(
) {
include placement::deps
include placement::params
exec { 'placement-manage-db-sync':
command => 'placement-manage db sync',
path => ['/bin', '/usr/bin', '/usr/local/bin'],
user => 'placement',
user => $::placement::params::user,
refreshonly => true,
try_sleep => 5,
tries => 10,

1
manifests/params.pp
View File

@ -5,6 +5,7 @@ class placement::params {
include placement::deps
include openstacklib::defaults
$user = 'placement'
$group = 'placement'
case $facts['os']['family'] {

4
manifests/wsgi/apache.pp
View File

@ -167,7 +167,7 @@ class placement::wsgi::apache (
::openstacklib::wsgi::apache { 'placement_wsgi':
bind_host => $bind_host,
bind_port => $port,
group => 'placement',
group => $::placement::params::group,
path => $path,
priority => $priority,
servername => $servername,
@ -180,7 +180,7 @@ class placement::wsgi::apache (
ssl_crl_path => $ssl_crl_path,
ssl_key => $ssl_key,
threads => $threads,
user => 'placement',
user => $::placement::params::user,
vhost_custom_fragment => $vhost_custom_fragment,
workers => $workers,
wsgi_daemon_process => 'placement-api',