Bug 2225156 - neutron-ovn-db-sync-util logs warning messages telling to update all router ports
Summary: neutron-ovn-db-sync-util logs warning messages telling to update all router p...
Keywords:
Status: POST
Alias: None
Product: Red Hat OpenStack
Classification: Red Hat
Component: python-networking-ovn
Version: 16.2 (Train)
Hardware: All
OS: All
medium
medium
Target Milestone: ---
: ---
Assignee: Lucas Alvares Gomes
QA Contact: Eran Kuris
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2023-07-24 12:29 UTC by Alex Stupnikov
Modified: 2023-09-06 10:21 UTC (History)
7 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed:
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
OpenStack gerrit 890799 0 None MERGED [OVN] ovn-db-sync check for router port differences 2023-08-28 13:13:43 UTC
Red Hat Issue Tracker OSP-26847 0 None None None 2023-07-24 12:33:48 UTC
Red Hat Knowledge Base (Solution) 7025531 0 None None None 2023-07-24 12:32:59 UTC

Description Alex Stupnikov 2023-07-24 12:29:16 UTC
Description of problem:

Warnings [1] are logged for all router ports when executing neutron-ovn-db-sync-util with --ovn-neutron_sync_mode=log argument:

It looks like ovn_db_sync.py adds all router ports to update_lrport_list instead of applying some filters and figuring out which ports shouldn't be added to this list.


Version-Release number of selected component (if applicable):
RHOSP 16.2


Steps to Reproduce:
1. Run neutron-ovn-db-sync-util with --ovn-neutron_sync_mode=log argument for ML2/OVN RHOSP deployment with routers


Actual results:
Warnings like [1] are logged for all router ports

Expected results:
Warnings like [1] are only logged if something is wrong with the port.


[1]
2023-07-24 11:46:31.391 952358 WARNING networking_ovn.ovn_db_sync [req-1081a8a6-82dd-431c-a2ab-f58741dc1677 - - - - -] Router Port port_id=f164c0f1-8ac8-4c45-bba9-8c723a30c701 needs to be updated for networks changed

Comment 2 Ihar Hrachyshka 2023-08-07 15:07:25 UTC
Lucas, please let us know if you are still looking at it. Thanks.

Comment 3 Lucas Alvares Gomes 2023-08-08 09:40:56 UTC
(In reply to Ihar Hrachyshka from comment #2)
> Lucas, please let us know if you are still looking at it. Thanks.

Hi Ihar,

I didn't have much time last week but I am planning to look at it this week.

Comment 4 Lucas Alvares Gomes 2023-08-08 14:29:31 UTC
Hi,

I pushed a patch upstream addressing the issue: https://review.opendev.org/c/openstack/neutron/+/890799, waiting for reviews


Note You need to log in before you can comment on or make changes to this bug.