Commit Graph

28053 Commits

Author SHA1 Message Date
Zuul f2342e0d17 Merge "Use native IntOpt for overlay_ip_version" 2024-04-16 21:47:49 +00:00
Zuul 73608c24a9 Merge "More precise flow table cleaning" 2024-04-16 21:47:43 +00:00
Zuul 234d7d4238 Merge "Mark L3HA dvr tests unstable" 2024-04-16 17:15:27 +00:00
yatinkarel 4d405ca675 Mark L3HA dvr tests unstable
These test randomly fails with NetworkInterfaceNotFound
from quite some time. Until the issue is fixed added unstable_test
decorator so the job do not fail for these.

Related-Bug: #1961740
Change-Id: I1a352526aac72b417f93f51ba50c3359359c240d
2024-04-16 18:59:29 +05:30
Zuul 019294c71d Merge "[OVN] Provide HA functionality to "Logical_Router" chassis pinning" 2024-04-11 11:01:43 +00:00
Zuul 4e9f00078b Merge "[OVN] "Logical_Router" pinned to chassis, OVN L3 scheduler" 2024-04-11 11:01:38 +00:00
Zuul 031aad583d Merge "Change exception messages to error log messages for DNS integration." 2024-04-10 23:53:01 +00:00
Jayce Houtman 5fe5188ce5 Change exception messages to error log messages for DNS integration.
Change non-harmful stack trace errors for dns_exc.DNSDomainNotFound and
dns_exc.DuplicateRecordSet to error log messages. This prevents the logs
from filling with stack traces where error messages would have been
sufficient enough.

Closes-Bug: #2042925
Change-Id: Icf1fff28bb560c506392f16c579de6d92cd56c23
2024-04-10 17:03:29 +02:00
LIU Yulong bac1b1f721 More precise flow table cleaning
OVS-agent wants to clean flows table by table during restart,
but actually it does not. If one table has same cookie with
other tables, all related flows will be clean at once.

This patch adds the table_id param to the related call
to limit the flow clean on one table at once.

Closes-Bug: #2060587
Change-Id: I266eb0f5115af718b91f930d759581616310999d
2024-04-09 14:10:55 +08:00
Rodolfo Alonso Hernandez bd31c23380 [OVN] Provide HA functionality to "Logical_Router" chassis pinning
When an external tunnelled network is used as gateway network in an
OVN router, the "Logical_Router_Port" is not bound to any chassis and
the "Logical_Router" is pinned to a gateway chassis, using the list
provided in a "HA_Chassis_Group".

This patch attends to any change in the "HA_Chassis" list of the
"HA_Chassis_Group" to update the "Logical_Router" chassis assigned.
This provides HA functionality in case that the bound chassis
(chassis pinned) fails.

Closes-Bug: #2052821
Change-Id: Ia3d4271d015386fbec3c3f2276a7f62c2f8ad5dd
2024-04-08 16:38:56 +00:00
Rodolfo Alonso Hernandez 25a1809964 [OVN] "Logical_Router" pinned to chassis, OVN L3 scheduler
Pin a "Logical_Router" to a chassis when the gateway network (external
network) is tunnelled. When the external network is tunnelled, the
"Logical_Router_Port" acting as gateway port is not bound to any
chassis (the network has no physical provider network defined).

In that case, the router is pinned to a chassis instead. A
"HA_Chassis_Group" is created per router. The highest "HA_Chassis" of
this group is assigned to the router. If the gateway port is deleted,
the pinned chassis is removed from the "options" field. If the
router is deleted, the "HA_Chassis_Group" is deleted too.

NOTE: in the a chassis belonging to the router "HA_Chassis_Group"
changes, the list of "HA_Chassis" will be updated in
``ChassisEvent.handle_ha_chassis_group_changes``. However, a
"HA_Chassis_Group" change is handled by OVN, when assiged.

But in this case we are using this artifact, as commented before,
to "manually assign" (from core OVN point of view) the highest
priority "HA_Chassis" to the router (this upcoming funcionality
will be implemented in core OVN). A new follow-up patch will be
pushed to provide HA functionality and update the "HA_Chassis"
assigned to the "Logical_Router" when the chassis list changes.

Partial-Bug: #2052821
Change-Id: I33555fc8a8441149b683ae68f1f10548ffb662a6
2024-04-08 16:38:09 +00:00
Takashi Kajinami a6e57754e0 Use native IntOpt for overlay_ip_version
The IntOpt class supports choices argument since oslo.config 9.4.0[1].

[1] 83bbc0df4316e8a17b8417d02c80cd0cf5a8568e

Change-Id: I27b825d7b65b6c40692785b50c8a8ccc3ca80b73
2024-04-08 15:04:15 +09:00
Zuul 4cad0eda59 Merge "Fix KeyError failure in _sync_subnet_dhcp_options()" 2024-04-06 00:26:55 +00:00
Zuul 8d0c4b92fc Merge "Add note on iptables cleanup after OVS firewall migration" 2024-04-06 00:26:51 +00:00
Zuul 59bee68b1d Merge "dhcp: ensure that cleaning DHCP process with one segment happens first" 2024-04-05 15:37:24 +00:00
Zuul 2db8620523 Merge "[OVN] Create an OVN DB transaction context decorator" 2024-04-02 03:03:52 +00:00
Zuul 38a53ea6ed Merge "Bandit: Remove bandit B311, B303 from skip list" 2024-04-02 00:24:04 +00:00
Zuul f08c967dd2 Merge "Update Neutron Liuetenants" 2024-04-01 22:31:47 +00:00
Zuul a017624a85 Merge "Refactor some docstrings" 2024-03-30 06:55:36 +00:00
Rodolfo Alonso Hernandez 00b64d289b
[OVN] Create an OVN DB transaction context decorator
This decorator allows to create a new OVN database transaction or
use the existing one in a method. It is needed to pass the transaction
object (if any) and the IDL (Northbound or Southbound).

Related-Bug: #2052821
Change-Id: I925c1d745197edd08a62ced66b275c7b1dad1d6a
2024-03-29 10:13:58 +01:00
elajkat 3c557b29f8 Bandit: Remove bandit B311, B303 from skip list
Remove B303 (md5, sha1 for python<3.9) and
remove B311 (Standard pseudo-random generators are
not suitable for security/cryptographic purpose) from
the skip list of bandit execution.

Change-Id: I6e9e61e7f94dc9ca339942529af8997adef45e38
2024-03-28 13:55:25 +01:00
Zuul 076d9ad47f Merge "Enable HA for OVN router flavors" 2024-03-27 21:31:11 +00:00
Zuul bcf1f707bc Merge "Disable skip-level jobs in check queue" 2024-03-27 09:26:21 +00:00
Zuul e7ef689e9b Merge "Update jobs based on testing runtime for 2024.2" 2024-03-27 07:34:13 +00:00
Miguel Lavalle 8ff8203fd7 Enable HA for OVN router flavors
In this change, we add the ability to create high availability
user defined router flavors under the ML2/OVN L3 service
plugin.

Closes-Bug: #2020823

Change-Id: I0d26f672d6239d840d3cf817a2553a06ef00a854
2024-03-26 20:07:52 -05:00
Zuul 01a6684dd0 Merge "Don't delete already deleted extra router routes" 2024-03-26 17:34:23 +00:00
Zuul 0590bcda68 Merge "Fixing the 500 HTTP code in the metadata service if Nova is down" 2024-03-26 16:42:10 +00:00
Zuul 73d647a7dd Merge "Check unspecified flavor in user defined driver" 2024-03-26 16:14:25 +00:00
Zuul b23bd63c2a Merge "Add documentation for aa-l3-gw-multihoming" 2024-03-26 14:24:42 +00:00
Anton Kurbatov 6395b4fe8e Fixing the 500 HTTP code in the metadata service if Nova is down
If the Nova metadata service is unavailable, the requests.request()
function may raise a ConnectionError. This results in the upper code
returning a 500 HTTP status code to the user along with a traceback.
Let's handle this scenario and instead return a 503 HTTP status code
(service unavailable).

If the Nova service is down and is behind another proxy (such as
Nginx), then instead of a ConnectionError, the request may result in
receiving a 502 or 503 HTTP status code. Let's also consider this
situation and add support for an additional 504 code.

Closes-Bug: #2059032
Change-Id: I16be18c46a6796224b0793dc385b0ddec01739c4
2024-03-26 12:14:08 +00:00
Miguel Lavalle 9d729bda20 Check unspecified flavor in user defined driver
In order to decide whether to process a router related
request, the user defined router flavor OVN driver needs to
check the flavor_id specified in the request. This change adds
the code to test the case when the API passed the flavor_id as
unspecified.

Change-Id: I4d7d9d5582b97246cad63ef7f5511b159d6c6791
Closes-Bug: #2059051
2024-03-25 17:30:01 -05:00
Brian Haley 056baab66e Update Neutron Liuetenants
This patch updates the list of the Neutron stadium
projects lieutenants and the list of the bugs' contact
person(s).

In detail this patch:
- sets Slawek Kaplonski as contact for RBAC issues
- sets Fernando Royo as contact for 'ovn-octavia-provider' issues,
  removing Flavio and myself
- adds new 'ovn-bgp-agent' item with Luis Tomas Bolivar and
  Lucas Alvares Gomes as contacts

Change-Id: Iaf344ee30a3500c18ae7facd9010d75af39e995f
2024-03-25 14:15:54 -04:00
Zuul e003fd73f6 Merge "Clean up state VRRP PID file" 2024-03-25 16:12:28 +00:00
Sahid Orentino Ferdjaoui 5453c92a2e dhcp: ensure that cleaning DHCP process with one segment happens first
Previously, the code used to clean up old DHCP processes for a network
before creating new ones supporting multiple segments per network
could potentially not be executed first. Since disabling applies to
cleaning the namespace, this could have led to the network setup being
destroyed after being done.

This change moves the part that cleans up the old DHCP setup to ensure
it is executed first.

Closes-bug: #2049615
Signed-off-by: Sahid Orentino Ferdjaoui <sahid.ferdjaoui@industrialdiscipline.com>
Change-Id: Iecdb2d81ee077c9b9057d0708c5c88e159970039
2024-03-25 10:22:05 +01:00
Youngjun 418d87b276 Refactor some docstrings
Change-Id: I0ed606d0b206f3bb361e602b4b8ec8f36355c54d
Signed-off-by: Youngjun <yj.yoo@okestro.com>
2024-03-25 00:39:47 +00:00
Zuul 4e9d03d29f Merge "Fix used-before-assignment warnings" 2024-03-22 01:27:48 +00:00
Zuul 57f48b03ac Merge "Fix disallowed-name warnings" 2024-03-22 01:27:40 +00:00
Brian Haley 882885b806 Update jobs based on testing runtime for 2024.2
Based on [0] only py39, py311 and py312 (non-voting) jobs
are in the testing runtime for 2024.2, updated relevant
files accordingly. Added timeout override for py312
similar to other versions.

[0] https://review.opendev.org/c/openstack/governance/+/908862

Change-Id: I6c3e94a88b7ef50b1fc74abe0ef4640ce3a68be3
2024-03-21 14:50:49 -04:00
Rodolfo Alonso Hernandez 70ddf4eef5 Add "socket" NUMA affinity policy
This new extension adds a new parameter to the NUMA affinity policy
list: "socket". The "socket" NUMA affinity policy has been supported
in Nova since [1].

[1]https://review.opendev.org/c/openstack/nova/+/773792

Closes-Bug: #2052786
Change-Id: Iad2d4c461a2aceef6ed2d5e622cce38362d79687
2024-03-21 16:04:24 +00:00
Zuul ca479b9e22 Merge "Open the 2024.2 (Dalmatian) DB branch" 2024-03-21 15:53:43 +00:00
Zuul e5d0877045 Merge "Enhance IptablesFirewallDriver with remote address groups" 2024-03-21 10:08:15 +00:00
Robert Breker 5e1188ef38 Enhance IptablesFirewallDriver with remote address groups
This change enhances the IptablesFirewallDriver with support for remote
address groups. Previously, this feature was only available in the
OVSFirewallDriver. This commit harmonizes the capabilities across both
firewall drivers, and by inheritance also to OVSHybridIptablesFirewallDriver.

Background -
The Neutron API allows operators to configure remote address groups [1],
however the OVSHybridIptablesFirewallDriver and IptablesFirewallDriver do
not implement these remote group restrictions. When configuring security
group rules with remote address groups, connections get enabled
based on other rule parameters, ignoring the configured remote address
group restrictions.
This behaviour undocumented, and may lead to more-open-than-configured network
access.

Closes-Bug: #2058138
Change-Id: I76b3cb46ee603fa5e829537af41316bb42a6f30f
2024-03-20 22:20:45 +00:00
Rodolfo Alonso Hernandez 8830fed191 Open the 2024.2 (Dalmatian) DB branch
According to the TC resolution [1] and the release identification
document [2], the release number will be used as primary identifier in
the development cycle. Release name will be used in marketing.

[1]https://governance.openstack.org/tc/resolutions/20220524-release-identification-process.html
[2]https://governance.openstack.org/tc/reference/release-naming.html

Change-Id: I6fc7a154fa77a8792f93eb40c9b4497c0e67ae2f
2024-03-20 10:00:19 +00:00
Zuul 4c9962e280 Merge "Fix making all user defined flavor routers HA" 2024-03-20 00:21:55 +00:00
Zuul 95e4e3b092 Merge "Use oslo_service's SignalHandler for signals" 2024-03-20 00:21:51 +00:00
Miguel Lavalle 26ff51bf05 Fix making all user defined flavor routers HA
Since [1] was merged, user defined flavor routers with the HA
attribute set to False cannot be created. This change fixes
it.

Closes-Bug: #2057983

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

Change-Id: Ic72979cfe535c1bb8cba77fb82a380c167509060
2024-03-18 19:20:03 -05:00
Brian Haley 46245c0154 Add note on iptables cleanup after OVS firewall migration
Add an item to the instructions on iptables to OVS
firewall migration that the admin should cleanup
any stale iptables rules after completion. It is
out of scope of our documents on how exactly an
adminstrator might do that.

Closes-bug: #1864374
Change-Id: Ie1bf6b82e57a00f61640a131a29d897a9cde4629
2024-03-18 16:38:35 -04:00
Zuul 1b3bc34f67 Merge "Add metadata options to generated ovn_agent.ini" 2024-03-18 17:29:38 +00:00
Zuul 91e8dc52b5 Merge "[OVN] Add documentation about the OVN agent" 2024-03-18 17:25:04 +00:00
Takashi Kajinami 446a7acf2c Add metadata options to generated ovn_agent.ini
The ovn-agent process now loads these options for metadata extension.

Change-Id: I3fcacc9d54e34746fd0ef55f5f74c9f4bd6fbf97
2024-03-16 16:32:32 +09:00