Comment 12 for bug 1622879

Reviewed: https://review.openstack.org/373933
Committed: https://git.openstack.org/cgit/openstack/dragonflow/commit/?id=038b62b5da426550c1ffd294ed844790951226de
Submitter: Jenkins
Branch: master

commit 038b62b5da426550c1ffd294ed844790951226de
Author: Hong Hui Xiao <email address hidden>
Date: Wed Sep 21 05:18:43 2016 +0800

    Refactor OvsdbSwitchApi

    Current OvsdbSwitchApi provides ovs command sometimes, and outputs result
    directly sometimes. It makes it hard to use. Also, OvsdbSwitchApi uses
    neutron framework to execute ovs command sometimes, and uses idlutils
    directly sometimes. Using idlutils directly will not reconnect ovsdb,
    if timeout.

    This patch does following things:
    1) Create a dedicated ovsdb api, which is a sub class of OvsdbIdl from
    neutron
    2) Make OvsdbSwitchApi execute ovs command, and remove explicit execute()
    call from consumers
    3) Remove directly using idlutils in OvsdbSwitchApi
    4) Remove obselete methods in OvsdbSwitchApi
    5) Remove duplicated command class, which already have defination in
    neutron

    Change-Id: I7e9a7f9337c703673b7817ad47de1223c1b01ec0
    Closes-Bug: #1622879