Skip to content

Commit

Permalink
Fix alarm package params for ubuntu havana
Browse files Browse the repository at this point in the history
The alarm-notifier and alarm-evaluator are
still part of ceilometer-common for havana ubuntu
packaging. The new packaging split is only available
for icehouse

Change-Id: I91098bd3f13ee4391dcf1b870705c0a0c5566acc
  • Loading branch information
Pradeep Kilambi committed Feb 17, 2014
1 parent 91045b3 commit 619a827
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 5 deletions.
2 changes: 1 addition & 1 deletion manifests/params.pp
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@
$collector_package_name = 'ceilometer-collector'
$common_package_name = 'ceilometer-common'
$client_package_name = 'python-ceilometerclient'
$alarm_package_name = ['ceilometer-alarm-notifier','ceilometer-alarm-evaluator']
$alarm_package_name = ['ceilometer-common']
# service names
$agent_central_service_name = 'ceilometer-agent-central'
$agent_compute_service_name = 'ceilometer-agent-compute'
Expand Down
3 changes: 1 addition & 2 deletions spec/classes/ceilometer_alarm_evaluator_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,6 @@
it { should contain_class('ceilometer::params') }

it 'installs ceilometer-alarm package' do
should contain_package(platform_params[:alarm_evaluator_package_name]).with_before('Service[ceilometer-alarm-evaluator]')
should contain_package(platform_params[:alarm_evaluator_package_name]).with(
:ensure => 'present',
:name => platform_params[:alarm_evaluator_package_name]
Expand Down Expand Up @@ -79,7 +78,7 @@
end

let :platform_params do
{ :alarm_evaluator_package_name => 'ceilometer-alarm-evaluator',
{ :alarm_evaluator_package_name => 'ceilometer-common',
:alarm_evaluator_service_name => 'ceilometer-alarm-evaluator' }
end

Expand Down
3 changes: 1 addition & 2 deletions spec/classes/ceilometer_alarm_notifier_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,6 @@
it { should contain_class('ceilometer::params') }

it 'installs ceilometer-alarm package' do
should contain_package(platform_params[:alarm_notifier_package_name]).with_before('Service[ceilometer-alarm-notifier]')
should contain_package(platform_params[:alarm_notifier_package_name]).with(
:ensure => 'present',
:name => platform_params[:alarm_notifier_package_name]
Expand Down Expand Up @@ -72,7 +71,7 @@
end

let :platform_params do
{ :alarm_notifier_package_name => 'ceilometer-alarm-notifier',
{ :alarm_notifier_package_name => 'ceilometer-common',
:alarm_notifier_service_name => 'ceilometer-alarm-notifier' }
end

Expand Down

0 comments on commit 619a827

Please sign in to comment.