]> review.fuel-infra Code Review - openstack-build/neutron-build.git/commit
Raise an error from ovs_lib list operations
authorAkihiro Motoki <motoki@da.jp.nec.com>
Mon, 10 Feb 2014 06:24:54 +0000 (15:24 +0900)
committerThomas Goirand <thomas@goirand.fr>
Thu, 13 Mar 2014 07:20:21 +0000 (15:20 +0800)
commit2cd2b0ee5ff3b4d07ee3cd726e103d6c52518e65
tree476ea2442184ad060cad774fa14918e08555ea06
parentc123a864e0c87afef7e90dc75cfc9667acf3603d
Raise an error from ovs_lib list operations

Previously list operations in ovs_lib returns an empty list
if RuntimeError occurs and a caller cannot distinguish an error
from normal results. This commit changes ovs_lib list operations
(get_vif_port_set, get_vif_ports, get_bridges) to raise an
exception when RuntimeError occurs.

Note: callers of these commands are ovs/nec/ryu-agent and ovs_cleanup.
- plugin agents: these commands are inside in try/except clause
  in daemon loop and there is no need to change.
- ovs_cleanup: there is no error catch logic in main() at now
  and it calls commands other than ovs_lib, so it can be cleanup
  later if required.

It also fixes the code to use excutils.save_and_reraise_exception
when reraising an exception.

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