9:31 p.m.
Yes the patches were applied though I forgot to restart the server so
they didn't take effect. I restarted it and it works fine now! Thanks!
On 09/04/2014 11:58 PM, Crístian Viana wrote:
Hi Hongliang. Thanks for testing this patchset!
I don't understand why this isn't working for you. I just executed the
same command here on my laptop and it worked fine. Take a look at my
commands:
$ curl -k -u vianac:<password> -H "Content-Type: application/json" -H
"Accept: application/json"
https://127.0.0.1:8001/storagepools/default/storagevolumes -X POST -d
'{"name": "CorePure64-5.3.iso", "url":
"http://distro.ibiblio.org/tinycorelinux/5.x/x86_64/release/CorePure64-5.3.iso"}'
{
"status":"running",
"message":"OK",
"id":"2",
"target_uri":""
}
$ curl -k -u vianac:<password> -H "Content-Type: application/json" -H
"Accept: application/json" https://127.0.0.1:8001/tasks/2
{
"status":"running",
"message":"4194304/10772480",
"id":"2"
}
$ curl -k -u vianac:<password> -H "Content-Type: application/json" -H
"Accept: application/json" https://127.0.0.1:8001/tasks/2
{
"status":"running",
"message":"5242880/10772480",
"id":"2"
}
And there was no error stack trace on the console.
Are you sure you have these 3 patches applied? Could you please check
if the function "_create_volume_with_url" is defined at
src/kimchi/model/storagevolumes.py, line 129? The exception you
mentioned is supposed to be raised when that function doesn't exist.
On 04-09-2014 12:40, Hongliang Wang wrote:
> Hi Cristian, seems something wrong when I send a request to create
> volume from remote URL.
>
> Remote Address:9.123.141.150:8001
> Request URL:https://9.123.141.150:8001/storagepools/ISO/storagevolumes
> Request Method:POST
> Status Code:400 Bad Request
>
> Request Payload:
> {
> name: "CorePure64-5.3.iso"
> url:
>
"http://distro.ibiblio.org/tinycorelinux/5.x/x86_64/release/CorePure64-5.3.iso"
> }
>
> The response:
>
> {
> "reason":"KCHVOL0019E: Creating volume from url is not
supported",
> "code":"400 Bad Request"
> }
>
> On 09/04/2014 09:46 PM, Crístian Viana wrote:
>> Changelog from previous version (v1):
>>
>> - Remove the text "downloading" from the download task status message.
>> Now it only contains the data: <downloaded size>/<total size>.
>>
>> Crístian Viana (3):
>> storagevolume: Download remote images to a storage pool
>> storagevolume: Check storage pool before adding a volume
>> storagevolume: Add download progress to task
>>
>> docs/API.md | 1 +
>> src/kimchi/mockmodel.py | 39 ++++++++++++++++++++++-----
>> src/kimchi/model/storagevolumes.py | 54
>> +++++++++++++++++++++++++++++++++++---
>> tests/test_model.py | 24 +++++++++++++++++
>> tests/test_rest.py | 14 +++++++++-
>> 5 files changed, 120 insertions(+), 12 deletions(-)
>>
>