This is an automated email from the ASF dual-hosted git repository.
rohit pushed a commit to branch 4.11
in repository https://gitbox.apache.org/repos/asf/cloudstack.git
The following commit(s) were added to refs/heads/4.11 by this push:
new c9ce3e2 router: Persistent DHCP leases file on VRs and cleanup
/etc/hosts on VM deletion (#3351)
c9ce3e2 is described below
commit c9ce3e2344ab863dae09e63bf783331097a660cb
Author: Nicolas Vazquez <[email protected]>
AuthorDate: Mon Jun 3 08:34:16 2019 -0300
router: Persistent DHCP leases file on VRs and cleanup /etc/hosts on VM
deletion (#3351)
Since the CloudStack virtual router was redesigned on version 4.6 it has
been observed that the DHCP leases file is not persistent across network
operations. This causes conflicts on guest VMs static IPs, causing these static
IPs to not be renewed by the DHCP server running on isolated and VPC networks'
virtual routers (dnsmasq). On stopping or destroying a VM, its dhcp/dns records
are not removed from the virtual router causing ghost effects.
Fixes #3272
Fixes #3354
Signed-off-by: Rohit Yadav <[email protected]>
---
.../cloud/network/element/DhcpServiceProvider.java | 2 ++
.../cloud/agent/api/routing/DhcpEntryCommand.java | 9 ++++++
.../agent/resource/virtualnetwork/VRScripts.java | 1 -
.../virtualnetwork/VirtualRoutingResource.java | 2 +-
.../virtualnetwork/facade/DhcpEntryConfigItem.java | 2 +-
.../virtualnetwork/model/VmDhcpConfig.java | 14 ++++++++-
.../service/NetworkOrchestrationService.java | 4 +++
.../engine/orchestration/NetworkOrchestrator.java | 28 +++++++++++++++++
.../networkservice/BaremetalDhcpElement.java | 5 ++++
.../contrail/management/ContrailElementImpl.java | 5 ++++
.../com/cloud/network/element/NuageVspElement.java | 5 ++++
.../network/element/VirtualRouterElement.java | 28 +++++++++++++++++
.../cloud/network/router/CommandSetupHelper.java | 5 ++--
.../com/cloud/network/rules/DhcpEntryRules.java | 10 +++++++
server/src/com/cloud/vm/UserVmManagerImpl.java | 15 +++++++---
.../network/topology/AdvancedNetworkTopology.java | 15 ++++++++++
.../network/topology/AdvancedNetworkVisitor.java | 3 +-
.../network/topology/BasicNetworkTopology.java | 21 +++++++++++++
.../network/topology/BasicNetworkVisitor.java | 3 +-
.../network/topology/NetworkTopology.java | 2 ++
.../test/com/cloud/vpc/MockNetworkManagerImpl.java | 4 +++
systemvm/debian/opt/cloud/bin/cs/CsDhcp.py | 35 ++++++++++++++++++----
systemvm/debian/opt/cloud/bin/cs_dhcp.py | 4 +++
23 files changed, 205 insertions(+), 17 deletions(-)
diff --git a/api/src/com/cloud/network/element/DhcpServiceProvider.java
b/api/src/com/cloud/network/element/DhcpServiceProvider.java
index 12830f8..3f530c2 100644
--- a/api/src/com/cloud/network/element/DhcpServiceProvider.java
+++ b/api/src/com/cloud/network/element/DhcpServiceProvider.java
@@ -37,4 +37,6 @@ public interface DhcpServiceProvider extends NetworkElement {
boolean removeDhcpSupportForSubnet(Network network) throws
ResourceUnavailableException;
boolean setExtraDhcpOptions(Network network, long nicId, Map<Integer,
String> dhcpOptions);
+
+ boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile vmProfile) throws ResourceUnavailableException;
}
diff --git a/core/src/com/cloud/agent/api/routing/DhcpEntryCommand.java
b/core/src/com/cloud/agent/api/routing/DhcpEntryCommand.java
index b942806..7fb65fe 100644
--- a/core/src/com/cloud/agent/api/routing/DhcpEntryCommand.java
+++ b/core/src/com/cloud/agent/api/routing/DhcpEntryCommand.java
@@ -35,6 +35,15 @@ public class DhcpEntryCommand extends NetworkElementCommand {
String duid;
private boolean isDefault;
boolean executeInSequence = false;
+ boolean remove;
+
+ public boolean isRemove() {
+ return remove;
+ }
+
+ public void setRemove(boolean remove) {
+ this.remove = remove;
+ }
protected DhcpEntryCommand() {
diff --git a/core/src/com/cloud/agent/resource/virtualnetwork/VRScripts.java
b/core/src/com/cloud/agent/resource/virtualnetwork/VRScripts.java
index 838f087..bd108a9 100644
--- a/core/src/com/cloud/agent/resource/virtualnetwork/VRScripts.java
+++ b/core/src/com/cloud/agent/resource/virtualnetwork/VRScripts.java
@@ -68,5 +68,4 @@ public class VRScripts {
public static final String UPDATE_HOST_PASSWD = "update_host_passwd.sh";
public static final String VR_CFG = "vr_cfg.sh";
-
}
\ No newline at end of file
diff --git
a/core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
b/core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
index 1367218..8dd0620 100644
---
a/core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
+++
b/core/src/com/cloud/agent/resource/virtualnetwork/VirtualRoutingResource.java
@@ -22,7 +22,6 @@ package com.cloud.agent.resource.virtualnetwork;
import java.io.IOException;
import java.net.InetSocketAddress;
import java.nio.channels.SocketChannel;
-import org.joda.time.Duration;
import java.util.ArrayList;
import java.util.HashMap;
import java.util.List;
@@ -41,6 +40,7 @@ import org.apache.cloudstack.ca.SetupKeyStoreCommand;
import org.apache.cloudstack.ca.SetupKeystoreAnswer;
import org.apache.cloudstack.utils.security.KeyStoreUtils;
import org.apache.log4j.Logger;
+import org.joda.time.Duration;
import com.cloud.agent.api.Answer;
import com.cloud.agent.api.CheckRouterAnswer;
diff --git
a/core/src/com/cloud/agent/resource/virtualnetwork/facade/DhcpEntryConfigItem.java
b/core/src/com/cloud/agent/resource/virtualnetwork/facade/DhcpEntryConfigItem.java
index 77d436d..0710ecc 100644
---
a/core/src/com/cloud/agent/resource/virtualnetwork/facade/DhcpEntryConfigItem.java
+++
b/core/src/com/cloud/agent/resource/virtualnetwork/facade/DhcpEntryConfigItem.java
@@ -35,7 +35,7 @@ public class DhcpEntryConfigItem extends
AbstractConfigItemFacade {
final DhcpEntryCommand command = (DhcpEntryCommand) cmd;
final VmDhcpConfig vmDhcpConfig = new
VmDhcpConfig(command.getVmName(), command.getVmMac(), command.getVmIpAddress(),
command.getVmIp6Address(), command.getDuid(), command.getDefaultDns(),
- command.getDefaultRouter(), command.getStaticRoutes(),
command.isDefault());
+ command.getDefaultRouter(), command.getStaticRoutes(),
command.isDefault(), command.isRemove());
return generateConfigItems(vmDhcpConfig);
}
diff --git
a/core/src/com/cloud/agent/resource/virtualnetwork/model/VmDhcpConfig.java
b/core/src/com/cloud/agent/resource/virtualnetwork/model/VmDhcpConfig.java
index 150b8cf..d9cb8b0 100644
--- a/core/src/com/cloud/agent/resource/virtualnetwork/model/VmDhcpConfig.java
+++ b/core/src/com/cloud/agent/resource/virtualnetwork/model/VmDhcpConfig.java
@@ -30,12 +30,15 @@ public class VmDhcpConfig extends ConfigBase {
private String staticRoutes;
private boolean defaultEntry;
+ // Indicate if the entry should be removed when set to true
+ private boolean remove;
+
public VmDhcpConfig() {
super(VM_DHCP);
}
public VmDhcpConfig(String hostName, String macAddress, String
ipv4Address, String ipv6Address, String ipv6Duid, String dnsAddresses, String
defaultGateway,
- String staticRoutes, boolean defaultEntry) {
+ String staticRoutes, boolean defaultEntry, boolean remove) {
super(VM_DHCP);
this.hostName = hostName;
this.macAddress = macAddress;
@@ -46,6 +49,7 @@ public class VmDhcpConfig extends ConfigBase {
this.defaultGateway = defaultGateway;
this.staticRoutes = staticRoutes;
this.defaultEntry = defaultEntry;
+ this.remove = remove;
}
public String getHostName() {
@@ -64,6 +68,14 @@ public class VmDhcpConfig extends ConfigBase {
this.macAddress = macAddress;
}
+ public boolean isRemove() {
+ return remove;
+ }
+
+ public void setRemove(boolean remove) {
+ this.remove = remove;
+ }
+
public String getIpv4Address() {
return ipv4Address;
}
diff --git
a/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
b/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
index fa7601d..94a4259 100644
---
a/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
+++
b/engine/api/src/org/apache/cloudstack/engine/orchestration/service/NetworkOrchestrationService.java
@@ -309,4 +309,8 @@ public interface NetworkOrchestrationService {
*/
boolean areRoutersRunning(final List<? extends VirtualRouter> routers);
+ /**
+ * Remove entry from /etc/dhcphosts and /etc/hosts on virtual routers
+ */
+ void cleanupNicDhcpDnsEntry(Network network, VirtualMachineProfile
vmProfile, NicProfile nicProfile);
}
diff --git
a/engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
b/engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
index 9f4778e..9cceee8 100644
---
a/engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
+++
b/engine/orchestration/src/org/apache/cloudstack/engine/orchestration/NetworkOrchestrator.java
@@ -2913,6 +2913,34 @@ public class NetworkOrchestrator extends ManagerBase
implements NetworkOrchestra
}
/**
+ * Cleanup entry on VR file specified by type
+ */
+ @Override
+ public void cleanupNicDhcpDnsEntry(Network network, VirtualMachineProfile
vmProfile, NicProfile nicProfile) {
+
+ final List<Provider> networkProviders =
getNetworkProviders(network.getId());
+ for (final NetworkElement element : networkElements) {
+ if (networkProviders.contains(element.getProvider())) {
+ if
(!_networkModel.isProviderEnabledInPhysicalNetwork(_networkModel.getPhysicalNetworkId(network),
element.getProvider().getName())) {
+ throw new CloudRuntimeException("Service provider " +
element.getProvider().getName() + " either doesn't exist or is not enabled in
physical network id: "
+ + network.getPhysicalNetworkId());
+ }
+ if (vmProfile.getType() == Type.User && element.getProvider()
!= null) {
+ if
(_networkModel.areServicesSupportedInNetwork(network.getId(), Service.Dhcp)
+ &&
_networkModel.isProviderSupportServiceInNetwork(network.getId(), Service.Dhcp,
element.getProvider()) && element instanceof DhcpServiceProvider) {
+ final DhcpServiceProvider sp = (DhcpServiceProvider)
element;
+ try {
+ sp.removeDhcpEntry(network, nicProfile, vmProfile);
+ } catch (ResourceUnavailableException e) {
+ s_logger.error("Failed to remove dhcp-dns entry
due to: ", e);
+ }
+ }
+ }
+ }
+ }
+ }
+
+ /**
* rollingRestartRouters performs restart of routers of a network by first
* deploying a new VR and then destroying old VRs in rolling fashion. For
* non-redundant network, it will re-program the new router as final step
diff --git
a/plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetalDhcpElement.java
b/plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetalDhcpElement.java
index 5696048..807babc 100644
---
a/plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetalDhcpElement.java
+++
b/plugins/hypervisors/baremetal/src/com/cloud/baremetal/networkservice/BaremetalDhcpElement.java
@@ -185,4 +185,9 @@ public class BaremetalDhcpElement extends AdapterBase
implements DhcpServiceProv
return false;
}
+ @Override
+ public boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile vmProfile) throws ResourceUnavailableException {
+ return false;
+ }
+
}
diff --git
a/plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java
b/plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java
index 62ca29c..4771441 100644
---
a/plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java
+++
b/plugins/network-elements/juniper-contrail/src/org/apache/cloudstack/network/contrail/management/ContrailElementImpl.java
@@ -379,4 +379,9 @@ public class ContrailElementImpl extends AdapterBase
public boolean setExtraDhcpOptions(Network network, long nicId,
Map<Integer, String> dhcpOptions) {
return false;
}
+
+ @Override
+ public boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile vmProfile) {
+ return false;
+ }
}
diff --git
a/plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
b/plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
index 48c3d88..0f6bff9 100644
---
a/plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
+++
b/plugins/network-elements/nuage-vsp/src/com/cloud/network/element/NuageVspElement.java
@@ -542,6 +542,11 @@ public class NuageVspElement extends AdapterBase
implements ConnectivityProvider
}
@Override
+ public boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile vmProfile) {
+ return false;
+ }
+
+ @Override
public boolean applyStaticNats(Network config, List<? extends StaticNat>
rules) throws ResourceUnavailableException {
List<VspStaticNat> vspStaticNatDetails = new ArrayList<VspStaticNat>();
for (StaticNat staticNat : rules) {
diff --git a/server/src/com/cloud/network/element/VirtualRouterElement.java
b/server/src/com/cloud/network/element/VirtualRouterElement.java
index 58d9343..11a553c 100644
--- a/server/src/com/cloud/network/element/VirtualRouterElement.java
+++ b/server/src/com/cloud/network/element/VirtualRouterElement.java
@@ -947,6 +947,34 @@ NetworkMigrationResponder, AggregatedCommandExecutor,
RedundantResource, DnsServ
}
@Override
+ public boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile vmProfile) throws ResourceUnavailableException {
+ boolean result = true;
+ if (canHandle(network, Service.Dhcp)) {
+ if (vmProfile.getType() != VirtualMachine.Type.User) {
+ return false;
+ }
+
+ final List<DomainRouterVO> routers =
_routerDao.listByNetworkAndRole(network.getId(),
VirtualRouter.Role.VIRTUAL_ROUTER);
+
+ if (CollectionUtils.isEmpty(routers)) {
+ throw new ResourceUnavailableException("Can't find at least
one router!", DataCenter.class, network.getDataCenterId());
+ }
+
+ final DataCenterVO dcVO =
_dcDao.findById(network.getDataCenterId());
+ final NetworkTopology networkTopology =
networkTopologyContext.retrieveNetworkTopology(dcVO);
+
+ for (final DomainRouterVO domainRouterVO : routers) {
+ if (domainRouterVO.getState() != VirtualMachine.State.Running)
{
+ continue;
+ }
+
+ result = result && networkTopology.removeDhcpEntry(network,
nic, vmProfile, domainRouterVO);
+ }
+ }
+ return result;
+ }
+
+ @Override
public boolean removeDnsSupportForSubnet(Network network) throws
ResourceUnavailableException {
// Ignore if virtual router is already dhcp provider
if (_networkModel.isProviderSupportServiceInNetwork(network.getId(),
Service.Dhcp, getProvider())) {
diff --git a/server/src/com/cloud/network/router/CommandSetupHelper.java
b/server/src/com/cloud/network/router/CommandSetupHelper.java
index c8d85fe..fe231c3 100644
--- a/server/src/com/cloud/network/router/CommandSetupHelper.java
+++ b/server/src/com/cloud/network/router/CommandSetupHelper.java
@@ -211,7 +211,7 @@ public class CommandSetupHelper {
cmds.addCommand("users", cmd);
}
- public void createDhcpEntryCommand(final VirtualRouter router, final
UserVm vm, final NicVO nic, final Commands cmds) {
+ public void createDhcpEntryCommand(final VirtualRouter router, final
UserVm vm, final NicVO nic, boolean remove, final Commands cmds) {
final DhcpEntryCommand dhcpCommand = new
DhcpEntryCommand(nic.getMacAddress(), nic.getIPv4Address(), vm.getHostName(),
nic.getIPv6Address(),
_networkModel.getExecuteInSeqNtwkElmtCmd());
@@ -229,6 +229,7 @@ public class CommandSetupHelper {
dhcpCommand.setDefaultDns(ipaddress);
dhcpCommand.setDuid(NetUtils.getDuidLL(nic.getMacAddress()));
dhcpCommand.setDefault(nic.isDefaultNic());
+ dhcpCommand.setRemove(remove);
dhcpCommand.setAccessDetail(NetworkElementCommand.ROUTER_IP,
_routerControlHelper.getRouterControlIp(router.getId()));
dhcpCommand.setAccessDetail(NetworkElementCommand.ROUTER_NAME,
router.getInstanceName());
@@ -622,7 +623,7 @@ public class CommandSetupHelper {
final NicVO nic =
_nicDao.findByNtwkIdAndInstanceId(guestNetworkId, vm.getId());
if (nic != null) {
s_logger.debug("Creating dhcp entry for vm " + vm + " on domR
" + router + ".");
- createDhcpEntryCommand(router, vm, nic, cmds);
+ createDhcpEntryCommand(router, vm, nic, false, cmds);
}
}
}
diff --git a/server/src/com/cloud/network/rules/DhcpEntryRules.java
b/server/src/com/cloud/network/rules/DhcpEntryRules.java
index c4a91f4..530cf0a 100644
--- a/server/src/com/cloud/network/rules/DhcpEntryRules.java
+++ b/server/src/com/cloud/network/rules/DhcpEntryRules.java
@@ -36,6 +36,8 @@ public class DhcpEntryRules extends RuleApplier {
private final VirtualMachineProfile _profile;
private final DeployDestination _destination;
+ private boolean remove;
+
private NicVO _nicVo;
private UserVmVO _userVM;
@@ -77,4 +79,12 @@ public class DhcpEntryRules extends RuleApplier {
public UserVmVO getUserVM() {
return _userVM;
}
+
+ public boolean isRemove() {
+ return remove;
+ }
+
+ public void setRemove(boolean remove) {
+ this.remove = remove;
+ }
}
\ No newline at end of file
diff --git a/server/src/com/cloud/vm/UserVmManagerImpl.java
b/server/src/com/cloud/vm/UserVmManagerImpl.java
index 8857fe9..5e0bb1b 100644
--- a/server/src/com/cloud/vm/UserVmManagerImpl.java
+++ b/server/src/com/cloud/vm/UserVmManagerImpl.java
@@ -302,6 +302,7 @@ import com.cloud.vm.dao.VMInstanceDao;
import com.cloud.vm.snapshot.VMSnapshotManager;
import com.cloud.vm.snapshot.VMSnapshotVO;
import com.cloud.vm.snapshot.dao.VMSnapshotDao;
+import com.google.common.base.Strings;
public class UserVmManagerImpl extends ManagerBase implements UserVmManager,
VirtualMachineGuru, UserVmService, Configurable {
@@ -4377,10 +4378,16 @@ public class UserVmManagerImpl extends ManagerBase
implements UserVmManager, Vir
}
}
- List<NicVO> nics = _nicDao.listByVmId(vm.getId());
- for (NicVO nic : nics) {
- NetworkVO network = _networkDao.findById(nic.getNetworkId());
- if (network.getTrafficType() == TrafficType.Guest) {
+ final List<NicVO> nics = _nicDao.listByVmId(vm.getId());
+ for (final NicVO nic : nics) {
+ final NetworkVO network = _networkDao.findById(nic.getNetworkId());
+ if (network != null && network.getTrafficType() ==
TrafficType.Guest) {
+ final String nicIp =
Strings.isNullOrEmpty(nic.getIPv4Address()) ? nic.getIPv6Address() :
nic.getIPv4Address();
+ if (!Strings.isNullOrEmpty(nicIp)) {
+ NicProfile nicProfile = new
NicProfile(nic.getIPv4Address(), nic.getIPv6Address(), nic.getMacAddress());
+ nicProfile.setId(nic.getId());
+ _networkMgr.cleanupNicDhcpDnsEntry(network, profile,
nicProfile);
+ }
if (nic.getBroadcastUri() != null &&
nic.getBroadcastUri().getScheme().equals("pvlan")) {
NicProfile nicProfile = new NicProfile(nic, network,
nic.getBroadcastUri(), nic.getIsolationUri(), 0, false, "pvlan-nic");
setupVmForPvlan(false, vm.getHostId(), nicProfile);
diff --git
a/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkTopology.java
b/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkTopology.java
index f456fce..f35f142 100644
---
a/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkTopology.java
+++
b/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkTopology.java
@@ -173,6 +173,21 @@ public class AdvancedNetworkTopology extends
BasicNetworkTopology {
}
@Override
+ public boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile profile, VirtualRouter virtualRouter) throws
ResourceUnavailableException {
+ s_logger.debug("REMOVE VPC DHCP ENTRY RULES");
+
+ final String typeString = "dhcp entry";
+ final Long podId = null;
+ final boolean isPodLevelException = false;
+ final boolean failWhenDisconnect = false;
+
+ final DhcpEntryRules dhcpRules = new DhcpEntryRules(network, nic,
profile, null);
+ dhcpRules.setRemove(true);
+
+ return applyRules(network, virtualRouter, typeString,
isPodLevelException, podId, failWhenDisconnect, new
RuleApplierWrapper<RuleApplier>(dhcpRules));
+ }
+
+ @Override
public boolean associatePublicIP(final Network network, final List<?
extends PublicIpAddress> ipAddresses, final VirtualRouter router)
throws ResourceUnavailableException {
diff --git
a/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkVisitor.java
b/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkVisitor.java
index b5283da..c21b6d7 100644
---
a/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkVisitor.java
+++
b/server/src/org/apache/cloudstack/network/topology/AdvancedNetworkVisitor.java
@@ -80,8 +80,9 @@ public class AdvancedNetworkVisitor extends
BasicNetworkVisitor {
final Commands commands = new Commands(Command.OnError.Stop);
final NicVO nicVo = dhcp.getNicVo();
final UserVmVO userVM = dhcp.getUserVM();
+ final boolean remove = dhcp.isRemove();
- _commandSetupHelper.createDhcpEntryCommand(router, userVM, nicVo,
commands);
+ _commandSetupHelper.createDhcpEntryCommand(router, userVM, nicVo,
remove, commands);
return _networkGeneralHelper.sendCommandsToRouter(router, commands);
}
diff --git
a/server/src/org/apache/cloudstack/network/topology/BasicNetworkTopology.java
b/server/src/org/apache/cloudstack/network/topology/BasicNetworkTopology.java
index 0869b22..a7fbe31 100644
---
a/server/src/org/apache/cloudstack/network/topology/BasicNetworkTopology.java
+++
b/server/src/org/apache/cloudstack/network/topology/BasicNetworkTopology.java
@@ -442,4 +442,25 @@ public class BasicNetworkTopology implements
NetworkTopology {
}
return result;
}
+
+ @Override
+ public boolean removeDhcpEntry(Network network, NicProfile nic,
VirtualMachineProfile profile, VirtualRouter virtualRouter) throws
ResourceUnavailableException {
+ s_logger.debug("REMOVING DHCP ENTRY RULE");
+
+ final String typeString = "dhcp entry";
+ final Long podId = profile.getVirtualMachine().getPodIdToDeployIn();
+ boolean isPodLevelException = false;
+
+ if (podId != null && profile.getVirtualMachine().getType() ==
VirtualMachine.Type.User && network.getTrafficType() == TrafficType.Guest
+ && network.getGuestType() == Network.GuestType.Shared) {
+ isPodLevelException = true;
+ }
+
+ final boolean failWhenDisconnect = false;
+
+ final DhcpEntryRules dhcpRules = new DhcpEntryRules(network, nic,
profile, null);
+ dhcpRules.setRemove(true);
+
+ return applyRules(network, virtualRouter, typeString,
isPodLevelException, podId, failWhenDisconnect, new
RuleApplierWrapper<RuleApplier>(dhcpRules));
+ }
}
\ No newline at end of file
diff --git
a/server/src/org/apache/cloudstack/network/topology/BasicNetworkVisitor.java
b/server/src/org/apache/cloudstack/network/topology/BasicNetworkVisitor.java
index 53ddebb..35d846b 100644
--- a/server/src/org/apache/cloudstack/network/topology/BasicNetworkVisitor.java
+++ b/server/src/org/apache/cloudstack/network/topology/BasicNetworkVisitor.java
@@ -196,9 +196,10 @@ public class BasicNetworkVisitor extends
NetworkTopologyVisitor {
final NicVO nicVo = dhcp.getNicVo();
final UserVmVO userVM = dhcp.getUserVM();
final DeployDestination destination = dhcp.getDestination();
+ final boolean remove = dhcp.isRemove();
if (router.getPodIdToDeployIn().longValue() ==
destination.getPod().getId()) {
- _commandSetupHelper.createDhcpEntryCommand(router, userVM, nicVo,
commands);
+ _commandSetupHelper.createDhcpEntryCommand(router, userVM, nicVo,
remove, commands);
return _networkGeneralHelper.sendCommandsToRouter(router,
commands);
}
diff --git
a/server/src/org/apache/cloudstack/network/topology/NetworkTopology.java
b/server/src/org/apache/cloudstack/network/topology/NetworkTopology.java
index 5190d5e..fa76f83 100644
--- a/server/src/org/apache/cloudstack/network/topology/NetworkTopology.java
+++ b/server/src/org/apache/cloudstack/network/topology/NetworkTopology.java
@@ -87,4 +87,6 @@ public interface NetworkTopology {
boolean applyRules(final Network network, final VirtualRouter router,
final String typeString, final boolean isPodLevelException, final Long podId,
final boolean failWhenDisconnect, RuleApplierWrapper<RuleApplier>
ruleApplier) throws ResourceUnavailableException;
+
+ boolean removeDhcpEntry(final Network network, final NicProfile nic, final
VirtualMachineProfile profile, final VirtualRouter virtualRouter) throws
ResourceUnavailableException;
}
\ No newline at end of file
diff --git a/server/test/com/cloud/vpc/MockNetworkManagerImpl.java
b/server/test/com/cloud/vpc/MockNetworkManagerImpl.java
index 8cbf30c..980b440 100644
--- a/server/test/com/cloud/vpc/MockNetworkManagerImpl.java
+++ b/server/test/com/cloud/vpc/MockNetworkManagerImpl.java
@@ -925,6 +925,10 @@ public class MockNetworkManagerImpl extends ManagerBase
implements NetworkOrches
}
@Override
+ public void cleanupNicDhcpDnsEntry(Network network, VirtualMachineProfile
vmProfile, NicProfile nicProfile) {
+ }
+
+ @Override
public void finalizeUpdateInSequence(Network network, boolean success) {
return;
}
diff --git a/systemvm/debian/opt/cloud/bin/cs/CsDhcp.py
b/systemvm/debian/opt/cloud/bin/cs/CsDhcp.py
index ecbc12f..6ea9174 100755
--- a/systemvm/debian/opt/cloud/bin/cs/CsDhcp.py
+++ b/systemvm/debian/opt/cloud/bin/cs/CsDhcp.py
@@ -16,6 +16,7 @@
# under the License.
import CsHelper
import logging
+import os
from netaddr import *
from random import randint
from CsGuestNetwork import CsGuestNetwork
@@ -46,8 +47,8 @@ class CsDhcp(CsDataBag):
for item in self.dbag:
if item == "id":
continue
- self.add(self.dbag[item])
- self.write_hosts()
+ if not self.dbag[item]['remove']:
+ self.add(self.dbag[item])
self.configure_server()
@@ -64,6 +65,8 @@ class CsDhcp(CsDataBag):
if restart_dnsmasq:
self.delete_leases()
+ self.write_hosts()
+
if not self.cl.is_redundant() or self.cl.is_master():
if restart_dnsmasq:
CsHelper.service("dnsmasq", "restart")
@@ -114,10 +117,28 @@ class CsDhcp(CsDataBag):
idx += 1
def delete_leases(self):
+ macs_dhcphosts = []
+ interfaces = filter(lambda x: x.startswith('eth'),
os.listdir('/sys/class/net'))
try:
- open(LEASES, 'w').close()
- except IOError:
- return
+ logging.info("Attempting to delete entries from dnsmasq.leases
file for VMs which are not on dhcphosts file")
+ for host in open(DHCP_HOSTS):
+ macs_dhcphosts.append(host.split(',')[0])
+
+ removed = 0
+ for leaseline in open(LEASES):
+ lease = leaseline.split(' ')
+ mac = lease[1]
+ ip = lease[2]
+ if mac not in macs_dhcphosts:
+ for interface in interfaces:
+ cmd = "dhcp_release %s %s %s" % (interface, ip, mac)
+ logging.info(cmd)
+ CsHelper.execute(cmd)
+ removed = removed + 1
+ self.del_host(ip)
+ logging.info("Deleted %s entries from dnsmasq.leases file" %
str(removed))
+ except Exception as e:
+ logging.error("Caught error while trying to delete entries from
dnsmasq.leases file: %s" % e)
def preseed(self):
self.add_host("127.0.0.1", "localhost %s" % CsHelper.get_hostname())
@@ -170,3 +191,7 @@ class CsDhcp(CsDataBag):
def add_host(self, ip, hosts):
self.hosts[ip] = hosts
+
+ def del_host(self, ip):
+ if ip in self.hosts:
+ self.hosts.pop(ip)
diff --git a/systemvm/debian/opt/cloud/bin/cs_dhcp.py
b/systemvm/debian/opt/cloud/bin/cs_dhcp.py
index 88b4b75..735f847 100755
--- a/systemvm/debian/opt/cloud/bin/cs_dhcp.py
+++ b/systemvm/debian/opt/cloud/bin/cs_dhcp.py
@@ -15,6 +15,7 @@
# specific language governing permissions and limitations
# under the License.
+import logging
from netaddr import *
@@ -36,6 +37,9 @@ def merge(dbag, data):
remove_keys.add(key)
break
+ if data['remove'] and key not in remove_keys:
+ remove_keys.add(key)
+
for remove_key in remove_keys:
del(dbag[remove_key])
