cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Animesh Chaturvedi <animesh.chaturv...@citrix.com>
Subject RE: Review Request: CLOUDSTACK-420: listZones missing name parameter
Date Thu, 14 Mar 2013 18:52:36 GMT
David

Please mark patch as submitted

> -----Original Message-----
> From: Prasanna Santhanam [mailto:noreply@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
> >
> >

Mime
View raw message