ceilometer.openstack.common.log_handler is broken

Bug #1327076 reported by Ildiko Vancsa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
Nejc Saje

Bug Description

This log_handler uses ceilometer.openstack.common.notifier, which no longer exist in Ceilometer code base, it was removed in this patch: https://review.openstack.org/#/c/57457/

log_handler should support oslo.messaging and old rpc too at least until the 'K' cycle. IRC discussion about this topic can be found here: http://eavesdrop.openstack.org/irclogs/%23openstack-ceilometer/%23openstack-ceilometer.2014-06-04.log

Changed in ceilometer:
importance: Undecided → Medium
gordon chung (chungg)
Changed in ceilometer:
status: New → Triaged
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → juno-2
Eoghan Glynn (eglynn)
Changed in ceilometer:
assignee: nobody → Eoghan Glynn (eglynn)
Eoghan Glynn (eglynn)
Changed in ceilometer:
assignee: Eoghan Glynn (eglynn) → nobody
assignee: nobody → Nejc Saje (nejc-saje)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to ceilometer (master)

Fix proposed to branch: master
Review: https://review.openstack.org/107626

Changed in ceilometer:
status: Triaged → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to ceilometer (master)

Reviewed: https://review.openstack.org/107626
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=d294a0ed1aeed6ddf10b26052ab1f72e764f1c4e
Submitter: Jenkins
Branch: master

commit d294a0ed1aeed6ddf10b26052ab1f72e764f1c4e
Author: Nejc Saje <email address hidden>
Date: Thu Jul 17 11:32:21 2014 +0200

    Sync oslo log module and its dependencies

    Oslo log module has been updated to use oslo.messaging

    Closes-bug: #1327076
    Change-Id: I031d8f7bf92b6280bda2d2880bfdb4657ef5d3e0

Changed in ceilometer:
status: In Progress → Fix Committed
Changed in ceilometer:
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in ceilometer:
milestone: juno-2 → 2014.2
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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