anybody else got comments? I feel I am on the edge of solid ground and slippery slope. Nitin's argument is good as (almost) good as mine.
On Mon, May 19, 2014 at 11:05 PM, Nitin Mehta <nitin.me...@citrix.com> wrote: > 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 > -- Daan
