-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/11858/
-----------------------------------------------------------
Review request for cloudstack, Murali Reddy and Ram Ganesh.
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.
This addresses bug CLOUDSTACK-2340.
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
