----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/15280/#review30987 -----------------------------------------------------------
Reminder - This review has been pending for long. Request the submitter to please add relevant reviewers or seek volunteers from the community. Thanks! - Amogh Vasekar On Nov. 6, 2013, 7:09 p.m., David Grizzanti wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/15280/ > ----------------------------------------------------------- > > (Updated Nov. 6, 2013, 7:09 p.m.) > > > Review request for cloudstack. > > > Bugs: CLOUDSTACK-5062 > https://issues.apache.org/jira/browse/CLOUDSTACK-5062 > > > Repository: cloudstack-git > > > Description > ------- > > CLOUDSTACK-5062: Deleting Load Balancing Rule fails when generating usage > events are enabled > > Added logic so that a correct zoneID is passed when generating usage events > while deleting load balancing rules: > - Added logic to > server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java to grab > zone id and pass to publishUsageEvent > > > Diffs > ----- > > server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java be81a3e > > Diff: https://reviews.apache.org/r/15280/diff/ > > > Testing > ------- > > Testing only on 4.2 > > > Thanks, > > David Grizzanti > >