Prachi can you move the patch to "Submitted"
> -----Original Message-----
> From: Prachi Damle [mailto:nore...@reviews.apache.org] On Behalf Of Prachi
> Damle
> Sent: Wednesday, February 13, 2013 5:56 PM
> To: Prachi Damle
> Cc: cloudstack; Likitha Shetty
> Subject: Re: Review Request: [EC2 Query API] DescribeSnapshots, 'n'
> ListVolumes get fired on CS for displaying 'n' Snapshots taken from the same
> Volume
>
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/9247/#review16554
> -----------------------------------------------------------
>
> Ship it!
>
>
> Thanks for making the changes. Committed to master.
>
> - Prachi Damle
>
>
> On Feb. 13, 2013, 6:18 a.m., Likitha Shetty wrote:
> >
> > -----------------------------------------------------------
> > This is an automatically generated e-mail. To reply, visit:
> > https://reviews.apache.org/r/9247/
> > -----------------------------------------------------------
> >
> > (Updated Feb. 13, 2013, 6:18 a.m.)
> >
> >
> > Review request for cloudstack and Prachi Damle.
> >
> >
> > Description
> > -------
> >
> > For snapshots taken from the same volume re-use the response obtained
> > by calling listVolumes
> >
> >
> > This addresses bug CLOUDSTACK-1134.
> >
> >
> > Diffs
> > -----
> >
> > awsapi/src/com/cloud/bridge/service/EC2SoapServiceImpl.java 9fc581b
> > awsapi/src/com/cloud/bridge/service/core/ec2/EC2Engine.java a835d8a
> >
> > Diff: https://reviews.apache.org/r/9247/diff/
> >
> >
> > Testing
> > -------
> >
> > 1. Create multiple snapshots from the same volume 2. Run
> > ec2_get_all_snapshots - check CS api.log to ensure listVolumes has
> > been called just once for every volume
> >
> >
> > Thanks,
> >
> > Likitha Shetty
> >
> >
