----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12852/#review23684 -----------------------------------------------------------
Review 12852 PASSED the build test The url of build cloudstack-master-with-patch #21 is : http://jenkins.cloudstack.org/job/cloudstack-master-with-patch/21/ - Jenkins Cloudstack.org On July 23, 2013, 11:18 a.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/12852/ > ----------------------------------------------------------- > > (Updated July 23, 2013, 11:18 a.m.) > > > Review request for cloudstack and Kishan Kavala. > > > Bugs: 2508 > > > Repository: cloudstack-git > > > Description > ------- > > Usage entries for template and iso should include virtual_size parameter. > Added new column to usage_event table which contains the virtual size. > Modified the usage event creation when a template is created. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/response/UsageRecordResponse.java 4b355cb > api/src/org/apache/cloudstack/usage/Usage.java c74d3b7 > engine/schema/src/com/cloud/event/UsageEventVO.java c92972b > engine/schema/src/com/cloud/event/dao/UsageEventDaoImpl.java 004ab7c > engine/schema/src/com/cloud/usage/UsageStorageVO.java 1337edc > engine/schema/src/com/cloud/usage/UsageVO.java 18a3a6b > engine/schema/src/com/cloud/usage/dao/UsageDaoImpl.java 2237d56 > engine/schema/src/com/cloud/usage/dao/UsageStorageDaoImpl.java 297c8f4 > server/src/com/cloud/api/ApiResponseHelper.java e058105 > server/src/com/cloud/event/UsageEventUtils.java 2d92fae > server/src/com/cloud/template/TemplateManagerImpl.java 5005ee0 > usage/src/com/cloud/usage/UsageManagerImpl.java 4901dd9 > usage/src/com/cloud/usage/parser/StorageUsageParser.java 337e8e2 > > Diff: https://reviews.apache.org/r/12852/diff/ > > > Testing > ------- > > Usage event generated for creating template with virtual size. > > > Thanks, > > Saksham Srivastava > >