diff options
| author | Yun Mao <yunmao@gmail.com> | 2012-08-23 16:37:42 -0400 |
|---|---|---|
| committer | Yun Mao <yunmao@gmail.com> | 2012-08-23 16:54:32 -0400 |
| commit | 32b0346eb40a94ec6def3eed01e9424b0c660c53 (patch) | |
| tree | f12e0e59e5b9bf00c4be376892bd40c76db71d21 /nova/openstack | |
| parent | 13937c028df30de8649685b471c5fbbcf634cedd (diff) | |
| download | nova-32b0346eb40a94ec6def3eed01e9424b0c660c53.tar.gz nova-32b0346eb40a94ec6def3eed01e9424b0c660c53.tar.xz nova-32b0346eb40a94ec6def3eed01e9424b0c660c53.zip | |
Fix deallocate_fixed_ip() call by unifying signature
deallocate_fixed_ip() is defined in 3 classes with incompatible
signatures:
RPCAllocateFixedIP:
deallocate_fixed_ip(self, context, address, host, **kwargs)
NetworkManager:
deallocate_fixed_ip(self, context, address, **kwargs)
FlatManager:
deallocate_fixed_ip(self, context, address, **kwargs)
There is a non-trivial multiple inheritance relationship among
them. The complication leads to incorrect function call signatures
(see bug 1024789). The kwargs magic is also never used, and discarded
during rpc calls anyway.
This patch makes the call signature exactly the same for all
deallocate_fixed_ip() definitions, drops kwargs and fixes bug
1024789.
Change-Id: I9c139810d4a5c93fa9f50b89a07bfe44e73e6c8a
Diffstat (limited to 'nova/openstack')
0 files changed, 0 insertions, 0 deletions