----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13020/#review24232 -----------------------------------------------------------
COde changes are required in UsageManager to handle new usage event - Kishan Kavala On July 29, 2013, 9:28 p.m., Saksham Srivastava wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13020/ > ----------------------------------------------------------- > > (Updated July 29, 2013, 9:28 p.m.) > > > Review request for cloudstack, Devdeep Singh and Kishan Kavala. > > > Bugs: 3252 > > > Repository: cloudstack-git > > > Description > ------- > > If an instance is deployed using explicit or implicit dedication, or neither > of them a usage event isn't generated. > Added 2 new usage_events : > EVENT_DEDICATED_VM_CREATE_IMPLICIT and EVENT_DEDICATED_VM_CREATE_EXPLICIT > generated when VMs are deployed using corresponding dedications. > > > Diffs > ----- > > api/src/com/cloud/event/EventTypes.java 6f1eed6 > server/src/com/cloud/vm/UserVmManagerImpl.java 0ec296e > > Diff: https://reviews.apache.org/r/13020/diff/ > > > Testing > ------- > > usage_events table has now entries for new types: > select * from usage_event where type like "VM%CREATE%" > id: 25 > type: VM.CREATE.IMPLICIT > account_id: 2 > created: 2013-07-29 15:46:36 > zone_id: 1 > resource_id: 7 > resource_name: c85cbc76-36fb-4fdc-97d6-8179f670cc54 > offering_id: 15 > template_id: 5 > size: NULL > resource_type: XenServer > processed: 0 > virtual_size: NULL > > > Build passed. > Patch applies cleanly. > > > Thanks, > > Saksham Srivastava > >