[ovs-dev] [PATCH v2] ovsdb-tool: add db-set-election-timer command for clustered databases

Ben Pfaff blp at ovn.org
Thu May 27 20:25:25 UTC 2021


On Tue, May 25, 2021 at 11:53:57AM -0500, Dan Williams wrote:
> On Mon, 2021-05-24 at 14:52 -0700, Ben Pfaff wrote:
> > On Mon, May 24, 2021 at 02:59:58PM -0500, Dan Williams wrote:
> > > On Mon, 2021-05-24 at 10:49 -0700, Ben Pfaff wrote:
> > > > On Fri, May 21, 2021 at 02:25:52PM -0500, Dan Williams wrote:
> > > > > After creating the new cluster database write a raft entry that
> > > > > sets the desired election timer. This allows CMSes to set the
> > > > > election timer at cluster start and avoid an error-prone
> > > > > election timer modification process after the cluster is up.
> > > > > 
> > > > > Reported-at: https://bugzilla.redhat.com/1831778
> > > > > 
> > > > > Signed-off-by: Dan Williams <dcbw at redhat.com>
> > > > > ---
> > > > > v2:
> > > > > - Address Ben's comments; add --help and manpage docs
> > > > > - Write raft record directly instead of using private raft.c
> > > > > functions
> > > > 
> > > > Hmm, I don't think this change was what I asked for.  I asked for
> > > > a
> > > > new
> > > > option, like "--election-timer=TIMEOUT create-cluster DB CONTENTS
> > > > LOCAL".  Then, if we add some other new option later, we can type
> > > > "--foobar-option=QUUXWAG create-cluster DB CONTENTS LOCAL", and
> > > > so
> > > > on.
> > > > This adds a new command, which is quite different.
> > > 
> > > Aha, then I completely misunderstood you. Will do v3.
> > 
> > Sorry about that!  Thanks for humoring me.
> > 
> 
> Sent v3; I hope this is what you were actually thinking of.

Yes, that's what I meant!  Thanks.


More information about the dev mailing list