[ovs-dev] [PATCH v3 2/5] ovsdb-idl.c: Allows retry even when using a single remote.

Ben Pfaff blp at ovn.org
Wed Aug 21 18:27:58 UTC 2019


On Wed, Aug 21, 2019 at 11:21:12AM -0700, Han Zhou wrote:
> On Wed, Aug 21, 2019 at 11:13 AM Ben Pfaff <blp at ovn.org> wrote:
> >
> > On Mon, Aug 19, 2019 at 09:29:57AM -0700, Han Zhou wrote:
> > > From: Han Zhou <hzhou8 at ebay.com>
> > >
> > > When clustered mode is used, the client needs to retry connecting
> > > to new servers when certain failures happen. Today it is allowed to
> > > retry new connection only if multiple remotes are used, which prevents
> > > using LB VIP with clustered nodes. This patch makes sure the retry
> > > logic works when using LB VIP: although same IP is used for retrying,
> > > the LB can actually redirect the connection to a new node.
> > >
> > > Signed-off-by: Han Zhou <hzhou8 at ebay.com>
> > > ---
> > > v2 -> v3: Minor fix for test case.
> >
> > I see that I commented on v1 of this patch by mistake, but the same
> > comment applies here: please update the documentation to explain the new
> > behavior.
> 
> Hi Ben,
> 
> Yes we agreed to update ovn-nb/sb documentation, but this patch belongs to
> OVS repo and the documentation would belong to OVN repo. So we will send
> update to documentation of ovn-nbctl/sbctl separately on OVN repo.

OK.

Do you want this patch backported to 2.12?


More information about the dev mailing list