diff options
| author | Russell Bryant <rbryant@redhat.com> | 2012-02-16 15:52:16 -0500 |
|---|---|---|
| committer | Russell Bryant <rbryant@redhat.com> | 2012-02-16 18:02:42 -0500 |
| commit | c8c9a87b240b562d334c6875a1dd7614d4ae58d5 (patch) | |
| tree | a7ea6a3e872faf914ae553e66e9cda7ebe91229d /nova/openstack | |
| parent | 34d77ac8b1919a287865a4bef376579b6bf09b48 (diff) | |
| download | nova-c8c9a87b240b562d334c6875a1dd7614d4ae58d5.tar.gz nova-c8c9a87b240b562d334c6875a1dd7614d4ae58d5.tar.xz nova-c8c9a87b240b562d334c6875a1dd7614d4ae58d5.zip | |
Add RPC serialization checking, fix exposed problems.
Related to bug 933584.
In this bug, I hit a case where some code tried to send a SQLAlchemy
model over rpc, which failed since it couldn't be serialized by Qpid.
This patch adds a simple serialization check to the fake RPC driver
using json. It also fixes problems that were exposed by adding this
check.
If json can't serialize a message sent through the fake RPC
driver, it will raise TypeError, causing unit tests to fail. 18 unit
tests failed with the check in place, but it was due to only 2 places
in the compute API.
Change-Id: I63f3077c0fa35097d4f5d2c485f4e48eede2c751
Diffstat (limited to 'nova/openstack')
0 files changed, 0 insertions, 0 deletions