Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-10 Thread David Nalley
Prasanna, Sorry for taking so long to circle back to this - I'll commit this shortly. --David

Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread prasanna
>> This inherently looks ok, but Paramiko that you use in the above patch >> is licensed LGPLv2 - and needs to be removed[1]. Before we perpetuate >> its use, would you take a look at: >> >> http://bugs.cloudstack.org/browse/CS-14998 >> http://bugs.cloudstack.org/browse/CS-14999 >> http://bugs.clou

Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread David Nalley
> > Test client doesn't include the paramkio source code into its directory, the > migration tool(tools/migration) does. > Test client doesn't include it, but appears to make use of it. The migration tool (based on the page I linked to - assuming that's correct) says it was for 1.0->2.1 migratio

RE: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread Edison Su
> -Original Message- > From: David Nalley [mailto:da...@gnsa.us] > Sent: Tuesday, June 05, 2012 8:27 AM > To: cloudstack-dev@incubator.apache.org > Subject: Re: [PATCH] Adding support for file copy to marvin's > remoteSSHClient > > On Tue, Jun 5, 2012 a

Re: [PATCH] Adding support for file copy to marvin's remoteSSHClient

2012-06-05 Thread David Nalley
On Tue, Jun 5, 2012 at 6:38 AM, Prasanna Santhanam wrote: > > Signed-off-by: Prasanna Santhanam > --- >  tools/marvin/marvin/remoteSSHClient.py |   11 ++- >  1 file changed, 10 insertions(+), 1 deletion(-) > > diff --git a/tools/marvin/marvin/remoteSSHClient.py > b/tools/marvin/marvin/re