Re: [ovs-dev] [PATCH v3 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-11 Thread aginwala aginwala
Thanks Ben: Please backport it to 2.10 and 2.9 On Thu, Oct 11, 2018 at 2:06 PM Ben Pfaff wrote: > On Wed, Oct 10, 2018 at 11:42:08AM +0530, Numan Siddique wrote: > > On Wed, Oct 10, 2018 at 3:42 AM aginwala wrote: > > > > > When starting OVN DBs in HA using pacemaker with ssl, we need to pass

Re: [ovs-dev] [PATCH 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-08 Thread aginwala aginwala
On Mon, Oct 8, 2018 at 2:17 PM Han Zhou wrote: > > > On Mon, Oct 8, 2018 at 11:55 AM aginwala aginwala > wrote: > > > > Yes, that's right. I will send out v2 in a bit with Han's ack. > > > > > > Regards, > > Aliasgar > > > > On Mon

Re: [ovs-dev] [PATCH 1/2] ovn-ctl: Allow passing ssl certs when starting OVN DBs in ssl mode.

2018-10-08 Thread aginwala aginwala
On Mon, Oct 8, 2018 at 10:47 AM Han Zhou wrote: > > > On Fri, Oct 5, 2018 at 6:48 PM aginwala aginwala > wrote: > > > > Thanks for the review Han. Please find the comments inline below: > > On Thu, Oct 4, 2018 at 9:58 AM Han Zhou wrote: > >> > &

Re: [ovs-dev] [PATCH 1/2] ovn-ctl: Allow passing ssl certs when starting OVN DBs in ssl mode.

2018-10-08 Thread aginwala aginwala
Sure. I will add ssl usage example with some brief in the ovn-ctl.8.xml and send v3 for this patch . Does that sound good? On Mon, Oct 8, 2018 at 5:32 PM Han Zhou wrote: > > > On Mon, Oct 8, 2018 at 4:55 PM aginwala aginwala > wrote: > > > > > > > > On M

Re: [ovs-dev] [PATCH 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-09 Thread aginwala aginwala
Thanks Numan for review. Just to update that V3 will only have changes for ovn-ctl as per Han's suggestion to add details in help section there. ocf script will remain unchanged. That is why I have added acked-by on this patch. :) On Mon, Oct 8, 2018 at 11:59 PM Numan Siddique wrote: > > > On

Re: [ovs-dev] [PATCH 1/2] ovn-ctl: Allow passing ssl certs when starting OVN DBs in ssl mode.

2018-10-09 Thread aginwala aginwala
Hi Han: Have added the man section for ssl in v3. PTAL. Thanks, On Mon, Oct 8, 2018 at 6:49 PM aginwala aginwala wrote: > Sure. I will add ssl usage example with some brief in the ovn-ctl.8.xml > and send v3 for this patch . Does that sound good? > > On Mon, Oct 8, 2018 at 5:32

Re: [ovs-dev] [PATCH 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-05 Thread aginwala aginwala
Thanks for the review Han. Please find the comments inline below: On Thu, Oct 4, 2018 at 10:16 AM Han Zhou wrote: > Thanks Ali, please see my comm > > On Fri, Sep 21, 2018 at 5:38 PM wrote: > > > > When starting OVN DBs in HA using pacemaker with ssl, we need to pass > ssl > > certs for

Re: [ovs-dev] [PATCH 1/2] ovn-ctl: Allow passing ssl certs when starting OVN DBs in ssl mode.

2018-10-05 Thread aginwala aginwala
Thanks for the review Han. Please find the comments inline below: On Thu, Oct 4, 2018 at 9:58 AM Han Zhou wrote: > Thanks Ali, please see my comments below > > On Fri, Sep 21, 2018 at 5:34 PM wrote: > > > > For OVN DBs to work with SSL in HA, we need to have capability to > > pass ssl certs

Re: [ovs-dev] [PATCH 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-08 Thread aginwala aginwala
Yes, that's right. I will send out v2 in a bit with Han's ack. Regards, Aliasgar On Mon, Oct 8, 2018 at 11:04 AM Ben Pfaff wrote: > On Mon, Oct 08, 2018 at 10:58:49AM -0700, Han Zhou wrote: > > On Fri, Oct 5, 2018 at 6:34 PM aginwala aginwala > wrote: > > > > >

Re: [ovs-dev] [PATCH v1] ovsdb-tool: Convert clustered db to standalone db.

2019-08-22 Thread aginwala aginwala
Thanks Aron. I addressed this warning and sent v2. Didn't see it in subsequent make as it appeared only first time so couldn't notice. On Thu, Aug 22, 2019 at 5:00 PM 0-day Robot wrote: > Bleep bloop. Greetings Aliasgar Ginwala, I am a robot and I have tried > out your patch. > Thanks for your

Re: [ovs-dev] [PATCH v2] ovsdb-tool: Convert clustered db to standalone db.

2019-08-22 Thread aginwala aginwala
Thanks for the review Han. On Thu, Aug 22, 2019 at 7:27 PM Han Zhou wrote: > Ali, thanks for the patch. Please see my comments below. > > On Thu, Aug 22, 2019 at 5:53 PM wrote: > > > > From: Aliasgar Ginwala > > > > Add support in ovsdb-tool for migrating clustered dbs to standalone dbs. > >