Change in vdsm[ovirt-3.4]: introducing capability to stream data to image

ybronhei at redhat.com ybronhei at redhat.com
Tue Feb 18 09:02:58 UTC 2014


Yaniv Bronhaim has submitted this change and it was merged.

Change subject: introducing capability to stream data to image
......................................................................


introducing capability to stream data to image

This patch introduces the a new capabaility to vdsm
which allows to upload using streaming content to vdsm
images.

Previously we sent ovf data using XMLRPC (UpdateVM verb),
which limits the size of the data, having to encode the
payload into the xml, and make it hard and inefficient
to upload lot of data and store it on some image.

This patch adds the capabillity to stream data to image,
allowing efficient upload of data in any size and format
and storing it directly on an
image. As the  XML-RPC spec doesn't support streaming and
to avoid requiring another port by using dedicated http
server, in this patch we use the existing xmlrpc server to
handle upload requests.

General upload information:
-----------------------------------------------------------
PUT requests arriving to the server with content type of
application/octet-stream to default paths that we use
today for request handling ('/', '/RPC2') will be treated
as upload requests.
The upload itself is being executed within a task, that's
needed to indicate that there's an operation executed by
the host.

Change-Id: I768b84799ed9fb2769c6d4240519d036f8988b99
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1037500
Signed-off-by: Liron Aravot <laravot at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/23281
Reviewed-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
Reviewed-on: http://gerrit.ovirt.org/24597
Reviewed-by: Yaniv Bronhaim <ybronhei at redhat.com>
---
M vdsm/API.py
M vdsm/BindingXMLRPC.py
M vdsm/storage/hsm.py
M vdsm/storage/image.py
M vdsm/storage/imageSharing.py
M vdsm/storage/sp.py
6 files changed, 211 insertions(+), 3 deletions(-)

Approvals:
  Yaniv Bronhaim: Looks good to me, approved
  Liron Ar: Verified



-- 
To view, visit http://gerrit.ovirt.org/24597
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I768b84799ed9fb2769c6d4240519d036f8988b99
Gerrit-PatchSet: 4
Gerrit-Project: vdsm
Gerrit-Branch: ovirt-3.4
Gerrit-Owner: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Federico Simoncelli <fsimonce at redhat.com>
Gerrit-Reviewer: Liron Ar <laravot at redhat.com>
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybronhei at redhat.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list