<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 15, 2014 at 5:28 AM, Frittoli, Andrea (Cloud Services) <span dir="ltr"><<a href="mailto:frittoli@hp.com" target="_blank">frittoli@hp.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-GB" link="blue" vlink="purple"><div><p class="MsoNormal">Hi all,<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal">I’ve been working on a patch to ensure that all tempest tests are able to handle rate limiting errors with automatic retries [0].<u></u><u></u></p><p class="MsoNormal">API tests already implement retries, however scenario tests don’t, the main reason being that scenario tests rely on the official python bindings, most of which do not handle retries (with the exception of the swift client).<u></u><u></u></p>

<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">My approach so far has been to monkey-patch python-bindings in tempest to add the retry logic, however a few pointed out that the right place to implement the retries would be in the python bindings directly.<u></u><u></u></p>

<p class="MsoNormal">I filed a bug [1] about implementing retries in the various python bindings, and I’d like to get some feedback from the community.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">

Even if  we have enough consensus about [1], I expect it might be a while until all python bindings are updated. <u></u><u></u></p><p class="MsoNormal">In the meanwhile I think we could go ahead with [0], and then start removing the monkey-patch as soon as retries are implemented on the client side.</p>

</div></div></blockquote><div><br></div><div>Besides monkey patching being scary and dangerous, monkey patching the clients in tempest means that we are now gating the clients on if the monkey patch is still valid. So we can't change the clients in a way that would break the monkey patch.  Although we could put the monkey patching in a try/except clause as long as we don't exercise this in the gate.</div>

<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-GB" link="blue" vlink="purple"><div><p class="MsoNormal"><u></u><u></u></p><p class="MsoNormal">

<u></u> <u></u></p><p class="MsoNormal">andrea<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">[0] <a href="https://review.openstack.org/#/c/61978/" target="_blank">https://review.openstack.org/#/c/61978/</a><u></u><u></u></p>

<p class="MsoNormal">[1] <a href="https://bugs.launchpad.net/python-novaclient/+bug/1263069" target="_blank">https://bugs.launchpad.net/python-novaclient/+bug/1263069</a> <u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p>

<p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span>-- <u></u><u></u></span></p><p class="MsoNormal"><span>Andrea Frittoli<u></u><u></u></span></p><p class="MsoNormal"><span lang="EN-US">IaaS Systems Engineering Team<u></u><u></u></span></p>

<p class="MsoNormal"><span>HP Cloud Services<u></u><u></u></span></p></div></div><br>_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
<br></blockquote></div><br></div></div>