diff options
| author | Russell Bryant <rbryant@redhat.com> | 2012-07-27 17:37:02 -0400 |
|---|---|---|
| committer | Russell Bryant <rbryant@redhat.com> | 2012-07-27 18:02:00 -0400 |
| commit | fa9ad5e3585734f42ae235c0eb4c4518ba1c201c (patch) | |
| tree | 05e7f805c48d7d6a6d6c507e0c981136f01e2fd9 /nova/openstack | |
| parent | 3f315000755732c4780dae03ef65f67d3f87e8f7 (diff) | |
| download | nova-fa9ad5e3585734f42ae235c0eb4c4518ba1c201c.tar.gz nova-fa9ad5e3585734f42ae235c0eb4c4518ba1c201c.tar.xz nova-fa9ad5e3585734f42ae235c0eb4c4518ba1c201c.zip | |
Explicitly send primitive instances via rpc.
This fixes a few bugs in recent no-db-messaging commits. There were a
few places that I accidentally still used "instance" where I should have
used "instance_p", which is the instance converted to primitive types.
I noticed the problem looking at smokestack logs. I saw a Qpid
traceback that indicated the problem. As an interesting point, systems
using Kombu would not have seen this problem. Kombu uses anyjson. Our
jsonutils module sets anyjson to go through jsonutils, which will
automatically to jsonutils.to_primitive() if needed to serialize an
object. Qpid doesn't do json serialization, so it will blow up if it
hits a non-primitive type. Yay for testing multiple different
configurations to find problems.
Part of blueprint no-db-messaging.
Change-Id: I3487195110572b7ae9586cfcc618ab7ab28d5017
Diffstat (limited to 'nova/openstack')
0 files changed, 0 insertions, 0 deletions
