[ovs-git] [ovn-org/ovn] 4f366b: northd: Don't poll ovsdb before the connection is ...

Renat Nurgaliyev noreply at github.com
Thu Nov 12 09:32:50 UTC 2020


  Branch: refs/heads/branch-20.09
  Home:   https://github.com/ovn-org/ovn
  Commit: 4f366b75a6e4c5d9d00b29751a93b7d0aa7f6398
      https://github.com/ovn-org/ovn/commit/4f366b75a6e4c5d9d00b29751a93b7d0aa7f6398
  Author: Renat Nurgaliyev <impleman at gmail.com>
  Date:   2020-11-12 (Thu, 12 Nov 2020)

  Changed paths:
    M northd/ovn-northd.c

  Log Message:
  -----------
  northd: Don't poll ovsdb before the connection is fully established

Set initial SB and NB DBs probe interval to 0 to avoid connection
flapping.

Before configured in northd_probe_interval value is actually applied
to southbound and northbound database connections, both connections
must be fully established, otherwise ovnnb_db_run() will return
without retrieving configuration data from northbound DB. In cases
when southbound database is big enough, default interval of 5 seconds
will kill and retry the connection before it is fully established, no
matter what is set in northd_probe_interval. Client reconnect will
cause even more load to ovsdb-server and cause cascade effect, so
northd can never stabilise. We have more than 2000 ports in our lab,
and northd could not start before this patch, holding at 100% CPU
utilisation both itself and ovsdb-server.

After connections are established, any value in northd_probe_interval,
or default DEFAULT_PROBE_INTERVAL_MSEC is applied correctly.

Signed-off-by: Renat Nurgaliyev <impleman at gmail.com>
Signed-off-by: Numan Siddique <numans at ovn.org>

(cherry-picked from master commit 1e59feea933610b28fd4442243162ce35595cfee)




More information about the git mailing list