[ovs-git] [openvswitch/ovs] e15ca9: test-ovsdb: Log steps in idl test.

Ben Pfaff noreply at github.com
Thu Jan 21 23:53:14 UTC 2021


  Branch: refs/heads/branch-2.15
  Home:   https://github.com/openvswitch/ovs
  Commit: e15ca9053a75625c8cb891c4ecbf6d21aef9e544
      https://github.com/openvswitch/ovs/commit/e15ca9053a75625c8cb891c4ecbf6d21aef9e544
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2021-01-21 (Thu, 21 Jan 2021)

  Changed paths:
    M tests/test-ovsdb.c

  Log Message:
  -----------
  test-ovsdb: Log steps in idl test.

Until now, "test-ovsdb idl" has printed the steps that it goes through
to stdout.  This commit also makes it log the same information.  This
makes it easier to match up the steps with the rest of the log (in
particular with the jsonrpc logging).

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: d7a6f23ef5f2d0904979f8f8ee28ee3a69fed4db
      https://github.com/openvswitch/ovs/commit/d7a6f23ef5f2d0904979f8f8ee28ee3a69fed4db
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2021-01-21 (Thu, 21 Jan 2021)

  Changed paths:
    M lib/automake.mk
    A lib/ovsdb-cs.c
    A lib/ovsdb-cs.h
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-cs: New module that factors out code from ovsdb-idl.

This new module has a single direct user now.  In the future, it
will also be used by OVN.

Signed-off-by: Ben Pfaff <blp at ovn.org>
Acked-by: Ilya Maximets <i.maximets at ovn.org>


  Commit: 4fc82127cc0bcfccaaac20c50df5faf4a8f22a2c
      https://github.com/openvswitch/ovs/commit/4fc82127cc0bcfccaaac20c50df5faf4a8f22a2c
  Author: Ben Pfaff <blp at ovn.org>
  Date:   2021-01-21 (Thu, 21 Jan 2021)

  Changed paths:
    M lib/ovsdb-cs.c
    M lib/ovsdb-cs.h
    M lib/ovsdb-idl-provider.h
    M lib/ovsdb-idl.c

  Log Message:
  -----------
  ovsdb-idl: Break into two layers.

This change breaks the IDL into two layers: the IDL proper, whose
interface to its client is unchanged, and a low-level library called
the OVSDB "client synchronization" (CS) library.  There are two
reasons for this change.  First, the IDL is big and complicated and
I think that this change factors out some of that complication into
a simpler lower layer.  Second, the OVN northd implementation based
on DDlog can benefit from the client synchronization library even
though it would actually be made increasingly complicated by the IDL.

Signed-off-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/08c8ef73fe22...4fc82127cc0b


More information about the git mailing list