<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class="">
<div class="">Numan,</div>
<div class=""><br class="">
</div>
This patch is currently at v5:
<div class=""><br class="">
<div class=""><a href="https://patchwork.ozlabs.org/patch/1031257/" class="">https://patchwork.ozlabs.org/patch/1031257/</a></div>
<div class=""><br class="">
</div>
<div class="">Please let me know if there are additional comments that we need to address.</div>
<div class=""><br class="">
</div>
<div class="">Thanks,</div>
<div class="">Ted<br class="">
<div><br class="">
<blockquote type="cite" class="">
<div class="">On Dec 27, 2018, at 1:28 AM, Numan Siddique <<a href="mailto:nusiddiq@redhat.com" class="">nusiddiq@redhat.com</a>> wrote:</div>
<br class="Apple-interchange-newline">
<div class="">
<div dir="ltr" class="">
<div dir="ltr" class=""><br class="">
</div>
<br class="">
<div class="gmail_quote">
<div dir="ltr" class="">On Thu, Dec 27, 2018 at 2:54 AM Ted Elhourani <<a href="mailto:ted.elhourani@nutanix.com" class="">ted.elhourani@nutanix.com</a>> wrote:<br class="">
</div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div lang="EN-US" class="">
<div class="gmail-m_-5210034244606553256WordSection1">
<p class="MsoNormal">Numan,<u class=""></u><u class=""></u></p>
<p class="MsoNormal"><u class=""></u> <u class=""></u></p>
<p class="MsoNormal">I submitted a python idl patch for the leader-only option.<u class=""></u><u class=""></u></p>
<p class="MsoNormal"><u class=""></u> <u class=""></u></p>
<p class="MsoNormal"><a href="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=" target="_blank" class="">https://mail.openvswitch.org/pipermail/ovs-dev/2018-December/354605.html
[mail.openvswitch.org]</a><u class=""></u><u class=""></u></p>
<p class="MsoNormal"><u class=""></u> <u class=""></u></p>
<p class="MsoNormal">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.</p>
</div>
</div>
</blockquote>
<div class=""><br class="">
</div>
<div class="">Thanks. I will take a look at it.</div>
<div class=""><br class="">
</div>
<div class="">Numan</div>
<div class=""> </div>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<div lang="EN-US" class="">
<div class="gmail-m_-5210034244606553256WordSection1">
<p class="MsoNormal"><u class=""></u><u class=""></u></p>
<p class="MsoNormal"><u class=""></u> <u class=""></u></p>
<p class="MsoNormal">Thanks,<u class=""></u><u class=""></u></p>
<p class="MsoNormal">Ted<u class=""></u><u class=""></u></p>
<p class="MsoNormal"><u class=""></u> <u class=""></u></p>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(181,196,223);padding:3pt 0in 0in" class="">
<p class="MsoNormal"><b class=""><span style="font-size: 12pt;" class="">From: </span>
</b><span style="font-size: 12pt;" class="">Ted Elhourani <<a href="mailto:ted.elhourani@nutanix.com" target="_blank" class="">ted.elhourani@nutanix.com</a>><br class="">
<b class="">Date: </b>Tuesday, October 16, 2018 at 11:15 AM<br class="">
<b class="">To: </b>Numan Siddique <<a href="mailto:nusiddiq@redhat.com" target="_blank" class="">nusiddiq@redhat.com</a>><br class="">
<b class="">Cc: </b>ovs-discuss <<a href="mailto:ovs-discuss@openvswitch.org" target="_blank" class="">ovs-discuss@openvswitch.org</a>><br class="">
<b class="">Subject: </b>Re: Python: connect to leader in cluster<u class=""></u><u class=""></u></span></p>
</div>
<div class="">
<p class="MsoNormal"><u class=""></u> <u class=""></u></p>
</div>
<p class="MsoNormal">Hi Numan,<u class=""></u><u class=""></u></p>
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
<p class="MsoNormal">Thanks for letting me know.<u class=""></u><u class=""></u></p>
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
<p class="MsoNormal">Ted<u class=""></u><u class=""></u></p>
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(181,196,223);padding:3pt 0in 0in" class="">
<p class="MsoNormal"><b class=""><span style="font-size: 12pt;" class="">From: </span>
</b><span style="font-size: 12pt;" class="">Numan Siddique <<a href="mailto:nusiddiq@redhat.com" target="_blank" class="">nusiddiq@redhat.com</a>><br class="">
<b class="">Date: </b>Tuesday, October 16, 2018 at 11:03 AM<br class="">
<b class="">To: </b>Ted Elhourani <<a href="mailto:ted.elhourani@nutanix.com" target="_blank" class="">ted.elhourani@nutanix.com</a>><br class="">
<b class="">Cc: </b>ovs-discuss <<a href="mailto:ovs-discuss@openvswitch.org" target="_blank" class="">ovs-discuss@openvswitch.org</a>><br class="">
<b class="">Subject: </b>Re: Python: connect to leader in cluster</span><u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal" style="margin-bottom:12pt"> <u class=""></u><u class=""></u></p>
<div class="">
<div class="">
<p class="MsoNormal">On Tue, Oct 16, 2018 at 4:25 AM Ted Elhourani <<a href="mailto:ted.elhourani@nutanix.com" target="_blank" class="">ted.elhourani@nutanix.com</a>> wrote:<u class=""></u><u class=""></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin:5pt 0in 5pt 4.8pt" class="">
<div class="">
<div class="">
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">Hi Numan,</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">There was an earlier patch to allow the Python IDL to connect to one of multiple remotes.
</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">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?</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">Thanks,</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">Ted </span>
<u class=""></u><u class=""></u></p>
</div>
</div>
</blockquote>
<div class="">
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal">Hi Ted,<u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal">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.<u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal">Please feel free if you want to contribute to it.<u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal">Thanks<u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal">Numan<u class=""></u><u class=""></u></p>
</div>
<div class="">
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin:5pt 0in 5pt 4.8pt" class="">
<div class="">
<div class="">
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<b class=""><span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class=""> </span></b><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<b class=""><span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">From: Numan Siddique <<a href="mailto:nusiddiq@redhat.com" target="_blank" class="">nusiddiq@redhat.com</a>></span></b><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class=""> </span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">Python IDL implementation doesn't have the support to connect to the</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">cluster dbs. This patch adds this support. We are still missing the</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">support in python idl class to connect to the cluster master. That</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">support will be added in an upcoming patch.</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class=""> </span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">This patch is similar to the commit 8cf6bbb184 which added multiple remote</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal" style="margin-bottom:7.5pt;line-height:10.75pt;background:white;word-break:break-all">
<span style="font-size:10pt;font-family:Menlo;color:rgb(51,51,51)" class="">support in the C jsonrpc implementation.</span><u class=""></u><u class=""></u></p>
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
<p class="MsoNormal"> <u class=""></u><u class=""></u></p>
</div>
</div>
</blockquote>
</div>
</div>
</div>
</div>
</blockquote>
</div>
</div>
</div>
</blockquote>
</div>
<br class="">
</div>
</div>
</body>
</html>