Comment 9 for bug 1335891

Revision history for this message
Dmitry Borodaenko (angdraug) wrote :

Summary of current state of fixes for this bug:

https://review.openstack.org/104183 sets Glance rabbit_notification_exchange and Ceilometer glance_control_exchange to "openstack" and was merged into stable/5.0 on July 4. Glance control_exchange is not set in stable/5.0 which leaves it at oslo.messaging default of "openstack". The rabbit_notification_exchange in Glance (stable/icehouse) is only present in etc/glance-api.conf and isn't actually used for anything since Glance moved to oslo.messaging, so the first part of that change is meaningless but harmless.

https://review.openstack.org/107288 targeted for fuel-library master sets Glance control_exchange to "glance", leaves Glance rabbit_notification_exchange at its current state of explicitly setting it to "glance" (once again meaningless but harmless and matches recently merged upstream), and leaves Ceilometer glance_control_exchange not set, which leaves it at Ceilometer's default of "glance".

I like the master version of this fix much better so I'm going to merge it even though it deviates from what was done in stable/5.0. Please create a backport of https://review.openstack.org/107288 with the same Change-Id that brings stable/5.0 in sync with the current state of this fix for master.