----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11858/#review22950 -----------------------------------------------------------
Ship it! Ship It! - Ram Ganesh On June 30, 2013, 6:08 a.m., Rajesh Battala wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11858/ > ----------------------------------------------------------- > > (Updated June 30, 2013, 6:08 a.m.) > > > Review request for cloudstack, Alena Prokharchyk, Murali Reddy, Ram Ganesh, > and Vijay Venkatachalam. > > > Bugs: CLOUDSTACK-2340 > > > Repository: cloudstack-git > > > Description > ------- > > Issue: when healthcheck is created for LB rule then > listLoadBalancerRuleInstance api should have the service state populated by > LBHealthCheckManager. > Fixed: Fixed the response of listLoadBalancerRuleInstance to include a > "servicestate":"UP" which tell the service state of the instance. > if the healthcheck is not created on LB then api response then > "servicestate" field won't be in the response. > > > Diffs > ----- > > api/src/com/cloud/network/lb/LoadBalancingRulesService.java 5fc41e3 > api/src/org/apache/cloudstack/api/ApiConstants.java ab1402c > > api/src/org/apache/cloudstack/api/command/user/loadbalancer/ListLoadBalancerRuleInstancesCmd.java > 49ab42c > api/src/org/apache/cloudstack/api/response/UserVmResponse.java 1f9eb1a > server/src/com/cloud/network/lb/LoadBalancingRulesManagerImpl.java 67d31ab > > Diff: https://reviews.apache.org/r/11858/diff/ > > > Testing > ------- > > 1. Created LB rule with healthcheck policy verified the > listLoadBalancerRuleInstance api response, it has the field servicestate for > all the VM's assigned to the LB rule > 2. Created LB rule without healthcheck policy, verified the > listLoadBalancerRuleInstance api response wont have the servicestate field > for all the VM's assigned to the LB rule > > > Thanks, > > Rajesh Battala > >
