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 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-09 Thread Numan Siddique
On Tue, Oct 9, 2018 at 6:11 AM Han Zhou wrote: > >> > >> Giving a second thought, it seems there is still a problem. > >> > >> There should be two sets of SSL related parameters we should consider in > the active-standby scenario. > >> - One set of parameters is for the server side. For ipaddr2

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, Oct 8, 2018 at 11:04 AM Ben Pfaff wrote: > >> > >> On Mon,

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 Han Zhou
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, 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

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: > > > > > > Thanks for the review Han.

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 Ben Pfaff
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: > > > > 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 > >> > >>

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 Han Zhou
On Fri, Oct 5, 2018 at 6:34 PM aginwala aginwala wrote: > > 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

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 2/2] ovndb-servers.ocf: Add ssl support for managing OVN DB resources with pacemaker using LB VIP.

2018-10-04 Thread Han Zhou
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 starting standby DBs. Hence, we need this change. > > Signed-off-by: aginwala > --- > ovn/utilities/ovndb-servers.ocf | 74

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

2018-09-27 Thread Numan Siddique
On Sat, Sep 22, 2018 at 6:08 AM wrote: > When starting OVN DBs in HA using pacemaker with ssl, we need to pass ssl > certs for starting standby DBs. Hence, we need this change. > > Signed-off-by: aginwala > Hi Aliasgar, I will try this out and get back to you with any comments. Thanks