openstack
/
puppet-neutron
Code Issues Proposed changes

Run db-manage command by the service user 

... instead of root, because the command doesn't require root. This
follows the guidance in the current installation guide[1][2].

[1] https://docs.openstack.org/neutron/latest/install/controller-install-rdo.html
[2] https://docs.openstack.org/neutron/latest/install/controller-install-ubuntu.html

Change-Id: Iad40023e370a978a58f2894bba2dd08cbab5d71a
This commit is contained in:
Takashi Kajinami 2022-02-21 01:18:14 +09:00
parent 46b552b1dc
commit da49c1914f
2 changed files with 4 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
manifests/db/sync.pp
View File

@ -26,6 +26,7 @@ class neutron::db::sync(
command => "neutron-db-manage ${extra_params} upgrade heads",
path => '/usr/bin',
refreshonly => true,
user => $::neutron::params::user,
try_sleep => 5,
tries => 10,
timeout => $db_sync_timeout,

3
spec/classes/neutron_db_sync_spec.rb
View File

@ -10,6 +10,7 @@ describe 'neutron::db::sync' do
:command => 'neutron-db-manage upgrade heads',
:path => '/usr/bin',
:refreshonly => 'true',
:user => 'neutron',
:try_sleep => 5,
:tries => 10,
:timeout => 300,
@ -34,6 +35,7 @@ describe 'neutron::db::sync' do
:command => 'neutron-db-manage --config-file /etc/neutron/neutron.conf upgrade heads',
:path => '/usr/bin',
:refreshonly => 'true',
:user => 'neutron',
:try_sleep => 5,
:tries => 10,
:timeout => 300,
@ -59,6 +61,7 @@ describe 'neutron::db::sync' do
:command => 'neutron-db-manage upgrade heads',
:path => '/usr/bin',
:refreshonly => 'true',
:user => 'neutron',
:try_sleep => 5,
:tries => 10,
:timeout => 750,