| commit | 2447b3811b8a6c839fe42691fa2596fa74fbe837 | [log] [tgz] |
|---|---|---|
| author | Armando Migliaccio <armamig@gmail.com> | Wed Jul 20 23:06:26 2016 -0700 |
| committer | Armando Migliaccio <armamig@gmail.com> | Thu Jul 21 15:41:40 2016 +0000 |
| tree | f24b856731373c546273a1c8e9c8a16d957fdced | |
| parent | 1c602061616e564dae2b1039bd048e2426585de5 [diff] |
Revisit (add|remove)_subports request body
The neutronclient [1] expects to serialize a dictionary any
time there is a body in the request. The (add|remove)_subports
methods were expecting a list of dicts instead.
This patch rectifies the issue by assuming a dict in the
body of the format {'sub_ports': [<my-subports]}. This is
indeed the most conventional way to pass request data to
Neutron server, and it was overlooked during a past review.
This was discovered during development of the client side
patch [2].
[1] https://github.com/openstack/python-neutronclient/blob/master/neutronclient/v2_0/client.py#L292
[2] https://review.openstack.org/#/c/340624/
Partially-implements: blueprint vlan-aware-vms
Change-Id: I8d36c61dd877d0ccb73c18702ed64897b71f001b