[ovs-dev] [PATCH ovn] ovn-controller: Unix command debug/delay-nb-cfg-report.

Han Zhou hzhou at ovn.org
Thu Sep 10 18:19:51 UTC 2020


On Thu, Sep 10, 2020 at 11:04 AM Ilya Maximets <i.maximets at ovn.org> wrote:

> On 9/10/20 7:05 PM, Han Zhou wrote:
> > On Thu, Sep 10, 2020 at 8:39 AM Numan Siddique <numans at ovn.org> wrote:
> >>
> >> On Wed, Sep 9, 2020 at 1:41 AM Han Zhou <hzhou at ovn.org> wrote:
> >>>
> >>> This command is added to avoid the flooding of nb_cfg updates from
> >>> a large nubmer of hypervisors interfering with the original SB DB
> >>> data distribution and handling, so that the e2e control plane latency
> >>> can be measured more accurately during scale testing with the command
> >>>     ovn-nbctl --print-wait-time --wait=hv ...
> >>>
> >>> Without this, there can be hypervisors updating back to SB before the
> >>> SB server finishes sending notifications to the rest of hypervisors.
> >>> In my test with 3K HVs, delaying the report for 2 seconds is enough to
> >>> get accurate latency report.
> >>>
> >>> Signed-off-by: Han Zhou <hzhou at ovn.org>
> >>
> >> Acked-by: Numan Siddique <numans at ovn.org>
> >>
> >> Note: You have to rebase before applying.
> >>
> > Thanks Numan. I applied this to master.
>
> The test in this patch is not stable and fails on travis from time to time:
>   https://travis-ci.org/github/ovn-org/ovn/jobs/726006756#L5318
>
> Probably, 3 seconds is not enough, or maybe there is a real bug somewhere.
>
> Best regards, Ilya Maximets.
>

Thanks Ilya, I just sent a fix:
https://patchwork.ozlabs.org/project/ovn/patch/1599761867-122691-1-git-send-email-hzhou@ovn.org/
Could you take a look? Sorry for the inconvenience.

Han


More information about the dev mailing list