puppet-ceilometer

Type error with alarm_package_name on Debian osfamily

  1. Series havana
  2. Bug #1263946
Bug #1263946 reported by Sebastien Badia
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
puppet-ceilometer
Fix Released
Undecided
Sebastien Badia
puppet-ceilometer 4.0.0 "4.0.0"
Havana
Fix Released
Undecided
Unassigned
puppet-ceilometer 3.1.0 "3.1.0"

Bug Description

Puppet-ceilometer alarm class is broken on debian based osfamily.

# ------- >8 ------- #
Error: Failed to apply catalog: Parameter name failed on Package[ceilometer-alarm]: Name must be a String not Array at /etc/puppet/modules/ceilometer/manifests/alarm/evaluator.pp:35
Wrapped exception:
Name must be a String not Array
# ------- >8 ------- #

* On RedHat osfamily the package (for both evaluator and notifier) is
openstack-ceilometer-alarm (see http://repos.fedorapeople.org/repos/openstack/openstack-havana/epel-6/)

* On Debian, we have the two packages (evaluator and notifier):
ceilometer-alarm-evaluator (see http://packages.debian.org/sid/ceilometer-alarm-evaluator)
ceilometer-alarm-notifier (see http://packages.debian.org/sid/ceilometer-alarm-notifier)

* And on Ubuntu… a little bug ;-)
https://bugs.launchpad.net/ubuntu/+source/ceilometer/+bug/1250002

Sebastien Badia (sbadia)
Changed in puppet-ceilometer:
assignee: nobody → Sebastien Badia (sbadia)
status: New → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceilometer (master)

Reviewed: https://review.openstack.org/63935
Committed: https://git.openstack.org/cgit/stackforge/puppet-ceilometer/commit/?id=5bc33119dfc2adb8abdd0c475422d0e8559513c1
Submitter: Jenkins
Branch: master

commit 5bc33119dfc2adb8abdd0c475422d0e8559513c1
Author: Sebastien Badia <email address hidden>
Date: Sat Jan 4 11:32:35 2014 +0100

    Fix package ceilometer-alarm type error on Debian

    This commit:
      - Fix type error about ceilometer-alarm package name
        - See bug #1263946
      - Fix indentation on Debian params.pp
      - Fix spec warnings (partition_rpc_topic,record_history)

    Closes-bug: #1263946

    Change-Id: I1ad19af429f2787d58837e51da95e981bdd66b40

Changed in puppet-ceilometer:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to puppet-ceilometer (stable/havana)

Fix proposed to branch: stable/havana
Review: https://review.openstack.org/68588

Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to puppet-ceilometer (stable/havana)

Reviewed: https://review.openstack.org/68588
Committed: https://git.openstack.org/cgit/stackforge/puppet-ceilometer/commit/?id=3acae1fd2a3ac641a09c8ce0b62bb152696923ae
Submitter: Jenkins
Branch: stable/havana

commit 3acae1fd2a3ac641a09c8ce0b62bb152696923ae
Author: Sebastien Badia <email address hidden>
Date: Sat Jan 4 11:32:35 2014 +0100

    Fix package ceilometer-alarm type error on Debian

    This commit:
      - Fix type error about ceilometer-alarm package name
        - See bug #1263946
      - Fix indentation on Debian params.pp
      - Fix spec warnings (partition_rpc_topic,record_history)

    Closes-bug: #1263946

    Change-Id: I1ad19af429f2787d58837e51da95e981bdd66b40
    (cherry-pick from 5bc33119dfc2adb8abdd0c475422d0e8559513c1)

tags: added: in-stable-havana
Mathieu Gagné (mgagne)
Changed in puppet-ceilometer:
milestone: none → 4.0.0
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.