Re: [Kimchi-devel] [PATCH V7] Add nfs server and target UI in create storage pool
On 01/24/2014 10:13 AM, Aline Manera wrote:
On 01/23/2014 10:41 PM, Sheldon wrote:
> comment below：
>
>
> On 01/16/2014 10:03 AM, zhoumeina wrote:
>> + },
>> +
>> + getStorageServers: function(type, suc, err) {
>> + var url = kimchi.url + 'storageservers?target_type=' + type;
> according to
> [PATCHv9 03/10] Storage server: Update API.md
>
> target_type -> _target_type ？
Yes. =/
It passed away from my eyes.
Could you send a patch to fix it?
On mail-list this is the latest committed by Meina.
I think, it is fixed by yourself, when you test this patch. :-)
$ grep target_type ui/js/src/kimchi.api.js
var url = kimchi.url + 'storageservers?_target_type=' + type;
var url = kimchi.url + 'storageservers/' + server +
'/storagetargets?_target_type=' + type;
>
>> + kimchi.requestJSON({
>> + url : url,
>> + type : 'GET',
>> + contentType : 'application/json',
>> + dataType : 'json',
>> + success : suc,
>> + error : err
>> + });
>> + },
>> +
>> + getStorageTargets: function(server,type, suc, err) {
>> + var url = kimchi.url + 'storageservers/' + server +
>> '/storagetargets?target_type=' + type;
> ^
>> + kimchi.requestJSON({
>> + url : url,
>> + type : 'GET',
>> + contentType : 'application/json',
>> + timeout: 2000,
>> + dataType : 'json',
>> + success : suc,
>> + error : err
>> + });
>> }
>
>
--
Thanks and best regards!
Sheldon Feng(冯少合)<shaohef(a)linux.vnet.ibm.com>
IBM Linux Technology Center