Bug 1037500

Summary: PRD34 - [RFE] add method local to donwloadImage / uploadImage
Product: Red Hat Enterprise Virtualization Manager Reporter: Sandro Bonazzola <sbonazzo>
Component: vdsmAssignee: Liron Aravot <laravot>
Status: CLOSED ERRATA QA Contact: Elad <ebenahar>
Severity: high Docs Contact:
Priority: high    
Version: 3.3.0CC: acanan, adahms, amureini, bazulay, dfediuck, eedri, fsimonce, gklein, iheim, knesenko, laravot, lpeer, movciari, scohen, tnisan, yeylon
Target Milestone: ---Keywords: FutureFeature
Target Release: 3.4.0   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: storage
Fixed In Version: 4.14.2-0.2.el6ev.x86_64 Doc Type: Enhancement
Doc Text:
This update adds a new action to the vdsClient command - downloadImageFromStream - which adds the ability to transfer the content of an image via a stream. This improves the efficiency of transferring data and makes it possible to transfer data of any size and format directly into an image.
Story Points: ---
Clone Of: Environment:
Last Closed: 2014-06-09 13:26:48 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Bug Depends On:    
Bug Blocks: 1078909, 1097708, 1142926    

Description Sandro Bonazzola 2013-12-03 09:46:23 UTC
Description of problem:
now that uploadVolume has been removed, downloadImage must be used in its place.
Actually for downloading an image from a local path it should be used method http with URL using file:// protocol.

It would be better having a method local requiring a file path instead.

Comment 1 Federico Simoncelli 2013-12-03 09:55:27 UTC
Technically it's not hard to implement, we could support something like:

 downloadImage({'method': 'local', 'path': '...'}, spUUID, sdUUID, imgUUID)

(probably based on dd)

Comment 2 Sandro Bonazzola 2013-12-03 12:16:34 UTC
raising severity since http + file:// seems not working (bug #1037584)

Comment 3 Sandro Bonazzola 2013-12-05 13:48:05 UTC
Re - targeting to 3.3.0 and proposing this as blocker, since it's blocking another bug, targeted to 3.3.0 and already acked as blocker.

Comment 4 Federico Simoncelli 2013-12-05 14:53:38 UTC
(In reply to Sandro Bonazzola from comment #3)
> Re - targeting to 3.3.0 and proposing this as blocker, since it's blocking
> another bug, targeted to 3.3.0 and already acked as blocker.

According to bug 1034777 comment 6 there's no need for this in 3.3.0. You can use uploadVolume, provided that the client connection doesn't timeout.

Comment 6 Ayal Baron 2013-12-24 08:21:45 UTC
This should be covered as part of the work for storing OVFs on any domain.

Comment 7 movciari 2014-01-13 11:54:59 UTC
how comes this is for 3.4, if it blocks  https://bugzilla.redhat.com/show_bug.cgi?id=1034777 which is targeted for 3.3?

Comment 8 Ayal Baron 2014-01-13 12:18:23 UTC
(In reply to movciari from comment #7)
> how comes this is for 3.4, if it blocks 
> https://bugzilla.redhat.com/show_bug.cgi?id=1034777 which is targeted for
> 3.3?

As you can see Bug 1034777 is ON_QA so clearly it is not blocked by this bug.

Comment 11 Sandro Bonazzola 2014-05-14 11:34:51 UTC
Hi, Liron, I can't access referenced patch 23197.
Where may I find the changes for using downloadImage verb for file:// protocol ?

Comment 17 errata-xmlrpc 2014-06-09 13:26:48 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory, and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

http://rhn.redhat.com/errata/RHBA-2014-0504.html