Should switch to get_notification_listener function for creating oslo.messaging serer

Bug #1570126 reported by Kirill Zaitsev
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Murano
Confirmed
High
Unassigned

Bug Description

https://review.openstack.org/#/c/297988/ — this commit changes public interfaces of several classes in a backwards-incompatible way.
https://review.openstack.org/#/c/297988/9/oslo_messaging/notify/dispatcher.py here NotificationDispatcher used to accept 5 obligatory params and now it only accpets 3.
This would cause major problems for murano, since we use the library here https://github.com/openstack/murano/blob/master/murano/common/server.py#L231 and the change is backward-incompatible.
This would probably mean that the next release of oslo.messaging would be a major one, but nevertheless it would break murano code.

We should switch to using https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/notify/listener.py#L186 get_notification_listener func, since oslo folks believe it would be left unchanged across major releases.

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.