routing notifier won't work

Bug #1316681 reported by Mehdi Abaakouk
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
oslo.messaging
Fix Released
Medium
Mehdi Abaakouk

Bug Description

Hi,

The routing notifier won't work because the driver implements

    def notify(self, ctxt, msg):

instead of:

    def notify(self, ctxt, msg, priority):

and it also changes the case of the of the priority.

Cheers,

Mehdi Abaakouk (sileht)
Changed in oslo.messaging:
assignee: nobody → Mehdi Abaakouk (sileht)
Revision history for this message
Openstack Gerrit (openstack-gerrit) wrote : Fix proposed to oslo.messaging (master)

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

Changed in oslo.messaging:
status: New → In Progress
Mark McLoughlin (markmc)
Changed in oslo.messaging:
importance: Undecided → Medium
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to oslo.messaging (master)

Reviewed: https://review.openstack.org/92403
Committed: https://git.openstack.org/cgit/openstack/oslo.messaging/commit/?id=cbbb90e28c608d165df94b779c04593bd17d02f1
Submitter: Jenkins
Branch: master

commit cbbb90e28c608d165df94b779c04593bd17d02f1
Author: Mehdi Abaakouk <email address hidden>
Date: Tue May 6 18:24:49 2014 +0200

    Fix the notify method of the routing notifier

    The notify method of the routing notifier don't have the correct
    signature and change the case of the priority.

    This change fixes it.

    Change-Id: I833b19d0c054c94b861a4f6cc263c3487fa0e544
    Closes-bug: #1316681

Changed in oslo.messaging:
status: In Progress → Fix Committed
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to oslo.messaging (stable/icehouse)

Fix proposed to branch: stable/icehouse
Review: https://review.openstack.org/107909

Changed in oslo.messaging:
milestone: none → juno-2
status: Fix Committed → Fix Released
tags: added: in-stable-icehouse
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.