OVS: merge the required OpenFlow version rather than replace
This change modifies the behavior of OVS native and ovs-ofctl bridge
implementations so that instead of configuring the bridge only for the
required OVS protocol version, they add the required version to the
already configured versions.
To achieve this, an add_protocols method is added to the OVSBridge
class, relying on the OVSDB add_db_attribute added in
Ib6ce75846f9b13c1c33f0ced5ccc619ee7860dc1, with the behavior of
making the provided set of versions supported in addition to already
configured ones.
It is aimed to be a cleaner solution to bug 1622644 than the quickfix merge
from I4475865c4f83cb9f3e12c709af752bc490692ca3 .
After this change, the set_protocols method appears useless and is
hence marked for future removal.
Reviewed: https:/ /review. openstack. org/371455 /git.openstack. org/cgit/ openstack/ neutron/ commit/ ?id=271a4ffd6d7 00b5d092ff24592 ced324ded41e43
Committed: https:/
Submitter: Jenkins
Branch: master
commit 271a4ffd6d700b5 d092ff24592ced3 24ded41e43
Author: Thomas Morin <email address hidden>
Date: Fri Sep 16 09:46:27 2016 +0200
OVS: merge the required OpenFlow version rather than replace
This change modifies the behavior of OVS native and ovs-ofctl bridge
implementations so that instead of configuring the bridge only for the
required OVS protocol version, they add the required version to the
already configured versions.
To achieve this, an add_protocols method is added to the OVSBridge 9b13c1c33f0ced5 ccc619ee7860dc1 , with the behavior of
class, relying on the OVSDB add_db_attribute added in
Ib6ce75846f
making the provided set of versions supported in addition to already
configured ones.
It is aimed to be a cleaner solution to bug 1622644 than the quickfix merge 9f3e12c709af752 bc490692ca3 .
from I4475865c4f83cb
After this change, the set_protocols method appears useless and is
hence marked for future removal.
Depends-On: I4386aa293f9b18 d2e17b4a80d9c7d a4b9b46f3c9 c70d8404b16f895 33b6f270eee
Change-Id: Id5ac7e6431c97f
Related-Bug: 1622644