[Engine-devel] move api from /api to /ovirt-engine/api

[Michael Pasternak]

On 11/17/2013 07:32 PM, Alon Bar-Lev wrote:
> 
> 
> ----- Original Message -----
>> From: "Michael Pasternak" <mpastern at redhat.com>
>> To: "Alon Bar-Lev" <alonbl at redhat.com>
>> Cc: "Alexander Wels" <awels at redhat.com>, "engine-devel" <engine-devel at ovirt.org>
>> Sent: Sunday, November 17, 2013 2:01:56 PM
>> Subject: Re: move api from /api to /ovirt-engine/api
>>
>> On 11/17/2013 01:23 PM, Alon Bar-Lev wrote:
>>>
>>>
>>> ----- Original Message -----
>>>> From: "Michael Pasternak" <mpastern at redhat.com>
>>>> To: "Alon Bar-Lev" <alonbl at redhat.com>
>>>> Cc: "engine-devel" <engine-devel at ovirt.org>
>>>> Sent: Sunday, November 17, 2013 1:20:58 PM
>>>> Subject: move api from /api to /ovirt-engine/api
>>>>
>>>>
>>>> Hey Alon,
>>>>
>>>> i've noticed that ForwardServlet does not address URI parameters (such as
>>>> query/matrix),
>>>> i.e all destinations of this servlet will lack URI parameters been passed
>>>> at
>>>> source.
>>>
>>> Thanks!!!!
>>> Alexander, can you please look at it?
>>>
>>> Michael... What do you mean URI parameters? just for me to be able to
>>> investigate this today?
>>> Do you mean query parameters?
>>
>> actually it didn't worked for me with matrix [2] so i assumed it's true for
>> all URI
>> params such as query [1], but it fine afaics, so it's only about [2]
>>
>> [1] http://localhost:8080/api/vms?search=name%3Dtest
>> [2] http://localhost:8080/api/events;max=2
> 
> Done[3], I was not aware that this is valid and split out... Learn new thing any day :)

Alon,

Thanks a lot for addressing this so quick,

+2

> 
> I hope I got this right. Working for this test case as far as I can see.
> 
> [3] http://gerrit.ovirt.org/#/c/21335/
> 
>>
>> thanks.
>>
>>>
>>>>
>>>> On 11/06/2013 12:10 PM, Alon Bar-Lev wrote:
>>>>> Alon Bar-Lev has posted comments on this change.
>>>>>
>>>>> --
>>>>> To view, visit http://gerrit.ovirt.org/20786
>>>>> To unsubscribe, visit http://gerrit.ovirt.org/settings>
>>>>>
>>>> --
>>>>
>>>> Michael Pasternak
>>>> RedHat, ENG-Virtualization R&D
>>>>
>>
>>
>> --
>>
>> Michael Pasternak
>> RedHat, ENG-Virtualization R&D
>>


-- 

Michael Pasternak
RedHat, ENG-Virtualization R&D