Hi, just wanted to send a follow-up and a test result.

I got everything up and running (yay!). So I set up a server and ran a sandbox test using ovn with traditional C-based ovn-northd and then ran the same test using ovn with DDlog-based ovn-northd.

The test script can be found here: https://gist.github.com/putnopvut/3df7156b6d44b81d75a598262c5e959b

The test sets up a network with 159 logical switches all connected to a single logical router. Each logical switch has 92 logical switch ports. For every 2 logical switch ports we add, we create a new port group, and ACLs are created for each port group. The important part of this test is that the final call to ovn-nbctl in each iteration of the loop uses --wait=sb. In other words, each new logical switch port we add has to wait for ovn-northd to process it completely before we can start adding the next logical switch port.

With traditional C-based ovn-northd:
[root@wsfd-netdev67 tutorial]# time ./scale-daemon-new-pg.sh

real    128m59.280s
user    1m27.601s
sys     2m0.986s

With DDlog-based ovn-northd:
[root@wsfd-netdev67 tutorial]# time ./scale-daemon-new-pg.sh

real    33m40.586s
user    1m26.976s
sys     1m59.784s

As you can see, the test takes about a quarter of the time to complete when running with DDlog. So far, it seems like a good improvement.

As a point of reference, if I remove the --wait=sb from the test, then the result is:

[root@wsfd-netdev67 tutorial]# time ./scale-daemon-new-pg.sh

real    3m59.696s
user    1m18.937s
sys     1m44.669s

This test was run with DDlog-based ovn-northd, but the result is nearly identical with C-based ovn-northd since the test no longer is bound by ovn-northd's performance.

I'm a bit surprised that with DDlog, adding --wait=sb to each iteration causes such a large difference in the overall test time (~4 minutes up to ~34 minutes).

My next step is going to be to plot the time of each individual iteration during the test. I'll share those results when I have them ready. I don't want to speculate on anything until I have that data to share.

Thanks,
Mark Michelson

On 7/15/19 12:34 PM, Leonid Ryzhyk wrote:
Hi Mark,

Thanks for the feedback. We will add the `--version` flag.

Yes, `stack install` merely copies the two DDlog executable to `~/.local/bin/` (or a custom path specified using `--local-bin-path <custom_path>`).

Also, just want to mention that we maintain up-to-date binary releases of DDlog at `https://github.com/vmware/differential-datalog/releases`, so most users should not need to install the Haskell tool stack and compile DDlog. When using binary releases, all you have to do is add the `ddlog/bin` directory to `$PATH`.

Leonid

On Mon, Jul 15, 2019 at 6:30 AM Mark Michelson <[email protected] <mailto:[email protected]>> wrote:

    I did install DDlog originally back in early December last year. So
    there are probably remnants of that still present. However, I did
    perform a `stack install` using an updated pull of master on Friday. So
    I guess the `stack install` didn't get rid of the old installation?

    Also, there's no `ddlog --version` or anything similar to see what
    version of DDlog is installed. That could be a nice feature to have in
    the near future.

    On 7/12/19 12:40 PM, Leonid Ryzhyk wrote:
     > Thanks for trying it out!  Sounds like you have a very old
    version of
     > ddlog. If you install from source, please make sure that you run
    `stack
     > install` and that there is no other version off ddlog in your
    path other
     > than the one created by `stack install`.
     >
     > Leonid
     >
     > On Fri, Jul 12, 2019, 9:05 AM Mark Michelson <[email protected]
    <mailto:[email protected]>
     > <mailto:[email protected] <mailto:[email protected]>>> wrote:
     >
     >     On 7/12/19 3:29 AM, Leonid Ryzhyk wrote:
     >      > Dear OVN developers,
     >      >
     >      > This is a brief update on the state of the DDlog port of
    ovn-northd.
     >      >
     >      > We completed the initial implementation of ovn-northd in DDlog
     >     few months
     >      > ago.  Justin kindly
     >      > helped to integrate it with OVN, so that it can be used as
    a drop-in
     >      > replacement for the C
>      > version (and passes all the tests in the OVN test suite). The DDlog
     >      > implementation does
     >      > not have any of the new features/improvements added in
    April 2019
     >     or later.
     >      >
     >      > ## Repository
     >      >
     >      > The code is in the `ddlog-dev` branch of the `ovn-org/ovn`
     >     repository:
     >      > https://github.com/ovn-org/ovn/tree/ddlog-dev
     >
     >     Hi Leonid,
     >
     >     I ran into an issue when attempting to build ovn-northd. I
    successfully
     >     installed DDLog, but then I encountered this issue when
    building OVN:
     >
     >     ddlog -i ovn/northd/ovn_northd.dl -L
     >     /home/putnopvut/differential-datalog/lib
     >     ddlog: Failed to parse input file: "ovn/northd/ovn_northd.dl"
    (line 94,
     >     column 5):
     >     unexpected "&"
     >     expecting "not", variable name, relation name, "var",
    expression or "."
     >
     >     The line in question looks like this:
     >
     >           &SwitchPort(.lsp = lsp, .sw = &sw),
     >
     >     Any idea what's gone wrong here?
     >
     >     Thanks,
     >
     >     Mark Michelson
     >
     >
     >
     >      >
     >      > ## Documentation
     >      >
     >      > Building and using ovn-northd-ddlog:
     >      >
     >
    https://github.com/ovn-org/ovn/blob/ddlog-dev/ovn/northd/docs/design.md
     >      >
     >      > Debugging ovn-northd-ddlog:
     >      >
     >
    https://github.com/ovn-org/ovn/blob/ddlog-dev/ovn/northd/docs/debugging.md
     >      >
     >      > ## Preliminary performance results
     >      >
     >      > Han Zhou kindly tested ovn-northd-ddlog with his OVN scale
    test
     >     and even
     >      > found a nasty
     >      > performance bug in the process (thanks, Han!).  He reports
    that DDlog
     >      > speeds up the test
     >      > by almost a factor of 10:
     >      >
     >      > - ddlog version: 7:39min
     >      > - C version: 67:47min
     >      >
     >      > This is great, and in fact profiling shows that there is still
     >     plenty of
     >      > space for
     >      > improvement.  He also reports a 10+ times increase in memory
     >     footprint:
     >      >
     >      > - ddlog version: 1944696KB
     >      > - C version: 147984KB
     >      >
     >      > Again, we are working on a number of optimizations, which
    should
     >     reduce
     >      > this overhead; although it will never be as low as C,
    since DDlog
     >      > fundamentally
     >      > needs to cache more state to enable fast incremental
    computation.
     >      >
     >      > Han also used DDlog's record&replay feature to capture all
    northd
     >      > transactions
     >      > performed by the scale test in a format that can be replayed
     >     against the
     >      > standalone DDlog executable without having to reproduce Han's
     >     OpenStack
     >      > setup.
     >      > The replay file is here: http://ryzhyk.net/replay.tgz
     >      >
     >      > Instructions for replaying this script:
     >      >
     >
    
https://github.com/ovn-org/ovn/blob/ddlog-dev/ovn/northd/docs/debugging.md#record-and-replay-ddlog-execution
     >      > The script will run for a few
     >      > minutes and finally print some profiling information,
    including the
     >      > breakdown of
     >      > DDlog's CPU and memory usage.
     >      >
     >      > ## Next steps
     >      >
     >      > We seek help from the OVN community in maintaining
     >     ovn-northd-ddlog.  The
     >      > first
     >      > step is to start porting new OVN features introduced in
    the last
     >     few months
     >      > to
     >      > DDlog.
     >      >
     >      > Leonid
     >
     >     Hi Leonid,
     >
     >     I attempted
     >
     >      > _______________________________________________
     >      > dev mailing list
     >      > [email protected] <mailto:[email protected]>
    <mailto:[email protected] <mailto:[email protected]>>
     >      > https://mail.openvswitch.org/mailman/listinfo/ovs-dev
     >      >
     >


_______________________________________________
dev mailing list
[email protected]
https://mail.openvswitch.org/mailman/listinfo/ovs-dev

Reply via email to