<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 9:34 AM, 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"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I'd like to explore whether the paramiko team will accept this code (or<br>
something like it). This seems like a perfect opportunity for us to contribute<br>
upstream.<br>
</blockquote>
<br>
+1<br>
<br>
The patch is not big and the code seems simple and reasonable enough<br>
to live within paramiko.<br>
<br>
Cheers,<br>
FF<br>
<br>
<br>
</blockquote>
<br></div>
I sent a pull request [0] but there is two things:<br>
<br>
 nobody know when (and if) it will be merged<br>
 it is still a bit low-level, unlike a patch in oslo<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Let's give the paramkio devs a little time to review it.</div><div class="gmail_default" style="font-size:small">
 <br></div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
About spur: spur is looks ok, but it a bit complicated inside (it uses<br>
separate threads for non-blocking stdin/stderr reading [1]) and I don't<br>
know how it would work with eventlet.<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">That does sound like it might cause issues. What would we need to do to test it?</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">
<br>
[0] <a href="https://github.com/paramiko/paramiko/pull/245" target="_blank">https://github.com/paramiko/<u></u>paramiko/pull/245</a><br>
[1] <a href="https://github.com/mwilliamson/spur.py/blob/master/spur/io.py#L22" target="_blank">https://github.com/<u></u>mwilliamson/spur.py/blob/<u></u>master/spur/io.py#L22</a><div class="HOEnZb"><div class="h5"><br>
<br>
-- <br>
Regards,<br>
Sergey Skripnick<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>