service id is not right in nova-api

Bug #1225579 reported by Guangya Liu (Jay Lau)
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
OpenStack Compute (nova)
Fix Released
Low
Guangya Liu (Jay Lau)

Bug Description

Now in nova/compute/api.py, the service id is using aggregate, this is not accurate. As the service is actually api.

This will cause some confuse if we add new notifications for others such as host operations, keypair operations and so on.

get_notifier = functools.partial(notifier.get_notifier, service='aggregate') << Better change this to api
wrap_exception = functools.partial(exception.wrap_exception,
                                   get_notifier=get_notifier)

Changed in nova:
assignee: nobody → Jay Lau (jay-lau-513)
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to nova (master)

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

Changed in nova:
status: New → In Progress
Andrew Laski (alaski)
Changed in nova:
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to nova (master)

Reviewed: https://review.openstack.org/46640
Committed: http://github.com/openstack/nova/commit/2c89a8b2606e0c219dfa3cf81969b9751bcbbbb4
Submitter: Jenkins
Branch: master

commit 2c89a8b2606e0c219dfa3cf81969b9751bcbbbb4
Author: Jay Lau <email address hidden>
Date: Sun Sep 15 14:03:42 2013 +0800

    Change service id to compute for compute/api.py

    Now the default service id for nova api notification is aggregate,
    this is not accurate, we should change it to compute.

    Fix bug 1225579

    Change-Id: Ife558024c57d9fd035cd1a618063c656e03c52a3

Changed in nova:
status: In Progress → Fix Committed
Thierry Carrez (ttx)
Changed in nova:
milestone: none → havana-rc1
status: Fix Committed → Fix Released
Thierry Carrez (ttx)
Changed in nova:
milestone: havana-rc1 → 2013.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.