The OVN tree for several years has had an OVS submodule that pulls
in the version of OVS it is guaranteed to work with. There is
currently a patch in OVS master that breaks the build of OVN main
so switch to using the submodule.
Change-Id: I61195cd3ebf8a5015890d3ba3988c3d9795ee817
To initialize OvsOvnVenvFixture need to pass 'ovsdir' as the keyword
argument, otherwise an TypeError will occur.
Change-Id: I0172d773f1ac92f1101311560b70fd001270140a
Change tools/setup-ovs.sh to take an OVS_BRANCH env
variable to use a specific branch.
Also, make OvsOvnVenvFixture work with an ovndir before
or after OVN repository split.
Change-Id: I9994c6e914bdb1deb5952c193e521f83ad74363d
Closes-Bug: 1856676
Signed-off-by: Flavio Fernandes <flaviof@redhat.com>
This is the bare minimum to run against ovs/ovn master after the
split. It should be updated to handle running against system-
installed OVN.
Change-Id: I264200ae016e3928fb4aff3bdcc7c1a94f91cae4
Occasionally oslo library usage sneaks in, so add a coding check
that checks for ^oslo\. in requirements.txt. We currently still
use oslo.test, so don't check test-requirements.txt. This should
fail to pass tests until I34357d5a594330d07f5cad372859d98efb5d3fde
merges.
Change-Id: I53108584ee47a7f1b6c7df90c49f95725cec69f9
We do not need tox_install.sh, pip can handle constraints itself
and install the project correctly. Thus update tox.ini and remove
the now obsolete tools/tox_install.sh file.
This follows https://review.openstack.org/#/c/508061 to remove
tools/tox_install.sh.
tox_install.sh was also building ovs for functional test. Move this into
tools/setup-ovs.sh and call it from functional tox environment.
Change-Id: I5857c8ae3e945328c0f24bda37629ce7e0faefa7
The change separating ovs and ovn venv classes requires debug_venv
to be updated to debug OVN tests.
Change-Id: Ie5ced4a499e5921636f726533bcab1e4bb52c77b
When writing tests, it is useful to be able to look at the database
being modified. Since we use OVS virtual environments and delete
them when the test ends, this is difficult to do.
This patch adds the ability to pass KEEP_ENV=1 when running tox to
keep the virtual environment around and adds tools/debug_venv to
launch an OVS sandbox using the OVS virtual environment.
Change-Id: Idbec2d544d913ffa8888a050b8a53d016c0315f2
This patch implments the ovn-nbctl API for the Logical_Switch,
Logical_Switch_Port, and DHCP_Options commands. Additional patches
will implement logical router and load balancer functionality.
As a convenience, the add/list/get commands return a special
read-only version of an ovs.db.idl.Row object called a RowView.
This object can be compared to a Row for equality and hashing. This
saves having to return uuids and then look them up. This behavior
differs from the Open_vSwitch schema implementation. This wrapper
serves both to keep people from modifying returned values outside of
a transaction and as an interface for any future backend to implement.
In addition, an ovs virtual environment fixture based on ovs-sandbox
is added to set up a sandboxed ovs/ovn install for running functional
tests.
Change-Id: I93689158467ff73a1b02588510d168b50ed6292a
This patch enables pylint 1.4.5 checking in ovsdbapp.
Script tools/coding-checks.sh is copied from neutron and adapted for
ovsdbapp.
.pylintrc is also copied from neutron, message options that do not
need to be suppressed have been removed.
This patch also adjusts code to fit pylint 1.7.1
Change-Id: If27c9a13798f20f3c22ca62a38cb93e7ad602a18
This patch consists of two squashed patches:
First patch copies in tox_install.sh from oslo libraries. The script
removes ovsdbapp from upper-constraints.txt as we want to install
code from the repository and not from pip.
Second patch is squashed from abandoned Iff59bbf41d166a0918a217cb72a37c547c6506b9.
Issue there was that code uses 'sudo' for running ovs-vsctl but the tests
user is not member of sudoers. It sets the Manager prior to executing
the tests. As ovsdbapp isn't supposed to depend on oslo, there is not
guaranteed way running process with ovsdbapp code will have root
privileges. Hence the Manager should be configured before application
using ovsdbapp is started.
Closes-bug: #1674937
Change-Id: I634ae2eeab7b68a81cb7208f0650705ac5a9a9ef