hacking 3.0.x is too old.
Try to synchronize pylint ignore and extension list with
other Networking projects.
With new pip the order of packages is not relevant, so the
related comment from requirements.txts is removed, see pip
documentation:
https://pip.pypa.io/en/stable/cli/pip_install/#installation-order
Change-Id: I586eb25b42d432e0b2158ddebb2dac013f712506
Removes the deprecated bridge-utils command and
replaces it with an equivalent ipruote2 command.
Closes-Bug: #2024498
Signed off-by: Juan Pablo Suazo <jsuazo@whitestack.com>
Change-Id: I7b48770709bd0fb9ea5072ce6c8673074c39ab2b
1. While creating br-tap bridge, pass an additional argument, i.e.
datapath_type. This parameter is read from openvswitch.ini
conf file.
2. Add tunneling flows for taas only when tunneling is enabled.
Closes-Bug: #2020688
Signed-off-by: Juan Pablo Suazo <jsuazo@whitestack.com>
Change-Id: Ie14e5a8e7a6a237311a538f6f0bf7452ac42f2ae
With [1] all the CLI code was removed from python-neutronclient,
This causes test discovery failure for tap-as-a-servic, and
the old neutron tap-service-create type commads can be removed from
the tap-as-a-service repository.
[1]: https://review.opendev.org/c/openstack/python-neutronclient/+/871711
Change-Id: I44d2009ca661ff88d7b4a2ed89528aefc3a1f2eb
Related-Bug: #2003861
In Python 3.10, Mock spec arguments are no longer allowed.
More details are available at: https://bugs.python.org/issue43478
Closes-Bug: #1954476
Change-Id: I98c3d6d67ba518f9c6c3455e9c5b0e2ea669a2f6
Latest pecan release dropped this dependency. This is why now it is
needed to explicitly install it during testing.
This patch also registers the Neutron core common config options in
``TaasExtensionTestCase``. Since [1], the configuration options
must be explicitly loaded.
[1]https://review.opendev.org/c/openstack/neutron/+/837392
Closes-Bug: #1982110
Change-Id: I3d8cd840881a2c67e7b07458276d600e214d028e
Remove test-setup.sh as with zuul it is not necessary, and would like to
use mysqladmin which is not available on some nodesets.
Fix failing unit tests with a possible typo and change auto_spec to
autospec in calling mock.patch.object.
Change-Id: Idd7b6788152a336314d5b99ae89f5049182a269c
Change RPC calls like create_tap_service or create_tap_flow signature
to avoid parameter names like tap_service which is really a dict of
values.
Change-Id: I0ec7fef934e63eff76dd30bb1fb28c8277f1a749
[0] enhanced the failure handling of taas agent, but the handling of
periodic_task was left out, which resulted in exceptions:
KeyError: 'periodic_tasks'.
To avoid such exception TaasAgentRpcCallback's self.func_dict must be
extended with periodic_task entry.
[0]: https://review.opendev.org/c/openstack/tap-as-a-service/+/681946
Change-Id: I623e373b391605f69e9f3ed8f50c6d99f15198bf
Closes-Bug: #1958373
OvsTaasDriver _create_tunnel_flood_flow_action used osv-vsctl call,
change that to get_port_name_list.
Change-Id: Ib8e160115564316991112a3a32034c645a427b31
Ovs constants were moved from neutron to neutron_lib some time ago.
This patch switches to use them from neutron-lib already.
This patch bumps min version of neutron-lib to 2.11 as it is version
which has those constants already.
It also fixes some small issue in grouping of imports.
Depends-On: https://review.opendev.org/c/x/tap-as-a-service-tempest-plugin/+/797318
Change-Id: I7c44501b129673cc72fd48f3b508ea6278027762
Config options 'driver' and 'enabled' are not used anymore, perhaps
since this commit was merged: Ife1807c04d8f0f44f067e1905890262ae2f7e426
Change-Id: I1e456354189608e28610a6c5cff3fe98e89a3e9f
Add possibility to tap-as-a-service to use openstack commands instead
of deprecated neutron commands.
Closes-Bug: #1874701
Change-Id: I339f42a63789a946011700902724a58647c80b9d
Make the code compatible with hacking rules:
* import order problems
* line break before/after binary operator (for this added ignore for
W504, line break after binary operator
* indentation problems
I removed the rule N530 (Direct neutron imports not allowed), as that is
a longer story.
Fix mysql syntax in tools/test-setup.sh
Fix constraints files for testing on focal
Change-Id: Iacb12d5add773404468e2b755cf95c4e6e54ec99
Depends-On: https://review.opendev.org/#/c/708965/
Depends-On: https://review.opendev.org/752416
As per current handling if there is any failure in the taas agent driver
there is no way to convey that to the plugin. Due to which end user gets
the impression that everything went well. This is confusing.
Now, as part of this enhancement, when the tap resources are :-
1. Created: iniitally they shall be created with status 'DOWN' by TaaS plugin. Once
the TaaS agent driver exectuion is finished, it will ask the plugin to
set the status to either 'ACTIVE' or 'ERROR' depending on whether the
driver was able to do its tasks successfully or not.
2. Deleted: as per curretn handling plugin used to first delete the
resources from DB and then inform the agent. Now the sequence would be
that plugin informs agent first. Agent shall ask the plugin to set the
resource status t either INACTIVE (indicates successful deletion,
whereby plugin shall clear the resources from the DB) or
PENDING_DELETE (failure in deleting the resources from driver, whereby
plugin would simply update the status for the resource in DB).
Change-Id: If8b1aba3b3955fd705f2a13a79c7225a03369da6
Remove sphinx from test-requirements.txt as the whole doc building is
is fixed in this patch: https://review.opendev.org/721994
Change-Id: I7a065a0cde1344af38655c9f136526eaa2524923
Adding oslo.policy allows operators to use and customize RBAC for tap-as-a-service instead of
relying on Neutron's default policy.
Change-Id: I6132054ef3bd8423990f91fae6329dfd089660b4
The following patch allows VF to VF mirroring in Tap-as-a-Service
Code-changes are applicable for new tap agent driver for sriov on
Intel i40e nic.
Vlan Mirror input parameter is part of tap-flow-create API
The current TaaS SRIOV driver is based on Intel i40e NIC driver with
following requirements:-
hardware: Intel Ethernet Network Adapter XXV710 (25GbE)
Driver: Intel i40e v4.16.0
Ref Spec: openstack/neutron-specs/specs/rocky/port-mirroring-sriov-vfs.rst
Commit: https://review.openstack.org/#/c/574477/
Change-Id: Id3aa83d7e1e22ae1806cef0c93e5dd61169c6735
All of the methods of common db mixin are available via neutron-lib
and the mixin will be removed before long.
This patch switches the code over to use neutron-lib's APIs rather
than those of the mixin and stops using the mixin as a parent class.
Change-Id: I1e9790e47c0befebc6f0073098694440dbc62873
We recently removed nenutron.common.rpc [1] as the rpc module lives in
neutron-lib now. This project was missed because I didn't find
neutron.common.rpc as an import.
This patch switches rpc uses over to neutron-lib.
[1] https://review.openstack.org/#/c/634790/
This commit includes a part of the following commit
to pass the gate.
----------------------------------------
Fix zuul job failures
Fix zuul job failures
Change-Id: I1d073b8c58e32669ecf6932512b94c8a2cabf70e
----------------------------------------
Co-Authored-By: Deepak Tiwari <deepak.tiwari@aricent.com>
Change-Id: I053b5c56594e1a914143a2308fef77349dfa7ea4
The model_query module is in neutron-lib and the CommonDBMixin will
eventually be removed. This patch swiches use of the _model_query
method over to query_with_hooks from neutron-lib.
This patch also bumps the neutron-lib version up to 1.20.0
Change-Id: I8ff41c9d914b8329e4fca90de8a9ef9f41ccdcde
The neutron.common.rpc and exceptions were rehomed into neutron-lib and
are currently shimmed in neutron [1]
This patch consumes those modules from neutron-lib by using lib's
modules rather than neutrons.
[1] https://review.openstack.org/#/c/586525/
Change-Id: Ifb87c315953a02291201a25aff05910149023d10
elajkat