]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Parse JSON in ovs_lib.get_vif_port_by_id
authorSalvatore Orlando <salv.orlando@gmail.com>
Sun, 16 Feb 2014 23:47:34 +0000 (15:47 -0800)
committerThomas Goirand <thomas@goirand.fr>
Thu, 13 Mar 2014 07:20:23 +0000 (15:20 +0800)
commit32317758e6338346c2fe9f5c13944532571eb480
tree14b1c937348b7a5d29ad4146c56c687564bfb34a
parent153cd77ffa94d661088412aa5dd36df07999ce1f
Parse JSON in ovs_lib.get_vif_port_by_id

This patch replaces regex matching of text output with parsing
of JSON output in ovs_lib.get_vif_port_by_id.
This makes the code more reliable as subtle, possibly even
cosmetic, changes in ovs-vsctl output format could cause the
regular expression match to fail.

Also, this makes the code consistent with ovs_lib.get_vif_port_set
which already uses JSON output.

Finally this patch slightly changes the behaviour of
ovs_lib.get_vif_port_by_id returning None if elements such as
mac address or ofport were not available.

Change-Id: Ia985a130739c72b5b88414a79b2c6083ca6a0a00
Closes-Bug: #1280827
neutron/agent/linux/ovs_lib.py
neutron/tests/unit/openvswitch/test_ovs_lib.py