incubator-cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Pranav Saxena" <pranav.sax...@citrix.com>
Subject Re: Review Request: CLOUDSTACK-420: listZones missing name parameter
Date Tue, 26 Feb 2013 14:36:37 GMT

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


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message