----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11458/#review21409 -----------------------------------------------------------
Should set the owner to be entity owner passed as ipowner instead of caller. - Murali Reddy On May 28, 2013, 7:27 a.m., Jayapal Reddy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11458/ > ----------------------------------------------------------- > > (Updated May 28, 2013, 7:27 a.m.) > > > Review request for cloudstack, Abhinandan Prateek and Murali Reddy. > > > Description > ------- > > Updated account and domain id for nic secondary ips for shared networks. > Fixed by getting accoundId and domainId from the caller instead of network > > > This addresses bug CLOUDSTACK-2609. > > > Diffs > ----- > > server/src/com/cloud/network/NetworkServiceImpl.java f880bcc > > Diff: https://reviews.apache.org/r/11458/diff/ > > > Testing > ------- > > Tested on isolated and shared networks. > > > Thanks, > > Jayapal Reddy > >
