Numan, Thanks! Ted On Mar 20, 2019, at 11:56 PM, Numan Siddique <[email protected]<mailto:[email protected]>> wrote:
On Thu, Mar 21, 2019 at 1:22 AM Ted Elhourani <[email protected]<mailto:[email protected]>> wrote: Numan, This patch is currently at v5: https://patchwork.ozlabs.org/patch/1031257/ [patchwork.ozlabs.org]<https://urldefense.proofpoint.com/v2/url?u=https-3A__patchwork.ozlabs.org_patch_1031257_&d=DwMFaQ&c=s883GpUCOChKOHiocYtGcg&r=mB3ItZGjEYI9nn4mb7sBHd5RJ9XbfVZnYVRmdST65Cg&m=a0u1glq6Edh4HNoYLHFq3x-HDY_GlGyuV9O4Et81o8Y&s=FdFX0U-PEAOk5bZQrp-uGb-AopDVMpYddYzQIi4twIA&e=> 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]<mailto:[email protected]>> wrote: On Thu, Dec 27, 2018 at 2:54 AM Ted Elhourani <[email protected]<mailto:[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]<mailto:[email protected]>> Date: Tuesday, October 16, 2018 at 11:15 AM To: Numan Siddique <[email protected]<mailto:[email protected]>> Cc: ovs-discuss <[email protected]<mailto:[email protected]>> Subject: Re: Python: connect to leader in cluster Hi Numan, Thanks for letting me know. Ted From: Numan Siddique <[email protected]<mailto:[email protected]>> Date: Tuesday, October 16, 2018 at 11:03 AM To: Ted Elhourani <[email protected]<mailto:[email protected]>> Cc: ovs-discuss <[email protected]<mailto:[email protected]>> Subject: Re: Python: connect to leader in cluster On Tue, Oct 16, 2018 at 4:25 AM Ted Elhourani <[email protected]<mailto:[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]<mailto:[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
