Sure I understand. Its fine by me. But, its not a new feature. Its adding
a param to the api so a minor enhancement.

Thanks,
-Nitin

On 19/05/14 1:47 PM, "Daan Hoogland" <daan.hoogl...@gmail.com> wrote:

>even so Nitin, it is a new feature with no unit or integration tests.
>I feel I would be neglecting my task as release manager if I let this
>in.
>
>sorry,
>Daan
>
>On Mon, May 19, 2014 at 7:46 PM, Nitin Mehta <nitin.me...@citrix.com>
>wrote:
>> Sure Daan. The current listResouceDetails api lists the metadata for all
>> the first class entities and currently doesn't have the flexibility to
>> list it by key,value pair which I think is a basic must have. That¹s
>>why I
>> propose it to be included in 4.4.
>>
>> Thanks,
>> -Nitin
>>
>> On 19/05/14 2:10 AM, "Daan Hoogland" <daan.hoogl...@gmail.com> wrote:
>>
>>>Nitin,
>>>
>>>This seems like an enhancement to me. Is there anything broken that is
>>>fixed by this enhancement?
>>>Please spend some lines to justify why it should be in 4.4?
>>>
>>>thanks,
>>>Daan
>>>
>>>On Mon, May 19, 2014 at 8:50 AM, Nitin Mehta <nitin.me...@citrix.com>
>>>wrote:
>>>> Hi Daan,
>>>> Can you please cherry-pick the following commit
>>>> d5425e1a4a8ed4cadc2f9b97f82f6f608e9a51de in 4.4 for CLOUDSTACK-6699 ?
>>>>
>>>> Thanks,
>>>> -Nitin
>>>
>>>
>>>
>>>--
>>>Daan
>>
>
>
>
>-- 
>Daan

Reply via email to