----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19351/#review37590 -----------------------------------------------------------
And the reason why you have to do so - in SG enabled Advance zone, you can have non-sg enabled networks. For those, the SG shouldn't be allowed to add. - Alena Prokharchyk On March 18, 2014, 1:14 p.m., Jayapal Reddy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19351/ > ----------------------------------------------------------- > > (Updated March 18, 2014, 1:14 p.m.) > > > Review request for cloudstack, Abhinandan Prateek and edison su. > > > Bugs: CLOUDSTACK-6240 > https://issues.apache.org/jira/browse/CLOUDSTACK-6240 > > > Repository: cloudstack-git > > > Description > ------- > > Updated adding SG rules for nic secondary ips in Advacned SG. > > > Diffs > ----- > > api/src/org/apache/cloudstack/api/command/user/vm/AddIpToVmNicCmd.java > b5e2239 > > Diff: https://reviews.apache.org/r/19351/diff/ > > > Testing > ------- > > Tested 'ipset -L' output on xenserver after acquiring secondary ip for vm nic > > > Thanks, > > Jayapal Reddy > >
