----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/9110/ -----------------------------------------------------------
(Updated Jan. 29, 2013, 12:21 p.m.) Review request for cloudstack. Description ------- CLOUDSTACK-713: Limit Resources to domain/accountsi(CPU and RAM) Addition of two new resource types i.e. CPU and RAM in the existing pool of resource types. Added some methods to set the limits on these resources using updateResource API command and to get a count using updateResourceCount. Also added calls in Virtual machine life cycle to check these limits and to increment/decrement resource count Resource Name :: Resource type number CPU 8 RAM 9 This addresses bug CLOUDSTACK-713. Diffs (updated) ----- api/src/com/cloud/configuration/Resource.java 7f551d6 api/src/org/apache/cloudstack/api/command/user/resource/UpdateResourceCountCmd.java 4aa694b api/src/org/apache/cloudstack/api/command/user/resource/UpdateResourceLimitCmd.java 9b6359f api/src/org/apache/cloudstack/api/response/AccountResponse.java 51d3352 api/src/org/apache/cloudstack/api/response/ResourceCountResponse.java 7a29194 api/src/org/apache/cloudstack/api/response/ResourceLimitResponse.java c01e12f server/src/com/cloud/api/query/dao/AccountJoinDaoImpl.java 6268724 server/src/com/cloud/api/query/vo/AccountJoinVO.java 6d37f4d server/src/com/cloud/baremetal/BareMetalVmManagerImpl.java 57cfb39 server/src/com/cloud/configuration/Config.java 4ae144e server/src/com/cloud/resourcelimit/ResourceLimitManagerImpl.java c17b0ea server/src/com/cloud/vm/UserVmManagerImpl.java ecf1242 server/test/com/cloud/resourcelimit/ResourceLimitManagerImplTest.java PRE-CREATION server/test/com/cloud/vpc/MockResourceLimitManagerImpl.java a0c7b70 setup/db/create-schema-view.sql f68a6ca setup/db/db/schema-40to410.sql ed4946e Diff: https://reviews.apache.org/r/9110/diff/ Testing ------- Manually tested on my local CloudStack setup. Also added unit tests file in the patch. Thanks, Sanjay Tripathi