cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "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
Date Wed, 13 Feb 2013 06:18:48 GMT

This is an automatically generated e-mail. To reply, visit:

(Updated Feb. 13, 2013, 6:18 a.m.)

Review request for cloudstack and Prachi Damle.


Prachi, i have updated the diff. I have verified that it cleanly applies on master.
I have also updated the diff to use containsKey method of the HashMap instead of iterating
through the Map with iterator.


For snapshots taken from the same volume re-use the response obtained by calling listVolumes

This addresses bug CLOUDSTACK-1134.

Diffs (updated)

  awsapi/src/com/cloud/bridge/service/ 9fc581b 
  awsapi/src/com/cloud/bridge/service/core/ec2/ a835d8a 



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


Likitha Shetty

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