On Wed, Jan 8, 2014 at 2:31 PM, Sergey Skripnick <[email protected]>wrote:
> > > >> >> On Wed, Jan 8, 2014 at 10:43 AM, Eric Windisch <[email protected]> >>> wrote: >>> >> >> >>>> >>>> >>>> >>>>> >>>>> >>>>> About spur: spur is looks ok, but it a bit complicated inside (it uses >>>>> >>>>> separate threads for non-blocking stdin/stderr reading [1]) and I don't >>>>> >>>>> know how it would work with eventlet. >>>>> >>>> >>>> That does sound like it might cause issues. What would we need to do to >>>> test it? >>>> >>>> >>>>>>> >>>> >>>> >>> 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. >>> >>> >> Thanks for that analysis Eric. >> >> Is there any reason for us to prefer one approach over the other, then? >> >> Doug >> > > So, there is only one reason left -- oslo lib is more simple and > lightweight > (not using threads). Anyway this class is used by stackforge/rally and > may be used by other projects instead of buggy oslo.processutils.ssh. I appreciate that we want to fix the ssh client. I'm not certain that writing our own is the best answer. 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? Doug > > > > > -- > Regards, > Sergey Skripnick > > > _______________________________________________ > OpenStack-dev mailing list > [email protected] > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev >
_______________________________________________ OpenStack-dev mailing list [email protected] http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
