<div dir="ltr"><div class="gmail_default" style="font-family:arial,helvetica,sans-serif"><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 19, 2017 at 2:05 PM, Martin Sivak <span dir="ltr"><<a href="mailto:msivak@redhat.com" target="_blank">msivak@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">>> So I think that we still have to fix it somehow.<br>
>> Are we really sure that nr_retries=2 and _timeout=20 are really the magic numbers that works on every conditions?<br>
><br>
><br>
> No, it should be tested on HE environment and it depends on your usage.<br>
<br>
What does happen when only timeout is specified and the connection<br>
fails after the command is sent? What are the defaults in that case?<br></blockquote><div><br><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">So, there are not magic numbers to fit all, here's the description</div> <div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">of those parameters:<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">nr_retries<br> - number of reconnection retries<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> - if not specified than defualt is 1<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">_timeout<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> - it's maximum time to wait for reply of a command/veb if client is connected<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"> - this does not affect reconnection any way, meaning the client could reconnect for example for 10 minutes (using high enough nr_retries value) and yet this timeout may not be reached<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">So here are 2 suggestions:<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">1. Set nr_retries=0 and client will behave the same way as in 4.1 (meaning no reconnection performed)<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">2. Set nr_retries to high enough number (for example 100 000) and hope that this number of retries is enough for host being deployed using host deploy. I know that setting this number is tricky for HE, because the host deploy can take really different amount of time, but there's no exact way how to define exact timeout of single reconnection as it depends to the failure during attempt to connect.<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">AFAIK no other functionality was mentioned in [1], so nothing else is implemented. If there are other requirement for reconnection functionality, then let's open a new RFE for that and discuss it<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">Thanks<br><br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default">Martin<br></div><div style="font-family:arial,helvetica,sans-serif;display:inline" class="gmail_default"><br>[1] <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1376843">https://bugzilla.redhat.com/show_bug.cgi?id=1376843</a><br><br></div></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
Martin<br>
<br>
On Tue, Dec 19, 2017 at 1:55 PM, Irit Goihman <<a href="mailto:igoihman@redhat.com">igoihman@redhat.com</a>> wrote:<br>
><br>
><br>
><br>
> On Tue, Dec 19, 2017 at 2:51 PM, Simone Tiraboschi <<a href="mailto:stirabos@redhat.com">stirabos@redhat.com</a>> wrote:<br>
>><br>
>><br>
>><br>
>> On Tue, Dec 19, 2017 at 12:56 PM, Martin Perina <<a href="mailto:mperina@redhat.com">mperina@redhat.com</a>> wrote:<br>
>>><br>
>>> As Irit mentioned the provided reproduction steps are wrong (misuse of the code) and she posted correct example showing that jsonrpc code works as expected. So Martin/Simone are you using somewhere in HE code the original example that is misusing the client?<br>
>><br>
>><br>
>> According to<br>
>> <a href="https://bugzilla.redhat.com/show_bug.cgi?id=1527155#c9" rel="noreferrer" target="_blank">https://bugzilla.redhat.com/<wbr>show_bug.cgi?id=1527155#c9</a><br>
>> It works in Irit example, at least on that host with that load and timings, setting nr_retries=2 and _timeout=20<br>
>><br>
>> While we have _timeout=5 and no custom nr_retries<br>
>> <a href="https://github.com/oVirt/ovirt-hosted-engine-ha/blob/master/ovirt_hosted_engine_ha/lib/util.py#L417" rel="noreferrer" target="_blank">https://github.com/oVirt/<wbr>ovirt-hosted-engine-ha/blob/<wbr>master/ovirt_hosted_engine_ha/<wbr>lib/util.py#L417</a><br>
>><br>
>> So I think that we still have to fix it somehow.<br>
>> Are we really sure that nr_retries=2 and _timeout=20 are really the magic numbers that works on every conditions?<br>
><br>
><br>
> No, it should be tested on HE environment and it depends on your usage.<br>
>><br>
>><br>
>>><br>
>>><br>
>>> Thanks<br>
>>><br>
>>> Martin<br>
>>><br>
>>><br>
>>> On Tue, Dec 19, 2017 at 12:53 PM, Oved Ourfali <<a href="mailto:oourfali@redhat.com">oourfali@redhat.com</a>> wrote:<br>
>>>><br>
>>>> From the latest comment it doesn't seem like a blocker to me.<br>
>>>> Martin S. - your thoughts?<br>
>>>><br>
>>>> On Tue, Dec 19, 2017 at 1:48 PM, Sandro Bonazzola <<a href="mailto:sbonazzo@redhat.com">sbonazzo@redhat.com</a>> wrote:<br>
>>>>><br>
>>>>> We have a proposed blocker for the release:<br>
>>>>> 1527155InfravdsmBindings-<wbr>APIigoihman@redhat.<wbr>comNEWjsonrpc reconnect logic does not work and gets stuckurgentunspecifiedovirt-4.<wbr>2.004:30:30<br>
>>>>><br>
>>>>> Please review and either approve the blcoker or postpone to 4.2.1.<br>
>>>>> Thanks,<br>
>>>>><br>
>>>>><br>
>>>>> --<br>
>>>>><br>
>>>>> SANDRO BONAZZOLA<br>
>>>>><br>
>>>>> ASSOCIATE MANAGER, SOFTWARE ENGINEERING, EMEA ENG VIRTUALIZATION R&D<br>
>>>>><br>
>>>>> Red Hat EMEA<br>
>>>>><br>
>>>>> TRIED. TESTED. TRUSTED.<br>
>>>>><br>
>>>>><br>
>>>>> ______________________________<wbr>_________________<br>
>>>>> Devel mailing list<br>
>>>>> <a href="mailto:Devel@ovirt.org">Devel@ovirt.org</a><br>
>>>>> <a href="http://lists.ovirt.org/mailman/listinfo/devel" rel="noreferrer" target="_blank">http://lists.ovirt.org/<wbr>mailman/listinfo/devel</a><br>
>>>><br>
>>>><br>
>>><br>
>>><br>
<span class="gmail-HOEnZb"><font color="#888888">>>><br>
>>> --<br>
>>> Martin Perina<br>
>>> Associate Manager, Software Engineering<br>
>>> Red Hat Czech s.r.o.<br>
>><br>
>><br>
><br>
><br>
><br>
> --<br>
><br>
> IRIT GOIHMAN<br>
><br>
> SOFTWARE ENGINEER<br>
><br>
> EMEA VIRTUALIZATION R&D<br>
><br>
> Red Hat EMEA<br>
><br>
> TRIED. TESTED. TRUSTED.<br>
> @redhatnews Red Hat Red Hat<br>
</font></span></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature"><div dir="ltr"><font size="1">Martin Perina<br>Associate Manager, Software Engineering<br>Red Hat Czech s.r.o.<br></font></div></div>
</div></div>