[ovs-git] [openvswitch/ovs] 977b53: ovsdb: Gracefully handle replication errors.

GitHub noreply at github.com
Mon Feb 13 20:27:19 UTC 2017


  Branch: refs/heads/branch-2.7
  Home:   https://github.com/openvswitch/ovs
  Commit: 977b53dce6d53fccb86894c8b9a63328a31f4d7e
      https://github.com/openvswitch/ovs/commit/977b53dce6d53fccb86894c8b9a63328a31f4d7e
  Author: Andy Zhou <azhou at ovn.org>
  Date:   2017-02-13 (Mon, 13 Feb 2017)

  Changed paths:
    M ovsdb/ovsdb-server.c

  Log Message:
  -----------
  ovsdb: Gracefully handle replication errors.

Sometimes replication session can fail mostly due to replication
configurations. i.e. replicating from a database with a different
version of the schema.

Currently, those errors are treated as fatal errors, and stops the
OVSDB server. A better way to handle those error may be to stop
only the replication session, and leave the OVSDB server up, so that
the replication can be restarted, may be with a different configuration,
at a later time.

Signed-off-by: Andy Zhou <azhou at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: b384f3027c8b2049d8fc45d98556cce4d6eac72c
      https://github.com/openvswitch/ovs/commit/b384f3027c8b2049d8fc45d98556cce4d6eac72c
  Author: Andy Zhou <azhou at ovn.org>
  Date:   2017-02-13 (Mon, 13 Feb 2017)

  Changed paths:
    M ovsdb/jsonrpc-server.c
    M ovsdb/jsonrpc-server.h
    M ovsdb/server.c
    M ovsdb/server.h

  Log Message:
  -----------
  ovsdb: Add OVSDB server per instance UUID.

Currently, there is no way for an OVSDB server to ID itself. This patch
adds a UUID field that is populated every time OVSDB server runs.
Later patch will make use this UUID to detect and stop and OVSDB server
from replicating itself.

Signed-off-by: Andy Zhou <azhou at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


  Commit: 0117ed0fcb18f75d445a34730adc1744f65c1ecb
      https://github.com/openvswitch/ovs/commit/0117ed0fcb18f75d445a34730adc1744f65c1ecb
  Author: Andy Zhou <azhou at ovn.org>
  Date:   2017-02-13 (Mon, 13 Feb 2017)

  Changed paths:
    M ovsdb/jsonrpc-server.c
    M ovsdb/ovsdb-server.1.in
    M ovsdb/ovsdb-server.c
    M ovsdb/replication.c
    M ovsdb/replication.h
    M tests/ovsdb-server.at

  Log Message:
  -----------
  ovsdb: Prevent OVSDB server from replicating itself.

Replication OVSDB server from itself is usually caused by configuration
errors. Such configuration errors can lead to OVSDB server data loss.
See "reported-at" for more details.

This patch adds logics that prevent OVSDB server from replicating
itself.

Reported-by: Guishuai Li <ligs at dtdream.com>
Reported-at: https://mail.openvswitch.org/pipermail/ovs-dev/2017-January/326963.html
Suggested-by: Ben Pfaff <blp at ovn.org>
Signed-off-by: Andy Zhou <azhou at ovn.org>
Acked-by: Ben Pfaff <blp at ovn.org>


Compare: https://github.com/openvswitch/ovs/compare/cd30b96b7943...0117ed0fcb18


More information about the git mailing list