<div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Jan 10, 2014 at 1:32 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>
<br>
On Fri, Jan 10, 2014 at 12:54 PM, Sergey Skripnick <<a href="mailto:sskripnick@mirantis.com" target="_blank">sskripnick@mirantis.com</a>> wrote:<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">
<br>
I appreciate that we want to fix the ssh client. I'm not certain that writing our own is the >>>best >answer.<br>
<br>
</blockquote>
<br>
<br>
<br>
I was supposed to fix oslo.processutils.ssh with this class, but it may<br>
<br>
be fixed without it, not big deal.<br>
<br>
<br>
<br>
<br>
<br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
<br>
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?<br>
<br>
<br>
<br>
</blockquote>
<br>
<br>
<br>
Fabric is too much for just command execution on remote server. Spur seems like<br>
<br>
good choice for this.<br>
<br>
<br>
<br>
But I still don't understand: why do we need oslo.processutils.execute? We can use<br>
<br>
subprocess module. Why do we need oslo.processutils.ssh_execute? We can use paramiko<br>
<br>
instead.<br>
</blockquote>
<br>
Well, as you've shown, having a wrapper around subprocess to deal with the I/O properly is useful, >especially commands that produce a lot of it. :-)<br>
<br>
<br>
As far as ssh_execute goes, I don't know the origin but I imagine the author didn't know about >paramiko.<br>
<br>
<br>
Doug<br>
</blockquote>
<br>
<br></div>
ssh_execute is using paramiko :)</blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">See, this is what I get for not looking at git blame. :-)</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"><div class="HOEnZb"><div class="h5">
<br>
<br>
<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>