delete event payload
This fix will allow the delete event to have the required data in the notification payload. Change-Id: I57a001ca2fddc2a750026e7da7980bfd8e5aab40 Closes-Bug: 1593127
This commit is contained in:
parent
9cbafbb016
commit
e92b68dd82
|
@ -550,10 +550,12 @@ class Controller(object):
|
|||
# usage trackers as dirty
|
||||
resource_registry.set_resources_dirty(request.context)
|
||||
notifier_method = self._resource + '.delete.end'
|
||||
result = {self._resource: self._view(request.context, obj)}
|
||||
notifier_payload = {self._resource + '_id': id}
|
||||
notifier_payload.update(result)
|
||||
self._notifier.info(request.context,
|
||||
notifier_method,
|
||||
{self._resource + '_id': id})
|
||||
result = {self._resource: self._view(request.context, obj)}
|
||||
notifier_payload)
|
||||
registry.notify(self._resource, events.BEFORE_RESPONSE, self,
|
||||
context=request.context, data=result,
|
||||
method_name=notifier_method, action=action,
|
||||
|
|
|
@ -1329,6 +1329,11 @@ class NotificationTest(APIv2TestBase):
|
|||
for msg, event in zip(fake_notifier.NOTIFICATIONS, expected_events):
|
||||
self.assertEqual('INFO', msg['priority'])
|
||||
self.assertEqual(event, msg['event_type'])
|
||||
if opname == 'delete' and event == 'network.delete.end':
|
||||
self.assertIn('payload', msg)
|
||||
resource = msg['payload']
|
||||
self.assertIn('network_id', resource)
|
||||
self.assertIn('network', resource)
|
||||
|
||||
self.assertEqual(expected_code, res.status_int)
|
||||
|
||||
|
|
Loading…
Reference in New Issue