openstack
/
ceilometer
Code Issues Proposed changes

Fixing event types pattern for Role Noti. handler 

Identity(Keystone) notifications have a handler
for Role that has event type as identity.role.*
Another for RoleAssignment which accepts
identity.role_assignment.* events.

Since event types are regular expressions,
an event like identity.role_assignment.created
is matched with identity.role.* and gets incorrectly
routed to the Role Handler.

This fix restricts the event type regex for Role
handler to just identity.role.* events and excludes
identity.role_* events.

Change-Id: I526467ef2f8204e8093f7470b86949e3e0e69990
Closes-Bug: #1457785
This commit is contained in:
Rohit Jaiswal 2015-05-22 21:04:50 +00:00
parent 82031de3e0
commit 02bab799ba
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ceilometer/identity/notifications.py
View File

@ -99,7 +99,7 @@ class Role(IdentityCRUD):
@property
def event_types(self):
return ['%s.*' % self.resource_name]
return ['%s\..*' % self.resource_name]
class Trust(IdentityCRUD):