The if_notifications_enabled decorator skips the execution of the
decorated function if the versioned notifications are not configured
to be emitted. The send_instance_update_notification() call was wrongly
decorated with this decorator as it not only sends versioned
notification but also send the legacy compute.instance.update
notification as well. This caused that the legacy instance.update
notification was not emitted when the notification_format config option
was set to unversioned.
As the _send_versioned_instance_update() call already has the decorator
the solution is simply to remove the decorator from the
send_instance_update_notification() call.
Reviewed: https:/ /review. openstack. org/510603 /git.openstack. org/cgit/ openstack/ nova/commit/ ?id=0ffe03a2650 a698de1b8328909 8ed535734b2360
Committed: https:/
Submitter: Jenkins
Branch: master
commit 0ffe03a2650a698 de1b83289098ed5 35734b2360
Author: Balazs Gibizer <email address hidden>
Date: Mon Oct 9 16:29:09 2017 +0200
Fix sending legacy instance.update notification
The if_notification s_enabled decorator skips the execution of the update_ notification( ) call was wrongly instance. update
decorated function if the versioned notifications are not configured
to be emitted. The send_instance_
decorated with this decorator as it not only sends versioned
notification but also send the legacy compute.
notification as well. This caused that the legacy instance.update
notification was not emitted when the notification_format config option
was set to unversioned.
As the _send_versioned _instance_ update( ) call already has the decorator instance_ update_ notification( ) call.
the solution is simply to remove the decorator from the
send_
Closes-Bug: #1721843 ff4e29f1ab3c953 99bbe9ff2e7
Change-Id: I9904adeb3de60c