*/
package org.opendaylight.genius.interfacemanager.listeners;
+import com.google.common.util.concurrent.ListenableFuture;
import org.opendaylight.controller.md.sal.binding.api.DataBroker;
-import org.opendaylight.controller.md.sal.binding.api.DataChangeListener;
-import org.opendaylight.controller.md.sal.common.api.data.AsyncDataBroker;
-import org.opendaylight.genius.datastoreutils.AsyncDataChangeListenerBase;
-import org.opendaylight.genius.datastoreutils.AsyncDataTreeChangeListenerBase;
+import org.opendaylight.genius.datastoreutils.AsyncClusteredDataTreeChangeListenerBase;
import org.opendaylight.genius.datastoreutils.DataStoreJobCoordinator;
+import org.opendaylight.genius.interfacemanager.IfmConstants;
+import org.opendaylight.genius.interfacemanager.IfmUtil;
+import org.opendaylight.genius.interfacemanager.commons.InterfaceManagerCommonUtils;
import org.opendaylight.genius.interfacemanager.commons.InterfaceMetaUtils;
import org.opendaylight.genius.interfacemanager.renderer.ovs.statehelpers.OvsInterfaceStateAddHelper;
import org.opendaylight.genius.interfacemanager.renderer.ovs.statehelpers.OvsInterfaceStateRemoveHelper;
import org.opendaylight.genius.interfacemanager.renderer.ovs.statehelpers.OvsInterfaceStateUpdateHelper;
+import org.opendaylight.genius.interfacemanager.renderer.ovs.utilities.IfmClusterUtils;
import org.opendaylight.genius.mdsalutil.interfaces.IMdsalApiManager;
import org.opendaylight.yang.gen.v1.urn.opendaylight.flow.inventory.rev130819.FlowCapableNodeConnector;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev160411.AlivenessMonitorService;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.IdManagerService;
import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.meta.rev160406._interface.child.info.InterfaceParentEntry;
import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.meta.rev160406._interface.child.info._interface.parent.entry.InterfaceChildEntry;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeConnectorId;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.Nodes;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.node.NodeConnector;
import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev160411.AlivenessMonitorService;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.IdManagerService;
import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-
+import java.math.BigInteger;
import java.util.ArrayList;
import java.util.List;
import java.util.concurrent.Callable;
* If PortName is not unique across DPNs, this implementation can have problems.
*/
-public class InterfaceInventoryStateListener extends AsyncDataTreeChangeListenerBase<FlowCapableNodeConnector, InterfaceInventoryStateListener> {
+public class InterfaceInventoryStateListener extends AsyncClusteredDataTreeChangeListenerBase<FlowCapableNodeConnector, InterfaceInventoryStateListener> {
private static final Logger LOG = LoggerFactory.getLogger(InterfaceInventoryStateListener.class);
- private DataBroker dataBroker;
- private IdManagerService idManager;
- private IMdsalApiManager mdsalApiManager;
- private AlivenessMonitorService alivenessMonitorService;
+ private final DataBroker dataBroker;
+ private final IdManagerService idManager;
+ private final IMdsalApiManager mdsalApiManager;
+ private final AlivenessMonitorService alivenessMonitorService;
public InterfaceInventoryStateListener(final DataBroker dataBroker, final IdManagerService idManager,
final IMdsalApiManager mdsalApiManager, final AlivenessMonitorService alivenessMonitorService) {
@Override
protected void remove(InstanceIdentifier<FlowCapableNodeConnector> key,
FlowCapableNodeConnector flowCapableNodeConnectorOld) {
- LOG.debug("Received NodeConnector Remove Event: {}, {}", key, flowCapableNodeConnectorOld);
- String portName = flowCapableNodeConnectorOld.getName();
- DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
-
- // Fetch all interfaces on this port and trigger remove worker for each of them
- List<String> interfaceChildEntries = getInterfaceChildEntries(dataBroker, portName);
- for(String interfaceName : interfaceChildEntries){
- InterfaceStateRemoveWorker interfaceStateRemoveWorker = new InterfaceStateRemoveWorker(idManager,
- key, flowCapableNodeConnectorOld, interfaceName);
- coordinator.enqueueJob(interfaceChildEntries.get(0), interfaceStateRemoveWorker);
- }
+ IfmClusterUtils.runOnlyInLeaderNode(() -> {
+ LOG.debug("Received NodeConnector Remove Event: {}, {}", key, flowCapableNodeConnectorOld);
+ String portName = flowCapableNodeConnectorOld.getName();
+ NodeConnectorId nodeConnectorId = InstanceIdentifier.keyOf(key.firstIdentifierOf(NodeConnector.class)).getId();
- InterfaceStateRemoveWorker portStateRemoveWorker = new InterfaceStateRemoveWorker(idManager,
- key, flowCapableNodeConnectorOld, portName);
- coordinator.enqueueJob(portName, portStateRemoveWorker);
+ if (!InterfaceManagerCommonUtils.isNovaOrTunnelPort(portName)) {
+ portName = getDpnPrefixedPortName(nodeConnectorId, portName);
+ }
+ remove(nodeConnectorId, null, flowCapableNodeConnectorOld, portName, true);
+ });
}
@Override
protected void update(InstanceIdentifier<FlowCapableNodeConnector> key, FlowCapableNodeConnector fcNodeConnectorOld,
FlowCapableNodeConnector fcNodeConnectorNew) {
- LOG.debug("Received NodeConnector Update Event: {}, {}, {}", key, fcNodeConnectorOld, fcNodeConnectorNew);
- String portName = fcNodeConnectorNew.getName();
- DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
+ IfmClusterUtils.runOnlyInLeaderNode(() -> {
+ LOG.debug("Received NodeConnector Update Event: {}, {}, {}", key, fcNodeConnectorOld, fcNodeConnectorNew);
+ String portName = fcNodeConnectorNew.getName();
+ DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
- InterfaceStateUpdateWorker portStateUpdateWorker = new InterfaceStateUpdateWorker(key, fcNodeConnectorOld,
- fcNodeConnectorNew, portName);
- coordinator.enqueueJob(portName, portStateUpdateWorker);
- List<String> interfaceChildEntries = getInterfaceChildEntries(dataBroker, portName);
- for(String interfaceName : interfaceChildEntries){
- InterfaceStateUpdateWorker interfaceStateUpdateWorker = new InterfaceStateUpdateWorker(key, fcNodeConnectorOld,
- fcNodeConnectorNew, interfaceName);
- coordinator.enqueueJob(interfaceChildEntries.get(0), interfaceStateUpdateWorker);
- }
+ InterfaceStateUpdateWorker portStateUpdateWorker = new InterfaceStateUpdateWorker(key, fcNodeConnectorOld,
+ fcNodeConnectorNew, portName);
+ coordinator.enqueueJob(portName, portStateUpdateWorker, IfmConstants.JOB_MAX_RETRIES);
+ });
}
@Override
protected void add(InstanceIdentifier<FlowCapableNodeConnector> key, FlowCapableNodeConnector fcNodeConnectorNew) {
- LOG.debug("Received NodeConnector Add Event: {}, {}", key, fcNodeConnectorNew);
- String portName = fcNodeConnectorNew.getName();
- NodeConnectorId nodeConnectorId = InstanceIdentifier.keyOf(key.firstIdentifierOf(NodeConnector.class)).getId();
- DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
+ IfmClusterUtils.runOnlyInLeaderNode(() -> {
+ LOG.debug("Received NodeConnector Add Event: {}, {}", key, fcNodeConnectorNew);
+ String portName = fcNodeConnectorNew.getName();
+ NodeConnectorId nodeConnectorId = InstanceIdentifier.keyOf(key.firstIdentifierOf(NodeConnector.class)).getId();
- InterfaceStateAddWorker ifStateAddWorker = new InterfaceStateAddWorker(idManager, nodeConnectorId,
- fcNodeConnectorNew, portName);
- coordinator.enqueueJob(portName, ifStateAddWorker);
- List<String> interfaceChildEntries = getInterfaceChildEntries(dataBroker, portName);
- for(String interfaceName : interfaceChildEntries){
- InterfaceStateAddWorker interfaceStateAddWorker = new InterfaceStateAddWorker(idManager, nodeConnectorId,
- fcNodeConnectorNew, interfaceName);
- coordinator.enqueueJob(interfaceChildEntries.get(0), interfaceStateAddWorker);
- }
+ //VM Migration: Delete existing interface entry for older DPN
+ if (InterfaceManagerCommonUtils.isNovaOrTunnelPort(portName)) {
+ NodeConnectorId nodeConnectorIdOld = IfmUtil.getNodeConnectorIdFromInterface(portName, dataBroker);
+ if (nodeConnectorIdOld != null && !nodeConnectorId.equals(nodeConnectorIdOld)) {
+ if (InterfaceManagerCommonUtils.isTunnelPort(portName)) {
+ BigInteger oldDpnId = IfmUtil.getDpnFromNodeConnectorId(nodeConnectorIdOld);
+ BigInteger newDpnId = IfmUtil.getDpnFromNodeConnectorId(nodeConnectorId);
+ LOG.warn("DPNID changed for tunnel interface {}: old {} new {}", portName, oldDpnId, newDpnId);
+ if(InterfaceMetaUtils.isPortConfiguredOnBridge(portName, oldDpnId, dataBroker) ||
+ !InterfaceMetaUtils.isPortConfiguredOnBridge(portName, newDpnId, dataBroker)) {
+ // Remove old interface only it wasn't configured through us and new one is.
+ return;
+ }
+ }
+ LOG.debug("Triggering NodeConnector Remove Event for the interface: {}, {}, {}", portName, nodeConnectorId, nodeConnectorIdOld);
+ remove(nodeConnectorId, nodeConnectorIdOld, fcNodeConnectorNew, portName, false);
+ //Adding a delay of 10sec for VM migration, so applications can process remove and add events
+ try {
+ Thread.sleep(IfmConstants.DELAY_TIME_IN_MILLISECOND);
+ } catch (InterruptedException e) {
+ LOG.error("Error while waiting for the vm migration remove events to get processed");
+ }
+ }
+ } else {
+ portName = getDpnPrefixedPortName(nodeConnectorId, portName);
+ }
+ DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
+ InterfaceStateAddWorker ifStateAddWorker = new InterfaceStateAddWorker(idManager, nodeConnectorId,
+ fcNodeConnectorNew, portName);
+ coordinator.enqueueJob(portName, ifStateAddWorker, IfmConstants.JOB_MAX_RETRIES);
+ });
}
+ private void remove(NodeConnectorId nodeConnectorIdNew, NodeConnectorId nodeConnectorIdOld,
+ FlowCapableNodeConnector fcNodeConnectorNew, String portName, boolean isNetworkEvent) {
+ boolean isNodePresent = InterfaceManagerCommonUtils.isNodePresent(dataBroker, nodeConnectorIdNew);
+ DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
+ InterfaceStateRemoveWorker portStateRemoveWorker = new InterfaceStateRemoveWorker(idManager, nodeConnectorIdNew,
+ nodeConnectorIdOld, fcNodeConnectorNew, portName, portName, isNodePresent, isNetworkEvent, true);
+ coordinator.enqueueJob(portName, portStateRemoveWorker, IfmConstants.JOB_MAX_RETRIES);
+ }
+
+ private String getDpnPrefixedPortName(NodeConnectorId nodeConnectorId, String portName) {
+ portName = IfmUtil.getDpnFromNodeConnectorId(nodeConnectorId).toString() +
+ IfmConstants.OF_URI_SEPARATOR +
+ portName;
+ return portName;
+ }
private class InterfaceStateAddWorker implements Callable {
private final NodeConnectorId nodeConnectorId;
private final FlowCapableNodeConnector fcNodeConnectorNew;
public Object call() throws Exception {
// If another renderer(for eg : CSS) needs to be supported, check can be performed here
// to call the respective helpers.
- return OvsInterfaceStateAddHelper.addState(dataBroker, idManager, mdsalApiManager, alivenessMonitorService, nodeConnectorId,
+ List<ListenableFuture<Void>> futures = OvsInterfaceStateAddHelper.addState(dataBroker, idManager, mdsalApiManager, alivenessMonitorService, nodeConnectorId,
interfaceName, fcNodeConnectorNew);
+ List<InterfaceChildEntry> interfaceChildEntries = getInterfaceChildEntries(dataBroker, interfaceName);
+ for (InterfaceChildEntry interfaceChildEntry : interfaceChildEntries) {
+ InterfaceStateAddWorker interfaceStateAddWorker = new InterfaceStateAddWorker(idManager, nodeConnectorId,
+ fcNodeConnectorNew, interfaceChildEntry.getChildInterface());
+ DataStoreJobCoordinator.getInstance().enqueueJob(interfaceName, interfaceStateAddWorker);
+ }
+ return futures;
}
@Override
}
private class InterfaceStateUpdateWorker implements Callable {
- private InstanceIdentifier<FlowCapableNodeConnector> key;
+ private final InstanceIdentifier<FlowCapableNodeConnector> key;
private final FlowCapableNodeConnector fcNodeConnectorOld;
private final FlowCapableNodeConnector fcNodeConnectorNew;
- private String interfaceName;
+ private final String interfaceName;
public InterfaceStateUpdateWorker(InstanceIdentifier<FlowCapableNodeConnector> key,
public Object call() throws Exception {
// If another renderer(for eg : CSS) needs to be supported, check can be performed here
// to call the respective helpers.
- return OvsInterfaceStateUpdateHelper.updateState(key, alivenessMonitorService, dataBroker, interfaceName,
+ List<ListenableFuture<Void>> futures = OvsInterfaceStateUpdateHelper.updateState(key, alivenessMonitorService, dataBroker, interfaceName,
fcNodeConnectorNew, fcNodeConnectorOld);
+ List<InterfaceChildEntry> interfaceChildEntries = getInterfaceChildEntries(dataBroker, interfaceName);
+ for (InterfaceChildEntry interfaceChildEntry : interfaceChildEntries) {
+ InterfaceStateUpdateWorker interfaceStateUpdateWorker = new InterfaceStateUpdateWorker(key, fcNodeConnectorOld,
+ fcNodeConnectorNew, interfaceChildEntry.getChildInterface());
+ DataStoreJobCoordinator.getInstance().enqueueJob(interfaceName, interfaceStateUpdateWorker);
+ }
+ return futures;
}
@Override
}
private class InterfaceStateRemoveWorker implements Callable {
- InstanceIdentifier<FlowCapableNodeConnector> key;
+ private final NodeConnectorId nodeConnectorIdNew;
+ private NodeConnectorId nodeConnectorIdOld;
FlowCapableNodeConnector fcNodeConnectorOld;
private final String interfaceName;
+ private final String parentInterface;
private final IdManagerService idManager;
+ private final boolean isNodePresent;
+ private final boolean isNetworkEvent;
+ private final boolean isParentInterface;
- public InterfaceStateRemoveWorker(IdManagerService idManager,
- InstanceIdentifier<FlowCapableNodeConnector> key,
+ public InterfaceStateRemoveWorker(IdManagerService idManager, NodeConnectorId nodeConnectorIdNew,
+ NodeConnectorId nodeConnectorIdOld,
FlowCapableNodeConnector fcNodeConnectorOld,
- String portName) {
- this.key = key;
+ String interfaceName,
+ String parentInterface,
+ boolean isNodePresent,
+ boolean isNetworkEvent,
+ boolean isParentInterface) {
+ this.nodeConnectorIdNew = nodeConnectorIdNew;
+ this.nodeConnectorIdOld = nodeConnectorIdOld;
this.fcNodeConnectorOld = fcNodeConnectorOld;
- this.interfaceName = portName;
+ this.interfaceName = interfaceName;
+ this.parentInterface = parentInterface;
this.idManager = idManager;
+ this.isNodePresent = isNodePresent;
+ this.isNetworkEvent = isNetworkEvent;
+ this.isParentInterface = isParentInterface;
}
@Override
public Object call() throws Exception {
// If another renderer(for eg : CSS) needs to be supported, check can be performed here
// to call the respective helpers.
- return OvsInterfaceStateRemoveHelper.removeInterfaceStateConfiguration(idManager, mdsalApiManager, alivenessMonitorService,
- key, dataBroker, interfaceName, fcNodeConnectorOld);
+
+ List<ListenableFuture<Void>> futures = null;
+ //VM Migration: Skip OFPPR_DELETE event received after OFPPR_ADD for same interface from Older DPN
+ if (isParentInterface && isNetworkEvent) {
+ nodeConnectorIdOld = IfmUtil.getNodeConnectorIdFromInterface(interfaceName, dataBroker);
+ if(nodeConnectorIdOld != null && !nodeConnectorIdNew.equals(nodeConnectorIdOld)) {
+ LOG.debug("Dropping the NodeConnector Remove Event for the interface: {}, {}, {}", interfaceName, nodeConnectorIdNew, nodeConnectorIdOld);
+ return futures;
+ }
+ }
+
+ futures = OvsInterfaceStateRemoveHelper.removeInterfaceStateConfiguration(idManager, mdsalApiManager, alivenessMonitorService,
+ nodeConnectorIdNew, nodeConnectorIdOld, dataBroker, interfaceName, fcNodeConnectorOld, isNodePresent, parentInterface);
+
+ List<InterfaceChildEntry> interfaceChildEntries = getInterfaceChildEntries(dataBroker, interfaceName);
+ for (InterfaceChildEntry interfaceChildEntry : interfaceChildEntries) {
+ // Fetch all interfaces on this port and trigger remove worker for each of them
+ InterfaceStateRemoveWorker interfaceStateRemoveWorker = new InterfaceStateRemoveWorker(idManager, nodeConnectorIdNew,
+ nodeConnectorIdOld, fcNodeConnectorOld, interfaceChildEntry.getChildInterface(), interfaceName, isNodePresent, isNetworkEvent, false);
+ DataStoreJobCoordinator.getInstance().enqueueJob(interfaceName, interfaceStateRemoveWorker);
+ }
+ return futures;
}
@Override
public String toString() {
return "InterfaceStateRemoveWorker{" +
- "key=" + key +
+ "nodeConnectorIdNew=" + nodeConnectorIdNew +
+ ", nodeConnectorIdOld=" + nodeConnectorIdOld +
", fcNodeConnectorOld=" + fcNodeConnectorOld +
", interfaceName='" + interfaceName + '\'' +
'}';
}
}
- public static List<String> getInterfaceChildEntries(DataBroker dataBroker, String portName){
- List<String> interfaceChildEntries = new ArrayList();
+ public static List<InterfaceChildEntry> getInterfaceChildEntries(DataBroker dataBroker, String interfaceName) {
InterfaceParentEntry interfaceParentEntry =
- InterfaceMetaUtils.getInterfaceParentEntryFromConfigDS(portName, dataBroker);
+ InterfaceMetaUtils.getInterfaceParentEntryFromConfigDS(interfaceName, dataBroker);
if (interfaceParentEntry != null && interfaceParentEntry.getInterfaceChildEntry() != null) {
- for(InterfaceChildEntry trunkInterface : interfaceParentEntry.getInterfaceChildEntry()){
- interfaceChildEntries.add(trunkInterface.getChildInterface());
- InterfaceParentEntry interfaceTrunkEntry =
- InterfaceMetaUtils.getInterfaceParentEntryFromConfigDS(trunkInterface.getChildInterface(), dataBroker);
- if(interfaceTrunkEntry != null && interfaceTrunkEntry.getInterfaceChildEntry() != null) {
- for (InterfaceChildEntry trunkMemberInterface : interfaceTrunkEntry.getInterfaceChildEntry()) {
- interfaceChildEntries.add(trunkMemberInterface.getChildInterface());
- }
- }
- }
+ return interfaceParentEntry.getInterfaceChildEntry();
}
- return interfaceChildEntries;
+ return new ArrayList<>();
}
-}
\ No newline at end of file
+}
Code Review / genius.git / blobdiff