----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/7288/#review11974 -----------------------------------------------------------
server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java <https://reviews.apache.org/r/7288/#comment25542> please include zone id while throwing the execption server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java <https://reviews.apache.org/r/7288/#comment25543> refine the comment to say it does not reach NetScaler resource layer - Ram Ganesh On Sept. 26, 2012, 6:13 p.m., Vijay Venkatachalam wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/7288/ > ----------------------------------------------------------- > > (Updated Sept. 26, 2012, 6:13 p.m.) > > > Review request for cloudstack, Devdeep Singh and Ram Ganesh. > > > Description > ------- > > The loadbalancer's network and it's VMs are present in the same network. > Enabling autoScaled VMs to be provisioned in the loadbalancer's network. > > > This addresses bug CS-15938. > > > Diffs > ----- > > api/src/com/cloud/agent/api/to/LoadBalancerTO.java 9a58d9a > api/src/com/cloud/network/lb/LoadBalancingRule.java 6647b8e > > plugins/network-elements/netscaler/src/com/cloud/network/resource/NetscalerResource.java > b78e770 > server/src/com/cloud/network/as/AutoScaleManagerImpl.java 62b5e47 > server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java 59ae8e6 > > Diff: https://reviews.apache.org/r/7288/diff/ > > > Testing > ------- > > Create autoscale config > DeployVirtualMachine command sent from NS succeeded. > > > Thanks, > > Vijay Venkatachalam > >
