diff options
| author | Jenkins <jenkins@review.openstack.org> | 2013-07-11 10:34:59 +0000 |
|---|---|---|
| committer | Gerrit Code Review <review@openstack.org> | 2013-07-11 10:34:59 +0000 |
| commit | 23f2ea29bcfbcfd27a1e4eb2712b0f65245cc7ec (patch) | |
| tree | a87c0ca9d5e72f5f6897da68dfd654fd5c68f477 | |
| parent | 1a2df89c05d33495d7e057dcdf0714d05beb1fc8 (diff) | |
| parent | f9f1b4f971f209feebb3bc6d32c397f4daa8ff1e (diff) | |
Merge "Revert "Add support for heartbeating in the kombu RPC driver""
| -rw-r--r-- | openstack/common/rpc/impl_kombu.py | 4 | ||||
| -rw-r--r-- | tests/unit/rpc/test_kombu.py | 21 |
2 files changed, 7 insertions, 18 deletions
diff --git a/openstack/common/rpc/impl_kombu.py b/openstack/common/rpc/impl_kombu.py index 67022ea..3e9e2d9 100644 --- a/openstack/common/rpc/impl_kombu.py +++ b/openstack/common/rpc/impl_kombu.py @@ -82,9 +82,6 @@ kombu_opts = [ default=0, help='maximum retries with trying to connect to RabbitMQ ' '(the default of 0 implies an infinite retry count)'), - cfg.IntOpt('rabbit_heartbeat', - default=60, - help='Seconds between connection keepalive heartbeats'), cfg.BoolOpt('rabbit_durable_queues', default=False, help='use durable queues in RabbitMQ'), @@ -452,7 +449,6 @@ class Connection(object): 'userid': self.conf.rabbit_userid, 'password': self.conf.rabbit_password, 'virtual_host': self.conf.rabbit_virtual_host, - 'heartbeat': self.conf.rabbit_heartbeat, } for sp_key, value in server_params.iteritems(): diff --git a/tests/unit/rpc/test_kombu.py b/tests/unit/rpc/test_kombu.py index 9838652..cbe948d 100644 --- a/tests/unit/rpc/test_kombu.py +++ b/tests/unit/rpc/test_kombu.py @@ -364,8 +364,7 @@ class RpcKombuTestCase(amqp.BaseRpcAMQPTestCase): 'password': FLAGS.rabbit_password, 'port': FLAGS.rabbit_port, 'virtual_host': FLAGS.rabbit_virtual_host, - 'transport': 'memory', - 'heartbeat': FLAGS.rabbit_heartbeat}]) + 'transport': 'memory'}]) def topic_send(_context, topic, msg): pass @@ -385,8 +384,7 @@ class RpcKombuTestCase(amqp.BaseRpcAMQPTestCase): 'password': 'fake_password', 'hostname': 'fake_hostname', 'port': 31337, - 'virtual_host': 'fake_virtual_host', - 'heartbeat': FLAGS.rabbit_heartbeat} + 'virtual_host': 'fake_virtual_host'} class MyConnection(impl_kombu.Connection): def __init__(myself, *args, **kwargs): @@ -398,8 +396,7 @@ class RpcKombuTestCase(amqp.BaseRpcAMQPTestCase): 'password': server_params['password'], 'port': server_params['port'], 'virtual_host': server_params['virtual_host'], - 'transport': 'memory', - 'heartbeat': FLAGS.rabbit_heartbeat}]) + 'transport': 'memory'}]) def topic_send(_context, topic, msg): pass @@ -767,29 +764,25 @@ class RpcKombuHATestCase(utils.BaseTestCase): 'password': FLAGS.rabbit_password, 'port': 1234, 'virtual_host': FLAGS.rabbit_virtual_host, - 'transport': 'memory', - 'heartbeat': FLAGS.rabbit_heartbeat}, + 'transport': 'memory'}, {'hostname': 'host2', 'userid': FLAGS.rabbit_userid, 'password': FLAGS.rabbit_password, 'port': 5678, 'virtual_host': FLAGS.rabbit_virtual_host, - 'transport': 'memory', - 'heartbeat': FLAGS.rabbit_heartbeat}, + 'transport': 'memory'}, {'hostname': '::1', 'userid': FLAGS.rabbit_userid, 'password': FLAGS.rabbit_password, 'port': 2345, 'virtual_host': FLAGS.rabbit_virtual_host, - 'transport': 'memory', - 'heartbeat': FLAGS.rabbit_heartbeat}, + 'transport': 'memory'}, {'hostname': '2001:0db8:85a3:0042:0000:8a2e:0370:7334', 'userid': FLAGS.rabbit_userid, 'password': FLAGS.rabbit_password, 'port': 5672, 'virtual_host': FLAGS.rabbit_virtual_host, - 'transport': 'memory', - 'heartbeat': FLAGS.rabbit_heartbeat}, + 'transport': 'memory'}, ] } |