[
https://issues.apache.org/jira/browse/CLOUDSTACK-9779?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15874205#comment-15874205
]
ASF GitHub Bot commented on CLOUDSTACK-9779:
--------------------------------------------
Github user niteshsarda commented on a diff in the pull request:
https://github.com/apache/cloudstack/pull/1937#discussion_r101966427
--- Diff: server/src/com/cloud/network/NetworkServiceImpl.java ---
@@ -852,7 +852,8 @@ public boolean releaseSecondaryIpFromNic(long
ipAddressId) {
throw new InvalidParameterValueException("Can' remove the
ip " + secondaryIp + "is associate with static NAT rule public IP address id "
+ publicIpVO.getId());
}
- if (_lbService.isLbRuleMappedToVmGuestIp(secondaryIp)) {
+ List<Long> lbRuleIdList =
_firewallDao.listIdByNetworkAndPurposeAndNotRevoked(network.getId(),
Purpose.LoadBalancing);
--- End diff --
@ustcweizhou : As per your comment, do you expect Null and Empty check for
list variable "lbRuleIdList " to be moved in LoadBalancerVMMapDao.java class ?
> Releasing secondary guest IP fails with error VM nic Ip x.x.x.x is mapped to
> load balancing rule
> ------------------------------------------------------------------------------------------------
>
> Key: CLOUDSTACK-9779
> URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9779
> Project: CloudStack
> Issue Type: Bug
> Security Level: Public(Anyone can view this level - this is the
> default.)
> Components: Management Server
> Reporter: Nitesh Sarda
>
> ISSUE
> =================
> Releasing secondary guest IP fails with error VM nic Ip x.x.x.x is mapped to
> load balancing rule
> REPRO STEPS
> ==================
> 1. Create two isolated guest networks with same CIDR
> 2. Deploy VMs on both networks
> 3. Acquire secondary IP on NICs of both VMs and make sure they have the same
> value, user can input the IP address.
> 4. Configure Loadbalancing rule on one of the secondary IP address and try
> releasing the other secondary IP address.
> 5. The operation would fail
> EXPECTED BEHAVIOR
> ==================
> Secondary IP address should be released if there are no LB rules associated
> with it.
> ACTUAL BEHAVIOR
> ==================
> Releasing secondary IP address even if there are no LB rules associated with
> it.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
