On Thu, Mar 21, 2019 at 1:22 AM Ted Elhourani <[email protected]> wrote:
> Numan, > > This patch is currently at v5: > > https://patchwork.ozlabs.org/patch/1031257/ > > Please let me know if there are additional comments that we need to > address. > Hi Ted. I remember searching for your v5 patch sometime back. But looks like I didn't search for it properly. I reviewed the v5 of your patch now. Thanks Numan > > Thanks, > Ted > > On Dec 27, 2018, at 1:28 AM, Numan Siddique <[email protected]> wrote: > > > > On Thu, Dec 27, 2018 at 2:54 AM Ted Elhourani <[email protected]> > wrote: > >> Numan, >> >> >> >> I submitted a python idl patch for the leader-only option. >> >> >> >> https://mail.openvswitch.org/pipermail/ovs-dev/2018-December/354605.html >> [mail.openvswitch.org] >> <https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_pipermail_ovs-2Ddev_2018-2DDecember_354605.html&d=DwMFaQ&c=s883GpUCOChKOHiocYtGcg&r=mB3ItZGjEYI9nn4mb7sBHd5RJ9XbfVZnYVRmdST65Cg&m=BjJq6WBShxhl7iSyfqMY5ni4Od2oUB3ZPAai7jkDfQs&s=LrVzcGccPZ-ZtURFyt94Wz42eHRKM2Xxw4Rv6etGzaw&e=> >> >> >> >> The code that implements the state machine of the IDL can be simplified. >> As a first step toward enhancing the implementation, I have only extended >> the current implementation with the leader-only selection of a cluster >> node, I haven’t made changes to the existing code or the way it is >> implemented. >> > > Thanks. I will take a look at it. > > Numan > > >> >> >> Thanks, >> >> Ted >> >> >> >> *From: *Ted Elhourani <[email protected]> >> *Date: *Tuesday, October 16, 2018 at 11:15 AM >> *To: *Numan Siddique <[email protected]> >> *Cc: *ovs-discuss <[email protected]> >> *Subject: *Re: Python: connect to leader in cluster >> >> >> >> Hi Numan, >> >> >> >> Thanks for letting me know. >> >> >> >> Ted >> >> >> >> *From: *Numan Siddique <[email protected]> >> *Date: *Tuesday, October 16, 2018 at 11:03 AM >> *To: *Ted Elhourani <[email protected]> >> *Cc: *ovs-discuss <[email protected]> >> *Subject: *Re: Python: connect to leader in cluster >> >> >> >> >> >> On Tue, Oct 16, 2018 at 4:25 AM Ted Elhourani <[email protected]> >> wrote: >> >> Hi Numan, >> >> There was an earlier patch to allow the Python IDL to connect to one of >> multiple remotes. >> >> Is an upcoming patch imminent to give the Python IDL the option to >> connect only to (or become active only when) the leader in an HA cluster? >> >> Thanks, >> >> Ted >> >> >> >> Hi Ted, >> >> >> >> Yes. We do need a patch in python idl for the option to connect only to >> the leader. I may not be able to work on this at the moment. >> >> Please feel free if you want to contribute to it. >> >> >> >> Thanks >> >> Numan >> >> >> >> >> >> *From: Numan Siddique <[email protected] <[email protected]>>* >> >> >> >> Python IDL implementation doesn't have the support to connect to the >> >> cluster dbs. This patch adds this support. We are still missing the >> >> support in python idl class to connect to the cluster master. That >> >> support will be added in an upcoming patch. >> >> >> >> This patch is similar to the commit 8cf6bbb184 which added multiple remote >> >> support in the C jsonrpc implementation. >> >> >> >> >> >> >
_______________________________________________ discuss mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-discuss
