----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15725/#review29656 -----------------------------------------------------------
patch does not apply on master. Please resubmit. Also do you want to push the changes to 4.3 as well? - Murali Reddy On Nov. 20, 2013, 8:15 p.m., David Grizzanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15725/ > ----------------------------------------------------------- > > (Updated Nov. 20, 2013, 8:15 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 a4be4dd > > Diff: https://reviews.apache.org/r/15725/diff/ > > > Testing > ------- > > built on master > > > Thanks, > > David Grizzanti > >