From 09022344eec00330dd5d7f65778d4208ee893de4 Mon Sep 17 00:00:00 2001 From: Jayapal Date: Tue, 1 Jul 2014 12:28:35 +0530 Subject: [PATCH] CLOUDSTACK-7027: Removing left over static nat rule (cherry picked from commit a69c1ee31d233d4a6f77d68fd70b9cfe3c85f89b) --- .../router/VirtualNetworkApplianceManagerImpl.java | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java b/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java index bf0a6cc7af4..2363841d364 100755 --- a/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java +++ b/server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java @@ -3391,15 +3391,28 @@ public class VirtualNetworkApplianceManagerImpl extends ManagerBase implements V final IpAddressTO[] ipsToSend = new IpAddressTO[ipAddrList.size()]; int i = 0; boolean firstIP = true; + boolean isSourceNatNw = false; for (final PublicIpAddress ipAddr : ipAddrList) { final boolean add = (ipAddr.getState() == IpAddress.State.Releasing ? false : true); boolean sourceNat = ipAddr.isSourceNat(); + + //set the isSourceNatNw from the first ip of ipAddrList + //For non source network ips the isSourceNatNw is always false + if (sourceNat) { + isSourceNatNw = ipAddr.isSourceNat(); + } + /* enable sourceNAT for the first ip of the public interface */ if (firstIP) { sourceNat = true; } + + // setting sourceNat=true to make sure the snat rule of the ip is deleted + if (!isSourceNatNw && !add ) { + sourceNat = true; + } final String vlanId = ipAddr.getVlanTag(); final String vlanGateway = ipAddr.getGateway(); final String vlanNetmask = ipAddr.getNetmask();