Sandro Bonazzola has posted comments on this change.

Change subject: vdsm: use downloadImage instead of uploadVolume
......................................................................


Patch Set 2:

Traceback (most recent call last):
  File "/usr/share/vdsm/storage/task.py", line 857, in _run
    return fn(*args, **kargs)
  File "/usr/share/vdsm/storage/task.py", line 318, in run
    return self.cmd(*self.argslist, **self.argsdict)
  File "/usr/share/vdsm/storage/securable.py", line 68, in wrapper
    return f(self, *args, **kwargs)
  File "/usr/share/vdsm/storage/sp.py", line 1790, in downloadImage
    .download(methodArgs, sdUUID, imgUUID, volUUID)
  File "/usr/share/vdsm/storage/image.py", line 1172, in download
    vol.extend(imageSharing.getSize(methodArgs) / volume.BLOCK_SIZE)
  File "/usr/share/vdsm/storage/imageSharing.py", line 78, in getSize
    return getSizeImpl(methodArgs)
  File "/usr/share/vdsm/storage/imageSharing.py", line 44, in httpGetSize
    raise RuntimeError("Unable to determine image size")
 RuntimeError: Unable to determine image size

The command causing that error is:
 # /usr/bin/curl -q --silent --fail --show-error --head 
file:///var/tmp/tmpnhXPRe
 Content-Length: 21474836480
 Accept-ranges: bytes
 Last-Modified: Tue, 03 Dec 2013 11:20:45 GMT
 # echo $?
 0

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ieb645d2a6d5bab6e6012baa8ece0b8e52ba54a4c
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-hosted-engine-setup
Gerrit-Branch: master
Gerrit-Owner: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Adam Litke <[email protected]>
Gerrit-Reviewer: Alex Lourie <[email protected]>
Gerrit-Reviewer: Ayal Baron <[email protected]>
Gerrit-Reviewer: Doron Fediuck <[email protected]>
Gerrit-Reviewer: Eduardo <[email protected]>
Gerrit-Reviewer: Federico Simoncelli <[email protected]>
Gerrit-Reviewer: Greg Padgett <[email protected]>
Gerrit-Reviewer: Martin Sivák <[email protected]>
Gerrit-Reviewer: Sandro Bonazzola <[email protected]>
Gerrit-Reviewer: Yedidyah Bar David <[email protected]>
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to