> On Aug. 31, 2012, 6:46 p.m., Prachi Damle wrote:
> >
> 
> Rajesh Battala wrote:
>     Thanks alot Prachi for the review and committing my changes.
>

As the patch got committed to ASF Master am closing this review request.


- Rajesh


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/6749/#review10950
-----------------------------------------------------------


On Aug. 31, 2012, 4 a.m., Rajesh Battala wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/6749/
> -----------------------------------------------------------
> 
> (Updated Aug. 31, 2012, 4 a.m.)
> 
> 
> Review request for cloudstack, Prachi Damle, Kishan Kavala, and Alex Huang.
> 
> 
> Description
> -------
> 
> Created/Modified new VO's and Dao Impl classes to use Generic Dao Framework
> 
> 
> Diffs
> -----
> 
>   awsapi-setup/setup/cloudstack-aws-api-register 19b53fd 
>   awsapi/.classpath 4dc46c4 
>   awsapi/conf/hibernate.cfg.xml d484849 
>   awsapi/src/com/cloud/bridge/auth/ec2/AuthenticationHandler.java 3b28c00 
>   awsapi/src/com/cloud/bridge/auth/s3/AuthenticationHandler.java f7a1a7e 
>   awsapi/src/com/cloud/bridge/lifecycle/ServiceEngineLifecycle.java 88c8729 
>   awsapi/src/com/cloud/bridge/model/BucketPolicyVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/CloudStackAccountVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/CloudStackConfigurationVO.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/CloudStackServiceOfferingVO.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/MHost.hbm.xml e151171 
>   awsapi/src/com/cloud/bridge/model/MHost.java 2187c7e 
>   awsapi/src/com/cloud/bridge/model/MHostMount.hbm.xml c186de5 
>   awsapi/src/com/cloud/bridge/model/MHostMount.java 771ed8b 
>   awsapi/src/com/cloud/bridge/model/MHostMountVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/MHostVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/MultiPartPartsVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/MultiPartUploadsVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/MultipartMetaVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/OfferingBundleVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/SAcl.hbm.xml 175e1a7 
>   awsapi/src/com/cloud/bridge/model/SAcl.java 7cbcabe 
>   awsapi/src/com/cloud/bridge/model/SAclVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/SBucket.hbm.xml 83f495d 
>   awsapi/src/com/cloud/bridge/model/SBucket.java 720907f 
>   awsapi/src/com/cloud/bridge/model/SBucketVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/SHost.hbm.xml 5308a08 
>   awsapi/src/com/cloud/bridge/model/SHost.java 758cbbf 
>   awsapi/src/com/cloud/bridge/model/SHostVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/SMeta.hbm.xml 0f3b12b 
>   awsapi/src/com/cloud/bridge/model/SMeta.java 2c98571 
>   awsapi/src/com/cloud/bridge/model/SMetaVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/SObject.hbm.xml 3929bcd 
>   awsapi/src/com/cloud/bridge/model/SObject.java 1692b68 
>   awsapi/src/com/cloud/bridge/model/SObjectItem.hbm.xml 02297a2 
>   awsapi/src/com/cloud/bridge/model/SObjectItem.java bbff267 
>   awsapi/src/com/cloud/bridge/model/SObjectItemVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/SObjectVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/model/UserCredentials.hbm.xml d2f3919 
>   awsapi/src/com/cloud/bridge/model/UserCredentials.java b674f40 
>   awsapi/src/com/cloud/bridge/model/UserCredentialsVO.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/EntityDao.java cc4a8c7 
>   awsapi/src/com/cloud/bridge/persist/GMTDateTimeUserType.java cf16233 
>   awsapi/src/com/cloud/bridge/persist/PersistContext.java 8e5e0de 
>   awsapi/src/com/cloud/bridge/persist/PersistException.java 920cf40 
>   awsapi/src/com/cloud/bridge/persist/dao/BucketPolicyDao.java dda0e2d 
>   awsapi/src/com/cloud/bridge/persist/dao/BucketPolicyDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/CloudStackAccountDao.java 31a5be8 
>   awsapi/src/com/cloud/bridge/persist/dao/CloudStackAccountDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/CloudStackConfigurationDao.java 
> ed16974 
>   awsapi/src/com/cloud/bridge/persist/dao/CloudStackConfigurationDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/CloudStackSvcOfferingDao.java 
> 5013eac 
>   awsapi/src/com/cloud/bridge/persist/dao/CloudStackSvcOfferingDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MHostDao.java 7250131 
>   awsapi/src/com/cloud/bridge/persist/dao/MHostDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MHostMountDao.java efee98c 
>   awsapi/src/com/cloud/bridge/persist/dao/MHostMountDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MultiPartPartsDao.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MultiPartPartsDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MultiPartUploadsDao.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MultiPartUploadsDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MultipartLoadDao.java 0ab83eb 
>   awsapi/src/com/cloud/bridge/persist/dao/MultipartMetaDao.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/MultipartMetaDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/OfferingDao.java 358ce3d 
>   awsapi/src/com/cloud/bridge/persist/dao/OfferingDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/SAclDao.java d62ae5b 
>   awsapi/src/com/cloud/bridge/persist/dao/SAclDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/SBucketDao.java 833bf6a 
>   awsapi/src/com/cloud/bridge/persist/dao/SBucketDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/SHostDao.java 1108bcb 
>   awsapi/src/com/cloud/bridge/persist/dao/SHostDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/SMetaDao.java b781bbb 
>   awsapi/src/com/cloud/bridge/persist/dao/SMetaDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/SObjectDao.java 3d87cbe 
>   awsapi/src/com/cloud/bridge/persist/dao/SObjectDaoImpl.java PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/SObjectItemDao.java f91d180 
>   awsapi/src/com/cloud/bridge/persist/dao/SObjectItemDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/persist/dao/UserCredentialsDao.java 4beb7a3 
>   awsapi/src/com/cloud/bridge/persist/dao/UserCredentialsDaoImpl.java 
> PRE-CREATION 
>   awsapi/src/com/cloud/bridge/service/EC2MainServlet.java 0c904a1 
>   awsapi/src/com/cloud/bridge/service/EC2RestServlet.java ceaf59d 
>   awsapi/src/com/cloud/bridge/service/S3RestServlet.java 282385b 
>   awsapi/src/com/cloud/bridge/service/controller/s3/S3BucketAction.java 
> 0ea862d 
>   awsapi/src/com/cloud/bridge/service/controller/s3/S3ObjectAction.java 
> 8fb89f9 
>   awsapi/src/com/cloud/bridge/service/controller/s3/ServiceProvider.java 
> c8ca2bd 
>   awsapi/src/com/cloud/bridge/service/core/ec2/EC2Engine.java 1363d0d 
>   awsapi/src/com/cloud/bridge/service/core/s3/S3Engine.java e8b73a4 
>   awsapi/src/com/cloud/bridge/service/core/s3/S3Grant.java b3c07be 
>   awsapi/src/com/cloud/bridge/util/CloudSessionFactory.java 404f463 
>   awsapi/src/com/cloud/bridge/util/CloudStackSessionFactory.java ac5e2d5 
>   awsapi/src/com/cloud/bridge/util/QueryHelper.java 1a1b582 
>   awsapi/src/com/cloud/stack/models/CloudStackAccount.hbm.xml 7996797 
>   awsapi/src/com/cloud/stack/models/CloudStackConfiguration.hbm.xml 134e6f1 
>   awsapi/src/com/cloud/stack/models/CloudStackServiceOffering.hbm.xml 2bbcfd1 
>   build/build-aws-api.xml 04db516 
>   client/tomcatconf/components.xml.in beaac13 
>   deps/awsapi-lib/cloud-cglib.jar PRE-CREATION 
>   deps/awsapi-lib/cloud-commons-dbcp-1.4.jar PRE-CREATION 
>   deps/awsapi-lib/cloud-commons-pool-1.5.6.jar PRE-CREATION 
>   deps/awsapi-lib/cloud-ehcache.jar PRE-CREATION 
>   deps/awsapi-lib/cloud-javax.persistence-2.0.0.jar PRE-CREATION 
>   deps/awsapi-lib/cloud-utils.jar PRE-CREATION 
>   deps/awsapi-lib/hibernate3.jar 33755a45fa81dbb318961eccdb8a1bbd8419e61d 
>   utils/src/com/cloud/utils/db/Transaction.java fc49bb0 
> 
> Diff: https://reviews.apache.org/r/6749/diff/
> 
> 
> Testing
> -------
> 
> EC2:
> ====
> Tested 
> User Registration
> instance creation
> volume creation
> address allocation
> describe instance
> 
> S3:
> ====
> 
> User Registration
> Bucket operations
>    creation
>    deletion
>    versioning
>    lookup
>    multi object delete
> Key Operations
>    creation
>    deletion
>    listing
>    lookup
>    Copy
>    acl
> 
> Multipart Upload
> 
> 
> Thanks,
> 
> Rajesh Battala
> 
>

Reply via email to