> On Dec. 3, 2013, 11:46 a.m., Murali Reddy wrote: > > patch does not apply on master. Please resubmit. Also do you want to push > > the changes to 4.3 as well?
Updated the diff so it will patch on master. This can be applied to 4.3 as well, though that would be the other patch I believe based on the state of that branch. Should i submit a separate patch for that branch? - David ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15725/#review29656 ----------------------------------------------------------- On Dec. 3, 2013, 4:50 p.m., David Grizzanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15725/ > ----------------------------------------------------------- > > (Updated Dec. 3, 2013, 4:50 p.m.) > > > Review request for cloudstack. > > > Bugs: CLOUDSTACK-5220 > https://issues.apache.org/jira/browse/CLOUDSTACK-5220 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-5022: NullPointerException when invalid zone is passed into > UsageEventUtils > > Updated UsageEventUtils code to not blow up when an invalid zone ID is passed > in. Now will still generate usage event but leaving out zone UUID. > > > Diffs > ----- > > engine/components-api/src/com/cloud/event/UsageEventUtils.java e71afb0 > > Diff: https://reviews.apache.org/r/15725/diff/ > > > Testing > ------- > > built on master > > > Thanks, > > David Grizzanti > >
