Ceilometer

ceilometer.openstack.common.middleware.notifier and audit should be cleaned up

Bug #1327084 reported by Ildiko Vancsa
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Ceilometer
Fix Released
Medium
ZhiQiang Fan
Ceilometer 2014.2 "juno"

Bug Description

The ceilometer.openstack.common.middleware.notifier code depends on the ceilometer.openstack.common.notifier.api, which was removed in this patch: https://review.openstack.org/#/c/57457/ This code part probably can be removed and after this the ceilometer.openstack.common.middleware.audit file should be cleaned up too.

Relevant IRC log for this topic can be found here: http://eavesdrop.openstack.org/irclogs/%23openstack-ceilometer/%23openstack-ceilometer.2014-06-04.log

Ildiko Vancsa (ildiko-vancsa)
Changed in ceilometer:
importance: Undecided → Medium
Revision history for this message
ZhiQiang Fan (aji-zqfan) wrote :

hi, Ildiko Vancsa , are you in progress for this issue? I want to have a try if you permit

Revision history for this message
gordon chung (chungg) wrote :

we can remove these middleware... they do not work with oslo.messaging... the oslo.messaging compatible version is available in pycadf

Changed in ceilometer:
status: New → Triaged
Revision history for this message
Ildiko Vancsa (ildiko-vancsa) wrote :

Hi,

@ZhiQiang: I do not have the time now to deal with this, so it's yours. Thanks for handling it.

@Gordon: Cool, thanks for confirming and providing additional info!

Best Regards,
Ildiko

ZhiQiang Fan (aji-zqfan)
Changed in ceilometer:
assignee: nobody → ZhiQiang Fan (aji-zqfan)
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/99246

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/99246
Committed: https://git.openstack.org/cgit/openstack/ceilometer/commit/?id=92f7ed1ac1691493d17920b942e7eafd55e94e3c
Submitter: Jenkins
Branch: master

commit 92f7ed1ac1691493d17920b942e7eafd55e94e3c
Author: Ildiko Vancsa <email address hidden>
Date: Mon Jun 9 18:13:04 2014 +0200

    Clean up oslo.middleware.{audit,notifier}

    After migrating to oslo.messaging, the oslo.middleware.notifier and
    oslo.middleware.audit can no longer work, we should remove these code.

    Change-Id: Ie848d9f356eec670bbeeafc83f9e704212b1d2e4
    Closes-Bug: #1327084

Changed in ceilometer:
status: In Progress → Fix Committed
Eoghan Glynn (eglynn)
Changed in ceilometer:
milestone: none → juno-2
Russell Bryant (russellb)
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.
Subscribing...

Other bug subscribers

Remote bug watches

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