----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/12968/ -----------------------------------------------------------
(Updated July 27, 2013, 6:58 p.m.) Review request for cloudstack and Min Chen. Changes ------- Updated diff rebased against latest. Bugs: 3686 Repository: cloudstack-git Description ------- Registering a template does not generate a usage event. The process should generate a usage event when the template is 100% downloaded. Added a new usage event that has virtual_size also as a parameter. Diffs (updated) ----- engine/schema/src/com/cloud/event/dao/UsageEventDao.java 01979e1 engine/schema/src/com/cloud/event/dao/UsageEventDaoImpl.java cda02ef server/src/com/cloud/template/HypervisorTemplateAdapter.java 4b3cade Diff: https://reviews.apache.org/r/12968/diff/ Testing ------- Now usage_event table is getting updated with the new usage_event. select * from usage_event where id = 23; +----+-----------------+------------+---------------------+---------+-------------+---------------+-------------+-------------+----------+---------------+-----------+--------------+ | id | type | account_id | created | zone_id | resource_id | resource_name | offering_id | template_id | size | resource_type | processed | virtual_size | +----+-----------------+------------+---------------------+---------+-------------+---------------+-------------+-------------+----------+---------------+-----------+--------------+ | 23 | TEMPLATE.CREATE | 2 | 2013-07-26 12:04:29 | 0 | 215 | r8 | NULL | NULL | 16777728 | NULL | 0 | 16777216 | +----+-----------------+------------+---------------------+---------+-------------+---------------+-------------+-------------+----------+---------------+-----------+--------------+ Build passed. Patch applies cleanly. Thanks, Saksham Srivastava