Bug 2222046

Summary: OVN should insert ARP / ND responder flows for ports that have `unknown` address if they also have IP/MAC set
Product: Red Hat Enterprise Linux Fast Datapath Reporter: Ihar Hrachyshka <ihrachys>
Component: ovn22.12Assignee: Dumitru Ceara <dceara>
Status: CLOSED WONTFIX QA Contact: Jianlin Shi <jishi>
Severity: unspecified Docs Contact:
Priority: high    
Version: FDP 23.JCC: ctrautma, dceara, jiji, mmichels
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard:
Fixed In Version: Doc Type: If docs needed, set a value
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2023-08-15 15:36:16 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:

Comment 2 Ihar Hrachyshka 2023-08-07 19:55:59 UTC 
I believe the current thinking is that if there's a fix on OVN side for this scenario, it would involve OVN controller splitting the list of ports for the UNKNOWN flow into pieces, each of which would be below the kernel/netlink limit. Then call to controller() at the end of each chunk, which would reinject the packet back into the table, for the next chunk, until all chunks are completed.

If so, the description here should be adjusted to reflect this.
Comment 3 Dumitru Ceara 2023-08-15 15:36:16 UTC 
(In reply to Ihar Hrachyshka from comment #2)
> I believe the current thinking is that if there's a fix on OVN side for this
> scenario, it would involve OVN controller splitting the list of ports for
> the UNKNOWN flow into pieces, each of which would be below the
> kernel/netlink limit. Then call to controller() at the end of each chunk,
> which would reinject the packet back into the table, for the next chunk,
> until all chunks are completed.
> 
> If so, the description here should be adjusted to reflect this.

Ihar, you're right but for our team's tracking it's better if we open a new bug to track the alternate approach.  I opened bug 2232152 for that and will be closing this one.