openstack
/
puppet-ceilometer
Code Issues Proposed changes

Manage ceilometer user before configuration phase 

This is follow-up of 7d5b0acc0371e60115e8e235f5fe843687a7868a and
fixes the resource dependency removed.

This also looses unnecessary dependency about the nova-common package
and the ceilometer-common package.

Change-Id: I4935c56fe61fa92b6e7b3fddc952bda6f61b05a2
This commit is contained in:
Takashi Kajinami 2022-05-18 23:13:08 +09:00
parent a569c41855
commit 3fb220a670
2 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
manifests/agent/polling.pp
View File

@ -121,7 +121,8 @@ class ceilometer::agent::polling (
$compute_namespace_name = 'compute'
Package <| title == 'ceilometer-common' |> -> User['ceilometer']
Package <| title == 'nova-common' |> -> Package['ceilometer-common']
Package <| title == 'nova-common' |> -> User['ceilometer']
User['ceilometer'] -> Anchor['ceilometer::install::end']
ceilometer_config {
'compute/instance_discovery_method': value => $instance_discovery_method;

5
spec/classes/ceilometer_agent_polling_spec.rb
View File

@ -25,8 +25,11 @@ describe 'ceilometer::agent::polling' do
end
}
it { should contain_package('ceilometer-common').with(
:before => /User\[ceilometer\]/
)}
it { should contain_package('nova-common').with(
:before => /Package\[ceilometer-common\]/
:before => /User\[ceilometer\]/
)}
it {