[ovs-dev] [PATCH 4/4] ovsdb: Make OVSDB backup sever read only

Andy Zhou azhou at ovn.org
Mon Aug 15 07:21:41 UTC 2016


On Sat, Aug 13, 2016 at 10:23 PM, Ben Pfaff <blp at ovn.org> wrote:

> On Wed, Aug 03, 2016 at 04:57:05PM -0700, Andy Zhou wrote:
> > When ovsdb-sever is running in the backup state, it would be nice to
> > make sure there is no un-intended changes to the backup database.
> >
> > This patch makes the ovsdb server only accepts 'read' transactions as
> > a backup server. When the server role is changed into an active server,
> > all existing client connections will be reset. After reconnect, all
> > clinet transactions will then be accepted.
> >
> > Signed-off-by: Andy Zhou <azhou at ovn.org>
>
> It's a little odd to add a manpage .so-include if only one manpage
> .so-includes it.  Is it likely that other manpages will include it
> later?
>
> Fair enough. I merged the content into the ovsdb-server man page.

> s/execept/except/ in the manpage.
>

Fixed.

>
> In jsonrpc_create_readonly_lock_error(), the error string is a little
> garbled, s/methodsno/methods not/.
>
> Fixed

> GCC 4.9.1 complains:
>
> ../ovsdb/execution.c: In function 'ovsdb_execute':
> ../ovsdb/execution.c:172:33: error: 'ro' may be used uninitialized in this
> function [-Werror=maybe-uninitialized]
>          if (!error && read_only && !ro) {
>                                  ^
> ../ovsdb/execution.c:173:19: error: 'op_name' may be used uninitialized in
> this function [-Werror=maybe-uninitialized]
>              error = ovsdb_error("not allowed",
>                    ^
> cc1: all warnings being treated as errors
>
> Sorry that I missed them. Fixed.

If you fix those things then:
> Acked-by: Ben Pfaff <blp at ovn.org>
>
> Thanks for the review. Pushed the series to master.

> Thanks!
>
> Ben
>



More information about the dev mailing list