Mailing-List: contact deltacloud-dev-help@incubator.apache.org; run by ezmlm
Precedence: bulk
Reply-To: deltacloud-dev@incubator.apache.org
Received-SPF: pass (nike.apache.org: domain of mmorsi@redhat.com designates
 209.132.183.28 as permitted sender)
Message-ID: <4C5877C9.4010903@redhat.com>
Date: Tue, 03 Aug 2010 16:10:49 -0400
From: Mohammed Morsi <mmorsi@redhat.com>
User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US;
 rv:1.9.2.7) Gecko/20100720 Fedora/3.1.1-1.fc13 Lightning/1.0b2pre
 Thunderbird/3.1.1
MIME-Version: 1.0
To: David Lutterkort <lutter@redhat.com>
CC: deltacloud-dev@incubator.apache.org
Subject: Re: [PATCH] added ability to scp one or more files to a running
 instance
References: <1280445820-3483-1-git-send-email-mmorsi@redhat.com>
	 <1280445820-3483-3-git-send-email-mmorsi@redhat.com>
 <1280797376.20090.48.camel@avon.watzmann.net>
In-Reply-To: <1280797376.20090.48.camel@avon.watzmann.net>
Content-Type: text/plain; charset=ISO-8859-15; format=flowed
Content-Transfer-Encoding: 7bit

  On 08/02/2010 09:02 PM, David Lutterkort wrote:
> On Thu, 2010-07-29 at 19:23 -0400, Mohammed Morsi wrote:
>> diff --git a/server/lib/deltacloud/models/instance.rb b/server/lib/deltacloud/models/instance.rb
>> index c2c9ff4..e0cd7a6 100644
>> --- a/server/lib/deltacloud/models/instance.rb
>> +++ b/server/lib/deltacloud/models/instance.rb
>> @@ -17,6 +17,7 @@
>>   # under the License.
>>
>>   require 'net/ssh'
>> +require 'net/scp'
>>
>>   class Instance<  BaseModel
>>
>> @@ -53,4 +54,15 @@ class Instance<  BaseModel
>>       return output
>>     end
>>
>> +  def copy_files(files, destination, username='', opts={})
>> +    files = [files] unless files.is_a? Array
>> +    hostname = self.public_addresses.first
>> +    return "No hostname/IP address specified" unless hostname
>> +    Net::SCP.start(hostname, username || 'root', opts) do |session|
>> +      files.each { |f|
>> +        session.upload! f, destination
>> +      }
>> +    end
>> +  end
> Doens't that copy all files into the same destination ?
>

If destination is a directory, all files will be copied there.


>> +  operation :copy_files, :method =>  :post, :member =>  true do
>> +    description "Securely copy one or more files to an instance"
>> +    param :id,           :string, :required
>> +    param :files,        :string, :required # FIXME should be array of strings
> That explains the above - only one file per copy. Would be nice to allow
> a number of files with their contents.
>
> David

Since the core wui has jquery support I can easily add more file 
selection boxes. The only thing is, I'm not sure how to specify the 
'files' param to this operation, as being able to accept an array of 
string file names (no other operation params seem to take :array as the 
type). Is there an easy way to do this? Or does the rabbit module need 
to be expanded?

   -Mo