----------------------------------------------------------- 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.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 > >