----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/19351/#review37589 -----------------------------------------------------------
Jayapal, you shoudldn't rely on SG attribute of the zone. You should check if the SG service is supported by the network you are adding your rule to. - 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 > >