On Thu, Mar 14, 2013 at 11:52:36AM -0700, Animesh Chaturvedi wrote: > David > > Please mark patch as submitted
I took care of that. > > > -----Original Message----- > > From: Prasanna Santhanam [mailto:nore...@reviews.apache.org] On Behalf > > Of Prasanna Santhanam > > Sent: Wednesday, February 27, 2013 12:17 AM > > To: cloudstack; Prasanna Santhanam; David Grizzanti > > Subject: Re: Review Request: CLOUDSTACK-420: listZones missing name > > parameter > > > > > > ----------------------------------------------------------- > > This is an automatically generated e-mail. To reply, visit: > > https://reviews.apache.org/r/9590/#review17141 > > ----------------------------------------------------------- > > > > Ship it! > > > > > > Merged after testing with cloudmonkey+devcloud. Thanks for the patch. Do > > consider using git format-patch for reviewboard so we can attribute author > > information easily. > > > > 53b6ba379899985ee1a40ade1aca7a1e9777ff71 (master) > > 2fc5a3ca5e1539ef42b92c59fb0e42e1c6c73bc5 (4.1) > > > > - Prasanna Santhanam > > > > > > On Feb. 26, 2013, 2:18 p.m., David Grizzanti wrote: > > > > > > ----------------------------------------------------------- > > > This is an automatically generated e-mail. To reply, visit: > > > https://reviews.apache.org/r/9590/ > > > ----------------------------------------------------------- > > > > > > (Updated Feb. 26, 2013, 2:18 p.m.) > > > > > > > > > Review request for cloudstack. > > > > > > > > > Description > > > ------- > > > > > > CLOUDSTACK-420: listZones missing name parameter > > > > > > When calling the listZones API command, you can now supply a name to > > pull a specific zone by name. > > > > > > Updated two classes to include name paramater and query data > > centers/zones by name if suppled: > > > - server/src/com/cloud/api/query/QueryManagerImpl.java > > > - > > api/src/org/apache/cloudstack/api/command/user/zone/ListZonesByCmd.ja > > va > > > > > > > > > This addresses bug CLOUDSTACK-420. > > > > > > > > > Diffs > > > ----- > > > > > > > > api/src/org/apache/cloudstack/api/command/user/zone/ListZonesByCmd.ja > > va 97fe2ffeb909e8cd5f985b5bf9f5276c67354110 > > > server/src/com/cloud/api/query/QueryManagerImpl.java > > 51312a60eb1c5b6b7a988d072731e3735765260c > > > > > > Diff: https://reviews.apache.org/r/9590/diff/ > > > > > > > > > Testing > > > ------- > > > > > > Verified on DevCloud running 4.1 > > > > > > > > > Thanks, > > > > > > David Grizzanti > > > > > > >
