-----------------------------------------------------------
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.
Changes
-------
ping (reviewers). this one's marked for 4.2
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
