David Please mark patch as submitted
> -----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 > > > >
