Bug 1922489 - ovn-controller refactor: Refactor the incremental engine
Summary: ovn-controller refactor: Refactor the incremental engine
Keywords:
Status: NEW
Alias: None
Product: Red Hat Enterprise Linux Fast Datapath
Classification: Red Hat
Component: OVN
Version: FDP 21.B
Hardware: Unspecified
OS: Unspecified
low
unspecified
Target Milestone: ---
: ---
Assignee: OVN Team
QA Contact: Jianlin Shi
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-01-29 21:48 UTC by Mark Michelson
Modified: 2023-07-13 07:25 UTC (History)
1 user (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
Red Hat Issue Tracker FD-1060 0 None None None 2021-12-03 14:27:29 UTC

Description Mark Michelson 2021-01-29 21:48:21 UTC
See https://docs.google.com/document/d/16PFU3OpjMvnYjILAY00FlnabfUyIOWOPR-1tTFh_I10/edit?ts=60129a42

This corresponds with phase 3 of the linked document.

The incremental engine's scope needs to be reworked so that it only compares received data to cached data, thus outputting the new, deleted, and updated data to lower layers.

The lower layers then need to operate on this "diff" data.

This issue may need to be divided into subtasks based on individual data types handled by the incremental engine. However, issue #1922486 needs to be completed first so that the data types are known before this can be subdivided.

Comment 1 Mark Michelson 2021-12-03 14:26:35 UTC
Prioritizing this "low" because since this issue was opened, the idea of an ovn-controller refactor has fallen way down the list of important things. Changes in ovn-controller have made it perform light years better than it used to. While it's still in need of cleanup, we don't also have the increased performance angle to lean on. In fact, it's possible that the refactor on its own would cause performance regressions initially, since the focus would mainly be on writing error-free clean code.


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