<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 8, 2014 at 2:31 PM, Sergey Skripnick <span dir="ltr"><<a href="mailto:sskripnick@mirantis.com" target="_blank">sskripnick@mirantis.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On Wed, Jan 8, 2014 at 10:43 AM, Eric Windisch <<a href="mailto:ewindisch@docker.com" target="_blank">ewindisch@docker.com</a>> wrote:<br>
</blockquote>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>
About spur: spur is looks ok, but it a bit complicated inside (it uses<br>
<br>
separate threads for non-blocking stdin/stderr reading [1]) and I don't<br>
<br>
know how it would work with eventlet.<br>
</blockquote>
<br>
That does sound like it might cause issues. What would we need to do to test it?<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
</blockquote></blockquote></blockquote>
<br>
<br>
</blockquote>
<br>
Looking at the code, I don't expect it to be an issue. The monkey-patching will cause eventlet.spawn >>to be called for threading.Thread. The code looks eventlet-friendly enough on the surface. Error >>handing around file read/write could be affected, but it also looks fine.<br>

<br>
</blockquote>
<br>
Thanks for that analysis Eric.<br>
<br>
Is there any reason for us to prefer one approach over the other, then?<br>
<br>
Doug<br>
</blockquote>
<br></div>
So, there is only one reason left -- oslo lib is more simple and lightweight<br>
(not using threads). Anyway this class is used by stackforge/rally and<br>
may be used by other projects instead of buggy oslo.processutils.ssh.</blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">I appreciate that we want to fix the ssh client. I'm not certain that writing our own is the best answer. </div>
<div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">In his comments on your pull request, the paramiko author recommended looking at Fabric. I know that Fabric has a long history in production. Does it provide the required features?</div>
<div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Doug</div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="HOEnZb"><font color="#888888"><br>
<br>
<br>
<br>
-- <br>
Regards,<br>
Sergey Skripnick</font></span><div class="HOEnZb"><div class="h5"><br>
<br>
______________________________<u></u>_________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org" target="_blank">OpenStack-dev@lists.openstack.<u></u>org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/<u></u>cgi-bin/mailman/listinfo/<u></u>openstack-dev</a><br>
</div></div></blockquote></div><br></div></div>