]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Fix the multi-backend storge issue for ZMQ.
authorVincent Hou <sbhou@cn.ibm.com>
Wed, 19 Jun 2013 10:48:15 +0000 (18:48 +0800)
committerVincent Hou <sbhou@cn.ibm.com>
Fri, 26 Jul 2013 04:05:41 +0000 (00:05 -0400)
commita4f6ab0f3d9710713cd58ed9dddf12a8d8b2160b
treee0ea37f5bdd6c7ed7beb50f6031255096d8d2862
parent213ad6b8cb7f28be78098eade08ce6f10f50ad4d
Fix the multi-backend storge issue for ZMQ.

This issue is not caused by the naming convention as described in Bug 1166899,
but due to the incorrect node_topic registration in ZMQ.

What have been done in this patch:
*Import the latest impl_zmq.py from oslo.
*Change the delimiter from "." to ":" between topic and host. "." will make
node_topic an invalid key for the registration in ZMQ.
*The node_topic should be registered correctly via
topic = '.'.join((topic, CONF.rpc_zmq_host)) in impl_zmq.py.
*Move init_host() in services.py downstairs to make sure the c-vol can be
launched successfully for ZMQ.

Fixed Bug 1166899.

Change-Id: Id982ab9482f08d69bdc68d389fb41a7752efa168
cinder/openstack/common/rpc/__init__.py
cinder/openstack/common/rpc/impl_zmq.py
cinder/service.py
cinder/tests/test_volume_rpcapi.py