diff --git a/neutron/openstack/common/rpc/impl_qpid.py b/neutron/openstack/common/rpc/impl_qpid.py index 026d39280cc..67e0f9c6280 100644 --- a/neutron/openstack/common/rpc/impl_qpid.py +++ b/neutron/openstack/common/rpc/impl_qpid.py @@ -18,7 +18,6 @@ import functools import itertools import time -import uuid import eventlet import greenlet @@ -123,7 +122,6 @@ def __init__(self, conf, session, callback, node_name, node_opts, }, }, "link": { - "name": link_name, "durable": True, "x-declare": { "durable": False, @@ -138,6 +136,7 @@ def __init__(self, conf, session, callback, node_name, node_opts, "link": { "x-declare": { "auto-delete": True, + "exclusive": False, }, }, } @@ -145,6 +144,8 @@ def __init__(self, conf, session, callback, node_name, node_opts, raise_invalid_topology_version() addr_opts["link"]["x-declare"].update(link_opts) + if link_name: + addr_opts["link"]["name"] = link_name self.address = "%s ; %s" % (node_name, jsonutils.dumps(addr_opts)) @@ -208,14 +209,16 @@ def __init__(self, conf, session, msg_id, callback): if conf.qpid_topology_version == 1: node_name = "%s/%s" % (msg_id, msg_id) node_opts = {"type": "direct"} + link_name = msg_id elif conf.qpid_topology_version == 2: node_name = "amq.direct/%s" % msg_id node_opts = {} + link_name = None else: raise_invalid_topology_version() super(DirectConsumer, self).__init__(conf, session, callback, - node_name, node_opts, msg_id, + node_name, node_opts, link_name, link_opts) @@ -266,16 +269,14 @@ def __init__(self, conf, session, topic, callback): if conf.qpid_topology_version == 1: node_name = "%s_fanout" % topic node_opts = {"durable": False, "type": "fanout"} - link_name = "%s_fanout_%s" % (topic, uuid.uuid4().hex) elif conf.qpid_topology_version == 2: node_name = "amq.topic/fanout/%s" % topic node_opts = {} - link_name = "" else: raise_invalid_topology_version() super(FanoutConsumer, self).__init__(conf, session, callback, - node_name, node_opts, link_name, + node_name, node_opts, None, link_opts)