Glance AMQP should use multiple rabbit hosts

[Vladimir Kuklin]

Currently we use single AMQP host for glance. We should change it to multiple hosts as we did for nova and other services using oslo.messaging library.

[Nadya Privalova (nprivalova)]

During testing we have found out that glance-api.conf doesn't contain all rabbit info: user/password, hosts, info for working with notifications. So Glance doesn't send notifications now. It affects Ceilometer output for Glance.

[Openstack Gerrit (openstack-gerrit)]

Fix proposed to branch: master
Review: https://review.openstack.org/88329

[Bogdan Dobrelya (bogdando)]

The suggested patch assumes glance to use single AMQP host and passes missed connection info as well. The issues is what I cannot find any multiple rabbit_hosts configuration options in glance-api.conf example and source code in upstream). Please elaborate, how it could be implemented.

Dmitriy Borodaenko referenced this patch https://review.openstack.org/57678 as multiple AMQP support, but I'm not sure how should I use a single rabbit_host config option for multiple ones (there is none changes in the patch with rabbit_host usage at all). Or should I just put multiple hosts "host:port, host:port" formatted strings into the rabbit_host?

[Dmitry Borodaenko (angdraug)]

Bogdan, you can refer here for the rabbit options supported by oslo.messaging:
https://github.com/openstack/oslo.messaging/blob/master/oslo/messaging/_drivers/impl_rabbit.py#L39

[Dmitry Borodaenko (angdraug)]

For the reference, this issue was originally raised here:
http://lists.openstack.org/pipermail/openstack-dev/2014-April/032425.html

[Bogdan Dobrelya (bogdando)]

I cannot find rabbit_hosts option for multiple AMQP rabbit hosts in glance sources & conf example. Looks like it was not synced.

[Bogdan Dobrelya (bogdando)]

Ok, looks like rabbit_hosts option is configurable despite the fact it was missed in config example. So I will use it then. Thanks.

[Openstack Gerrit (openstack-gerrit)]

Reviewed: https://review.openstack.org/88329
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=7a3f93c535a2317d6df73f00ae296380aa66aa63
Submitter: Jenkins
Branch: master

commit 7a3f93c535a2317d6df73f00ae296380aa66aa63
Author: Bogdan Dobrelya <email address hidden>
Date: Thu Apr 17 17:54:00 2014 +0300

    Use rabbitmq notifications for glance

    * Add support for rabbit_hosts and rabbit_ha_queues for glance.
    * Qpid support is TODO (sync it from upstream as well, then we start to support it again)
    * glance::notify::rabbitmq is synced from master trunc puppet-glance in upstream
      * synced its rspecs as well
      * applied a patch in order to fix some existing issues in upstream

    Closes bug: #1308104

    Change-Id: Ie7e29aa8c9427ec890f7247b6e93116e7dd83b0c
    Signed-off-by: Bogdan Dobrelya <email address hidden>