]> review.fuel-infra Code Review - openstack-build/cinder-build.git/commit
Code Review / openstack-build / cinder-build.git / commit
? search:
summary | shortlog | log | commit | commitdiff | review | tree
(parent: 22a9da3) | patch
Add missing ack to impl_qpid.
authorRussell Bryant <rbryant@redhat.com>
Tue, 12 Jun 2012 21:07:18 +0000 (17:07 -0400)
committerJenkins <jenkins@review.openstack.org>
Wed, 18 Jul 2012 16:42:22 +0000 (16:42 +0000)
commita5d16cfb8d6d953d2a9689ddc855cade604e1e9b
treeef5f51462e5473979cca425b7dca0ae5d1c361d2tree | snapshot
parent22a9da3cc4b0cc17d8d3483b95dc57ed43bf0ecbcommit | diff
Add missing ack to impl_qpid.

Fix bug 1012374.

Johannes Erdfelt pointed out that impl_qpid wasn't acking messages that
it received.  This turned out to be a nasty oversight, resulting in
unbounded message queue growth inside of the python-qpid library.  This
fixes it.

Change-Id: I0370293807f0282e1dbdd59246f70be031e888a9
Reviewed-on: https://review.openstack.org/9908
Reviewed-by: Vish Ishaya <vishvananda@gmail.com>
Approved: John Griffith <john.griffith@solidfire.com>
Tested-by: Jenkins
cinder/rpc/impl_qpid.py diff | blob | history
cinder/tests/rpc/test_qpid.py diff | blob | history