----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9590/#review17098 -----------------------------------------------------------
Changes look good to me .Patch applies cleanly and verified on master .Perhaps someone else in the community might want to have a look at it too . - Pranav Saxena 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.java > > > This addresses bug CLOUDSTACK-420. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/command/user/zone/ListZonesByCmd.java > 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 > >
