The FDP team is no longer accepting new bugs in Bugzilla. Please report your issues under FDP project in Jira. Thanks.
Bug 1964953 - [OVN] Add unit tests for ovn-northd with parallelization enabled.
Summary: [OVN] Add unit tests for ovn-northd with parallelization enabled.
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Enterprise Linux Fast Datapath
Classification: Red Hat
Component: OVN
Version: FDP 20.H
Hardware: Unspecified
OS: Unspecified
high
high
Target Milestone: ---
: ---
Assignee: xsimonar
QA Contact: Ehsan Elahi
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-05-26 12:23 UTC by Dumitru Ceara
Modified: 2023-03-13 07:13 UTC (History)
5 users (show)

Fixed In Version: ovn22.06
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2023-03-13 07:13:45 UTC
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker FD-1331 0 None None None 2022-04-11 14:14:15 UTC

Description Dumitru Ceara 2021-05-26 12:23:01 UTC 
Description of problem:

ovn-northd supports parallelization since [0].  The feature is disabled by default and the upstream CI matrix doesn't have any job that enables it.

Originally documented as item (4) at
https://bugzilla.redhat.com/show_bug.cgi?id=1960042#c4

[0] https://github.com/ovn-org/ovn/commit/8848c622bd01f54cab0be14a69478cc2ce0a19b7
Comment 1 Dumitru Ceara 2021-06-01 11:15:31 UTC 
ovn-northd parallelization is configurable via the NB database [0], e.g.:

ovn-nbctl set NB_Global . options:use_parallel_build=true

CI tests are run for different variations of ovn-northd using the OVN_FOR_EACH_NORTHD macro. [1]

[0] https://github.com/ovn-org/ovn/blob/master/ovn-nb.xml#L216
[1] https://github.com/ovn-org/ovn/blob/master/tests/ovn-macros.at#L539
Comment 2 Ilya Maximets 2021-06-01 13:23:07 UTC 
Suggestion:
Parallelization doesn't work if number of CPU cores is low (like in GHA).
To overcome this, it's probably natural and easy to port --dummy-numa
command line option form the OVS code base (northd uses ovs-numa library
to detect number of cores).
Comment 3 xsimonar 2022-04-11 14:08:04 UTC 
upstream fix: http://patchwork.ozlabs.org/project/ovn/patch/20220314082556.4035411-1-xsimonar@redhat.com/
Comment 4 Dan Williams 2022-06-21 14:55:08 UTC 
Accepted upstream for ovn 22.06 and later
Note You need to log in before you can comment on or make changes to this bug.