26247 Commits

Author SHA1 Message Date
Zuul
6a7bb5d0fa Merge "[OVN] Document baremetal provisioning with ML2/OVN" 2022-06-22 15:01:06 +00:00
Zuul
af0dd9bcb5 Merge "migration: Remove patch port between br-int and br-tun" 2022-06-22 06:43:24 +00:00
Zuul
e5369b4d00 Merge "Use ovs constants from neutron-lib" 2022-06-21 19:21:42 +00:00
Zuul
0eb2ff6302 Merge "[fips jobs] blacklist AttachInterfacesTestJSON tempest test" 2022-06-21 17:05:19 +00:00
Zuul
35feaeedd8 Merge "Set nslookup_target in FIPS jobs" 2022-06-21 17:05:14 +00:00
Zuul
e867a9b31d Merge "[OVS][QoS] Unset the min-bw QoS from the phys bridge interface" 2022-06-21 10:03:38 +00:00
a1384678b8 [fips jobs] blacklist AttachInterfacesTestJSON tempest test
libvirt-8.4.0-1.el9 caused a regression which resulted into
AttachInterfacesTestJSON and test_hotplug_nic tempest test to
fail, until fixed libvirt version is available in CentOS 9-Stream
repos, let's exclude these tests from jobs running
on CentOS 9-Stream.

Related-Bug: #1979047
Related-Bug: rhbz#2092856
Change-Id: Ia97fc084dcbdfbced0c2cc79f127a97b4d97d1e3
2022-06-21 12:55:38 +05:30
Zuul
ae2d4c1da1 Merge "Update port-mac-address-override shim extension" 2022-06-20 17:52:19 +00:00
51360b5a08 Set nslookup_target in FIPS jobs
FIPS jobs reboot nodes after enabling FIPS,
but unbound service takes some time to
be ready and until then DNS resolution fails.
[1] added support to check DNS is ready after
reboot, in order to utilize it we need to
set "nslookup_target" job var.

[1] https://review.opendev.org/c/zuul/zuul-jobs/+/844704

Closes-Bug: #1978938
Change-Id: Ia39753f7cc533ba9959136b95bc3401d2f34cad4
2022-06-20 20:59:53 +05:30
Zuul
92b70efca0 Merge "Remove unneeded contexts in `_create_port_security_group_binding`" 2022-06-17 19:21:38 +00:00
Rodolfo Alonso Hernandez
9f5e211796 [OVS][QoS] Unset the min-bw QoS from the phys bridge interface
The minimum bandwidth OVS QoS policy is set in the physical bridge
interface, not the port that receives the Neutron QoS policy.
If no other port has a minimum bandwidth rule, the OVS QoS policy
can be unset from this port and deleted. This patch fixes the
port that must be updated.

Closes-Bug: #1977819
Change-Id: I0ebcc85f5ffa9dc29ffedee0c7df1828c85e1576
2022-06-17 13:48:29 +00:00
Rodolfo Alonso Hernandez
4dbcd76517 Fix "ip route" commands to work with pyroute2 APIv2
Fixed "ip route" commands to work with pyroute2 >= 0.6.10, that
introduces APIv2 for link, neighbour and route [1].

[1]bc0f5e2209

Closes-Bug: #1979031
Change-Id: Id2239b6827485a4d466b0916947428ceabef9139
2022-06-17 08:39:44 +00:00
Slawek Kaplonski
a22d6d6a95 Use ovs constants from neutron-lib
Ovs constants were moved from neutron to neutron_lib some time ago.
This patch switches to use them from neutron-lib already.

That decision was agreed during the Neutron team meeting. See [1] for
details.

[1] https://meetings.opendev.org/meetings/networking/2021/networking.2021-11-09-14.00.log.html#l-83

Requires: https://review.opendev.org/c/openstack/neutron-lib/+/834908

Change-Id: I2fd1954bec6a52856195190441d77ac8b7d97055
2022-06-17 10:36:44 +05:30
Zuul
0e40dfe862 Merge "Support pps limitation for openvswitch agent" 2022-06-14 16:58:27 +00:00
Zuul
d76eab3122 Merge "Meter flows and ovsdb action for ovs bridge" 2022-06-14 16:58:22 +00:00
Zuul
beef31d423 Merge "migration: Remove restarting OVS" 2022-06-14 16:49:58 +00:00
Zuul
74a8e9fa78 Merge "Move `PortBindingChassisEvent checks to match_fn`" 2022-06-14 16:49:52 +00:00
Zuul
b7477be3df Merge "Fix tests involving pyroute2.netlink module" 2022-06-14 16:49:16 +00:00
Zuul
db93cc5555 Merge "Use SQL IN clause instead of OR with exact match comparisons" 2022-06-14 16:49:11 +00:00
Zuul
fab415cfe2 Merge "Return default value when get router's enable_ndp_proxy from request_body" 2022-06-14 15:03:01 +00:00
Dr. Jens Harbott
d01700034a Fix tests involving pyroute2.netlink module
pyroute2 0.6.11 did some further refactoring of their module imports,
so using pyroute2.netlink.some_func() doesn't work anymore. Since we
do "from pyroute2 import netlink" anyway, use the imported module
everywhere.

Also, mock.patch()ing pyroute2.netlink.rtnl.tcmsg.common.tick_in_usec
doesn't work anymore, but also doesn't seem to be necessary for the
tests to succeed, so just drop it.

Closes-Bug: 1978564
Change-Id: Ie06b987f7efbff8057293893e42bee7b682b9ba1
2022-06-14 11:47:56 +00:00
Zuul
2c4b1c970c Merge "[UT] Reduce the port binding retries to 1 in `TestPortsV2`" 2022-06-14 08:30:43 +00:00
yangjianfeng
81a330113b Return default value when get router's enable_ndp_proxy from request_body
In some cases, the creation of router was called by `l3 plugin`
directly, didn't call API. In this way, the router's request_body
of the `ndp_proxy` service plugin received has no `enable_ndp_proxy`,
this will result in `ndp_proxy` service plugin raise error.

Closes-Bug: #1978519
Related-Bug: #1877301
Change-Id: I413c33970002532d11de5bc2afb145704a8488a9
2022-06-14 12:05:44 +08:00
Zuul
3447e0ee6f Merge "Optimize queries for port operations" 2022-06-14 01:07:35 +00:00
Jakub Libosvar
fd6adb3e5b migration: Remove restarting OVS
There was a code that worked around a bug in openvswitch. The bug was
fixed in openvswitch 2.13 and we no longer use that version. It's safe
to be removed now.

Change-Id: I8d0e1dac4f6a6d201e29863cc76db3f1ff8595ae
Signed-off-by: Jakub Libosvar <libosvar@redhat.com>
2022-06-13 16:14:51 -04:00
Fernando Royo
c25097b0b0 Optimize queries for port operations
Port create/update are most time-consuming operations
on subnet creation. As example, in those cases where
several subnets are created over the same network the
response time for those port operations is linearly
increased as the total subnets increases.

This patch improves the number of queries required on port
operations in order to reduce the response time.

Closes-Bug: #1977831
Change-Id: I0fccf36a2035e8f6c2fa8dab0307358da600c8f7
2022-06-13 10:30:11 +02:00
Rodolfo Alonso Hernandez
5cc7175659 [UT] Reduce the port binding retries to 1 in `TestPortsV2`
In ``TestPortsV2`` tests, set the
``neutron.plugins.ml2.plugin.MAX_BIND_TRIES`` to 1 to minimize the
number of retries during a failed port binding. That shortens all
test cases execution time if a port binding is executed.

Trivial-Fix

Change-Id: If0473031797984aab5b36c479fcb774e57ff5624
2022-06-13 08:21:09 +00:00
Rodolfo Alonso Hernandez
a1d135e3c0 Use SQL IN clause instead of OR with exact match comparisons
SQL IN clause is faster than several exact match comparisons
in an OR clause.

Trivial-Fix

Change-Id: I0d1bc49927b061a91ff6d4ca3f5b826d529d0770
2022-06-13 06:35:25 +00:00
chenwei
3e1223f5ca Change vlan max value in intro-basic-networking.rst
Cloesed bug: #1978389

Change-Id: I23d2ec86bb719c89886c24186745f04c17d8964c
2022-06-13 07:45:04 +08:00
Zuul
275b458f14 Merge "openflow_processed_per_port should be ovs-agent only" 2022-06-10 18:55:05 +00:00
Zuul
357e345fe4 Merge "[OVN] Make binding profile validation more robust" 2022-06-10 18:55:00 +00:00
Zuul
e8125f002b Merge "Set "type=virtual" for OVN LSP with parent ports" 2022-06-10 18:54:56 +00:00
Zuul
370c6ddc95 Merge "remove unused updated_at parameter for AgentCache.update" 2022-06-10 18:54:35 +00:00
Zuul
a3d3622e07 Merge "[ovn]neutron agent show real heartbeat_timestamp" 2022-06-10 16:13:02 +00:00
Zuul
38cdfb8855 Merge ""_handle_lb_fip_cmds" to handle the LB members correctly" 2022-06-09 17:13:04 +00:00
Zuul
d77a969d3e Merge "Remove unused get_log_args" 2022-06-09 17:12:59 +00:00
Zuul
3733520df6 Merge "[sqlalchemy-20] Use session.transaction information to decide if active" 2022-06-09 15:29:12 +00:00
zhouhenglc
411ecc4865 [ovn]neutron agent show real heartbeat_timestamp
agent's heartbeat_timestamp returns the current time, not
conducive to agent status troubleshooting.
this patch use agent's updated_at as heartbeat_timestamp.

Closes-bug: #1977629

Change-Id: Idf522a20f9735829ee568020bfed46345a95e294
2022-06-09 18:34:09 +08:00
Zuul
e95174cfe6 Merge "Fix tests involving pyroute2.netns module" 2022-06-09 10:07:03 +00:00
Zuul
575766df1f Merge "Switch tick-tick job to upgrade from xena" 2022-06-09 09:38:29 +00:00
Zuul
4f7b138fae Merge "Add "max-rate" value to minimum bandwidth rules" 2022-06-09 09:38:23 +00:00
zhouhenglc
7dbc61346a remove unused updated_at parameter for AgentCache.update
we cannot find this parameter passed in anywhere except for some unit
tests. currently we have used nb_cfg_timestamp[1] as agent updated
time. there are no other scenarios for this parameter. this patch
remove it and update some unit test.

[1] https://review.opendev.org/c/openstack/neutron/+/802834

Closes-bug: #1978035

Change-Id: Ic964b7ddc70988bb1a822b07be6a1be4d197287e
2022-06-09 06:03:13 +00:00
Dr. Jens Harbott
fe4fa55642 Fix tests involving pyroute2.netns module
pyroute2 0.6.11 did some further refactoring of their module imports,
adapt some mocks to that the tests work with both the older and the new
versions.

Change-Id: I00c1b1e81c33cf93372dec21f4a9f409d784a160
2022-06-08 22:07:02 +02:00
Takashi Kajinami
edcb83f296 Remove unused get_log_args
This method has been unused since [1] was merged.

[1] 3b22541a2aa9a5b06e2bff256701dbe24554c17c

Change-Id: I6765c1277341d7ec19fd6eae398fdd4ef9ef66e0
2022-06-08 14:59:38 +00:00
Lucas Alvares Gomes
402a976820 [OVN] Document baremetal provisioning with ML2/OVN
This patch adds documentation for baremetal provisioning with ML2/OVN.

This patch also splits the section about external ports to a different
document so it can be shared between baremetal and SR-IOV as both
features uses the same feature from core OVN.

Change-Id: I48cbd73c3c0fcf0393e31356c081ab1561eecc79
Signed-off-by: Lucas Alvares Gomes <lucasagomes@gmail.com>
2022-06-08 13:39:28 +01:00
Zuul
e44dbe98e8 Merge "Defer flow deletion in openvswitch firewall" 2022-06-08 09:31:34 +00:00
Rodolfo Alonso Hernandez
6b529ea3c5 [sqlalchemy-20] Use session.transaction information to decide if active
The "session.transaction" member "_connections" is a dictionary with the
active database connections of the current transaction. Neutron will use
it to determine if the session is active or not. This parameter will
provide more accurate information than "new", "dirty" or "deleted". A
read (SELECT) operation won't be included on those lists while the
transaction is currently active.

Closes-Bug: #1975542
Change-Id: Ibb75b324950c639d30c77d4e5269a8c53d6a0bb5
2022-06-08 08:21:34 +00:00
Zuul
db9300ed11 Merge "migration: Fail task if executed script failed" 2022-06-08 00:03:27 +00:00
Rodolfo Alonso Hernandez
a575dbc4a1 Add "max-rate" value to minimum bandwidth rules
OVS translates the QoS and associated queue registers,
attached to a port, into "tc" linux-htb classfull traffic
shaper, applied on the port.

OVS creates a "tc" root class on the port device. On top of
this root class, the queues are represented as child "tc"
classes. In order to define a "min-rate" value ("tc rate"),
a "max-rate" value ("tc ceil") must be provided and higher than
"min-rate".

By default, OVS agent QoS minimum rules do not have a "max-rate"
defined. Before this patch, any minimum bandwidth rule was
limiting the maximum rate to 100Mbit/s, that is the default value
set in "tc ceil". This patch provides the maximum "max-rate" value
for any minimum bandwidth rule.

Closes-Bug: #1977752

Change-Id: I2c6f09548f39cadfe85e57032091a70a5bc978e5
2022-06-07 13:20:11 +00:00
Zuul
69b98d011a Merge "migration: Delete also SGRs for IPv6" 2022-06-07 11:17:16 +00:00