package org.opendaylight.netvirt.elan.internal;
-import com.google.common.base.Optional;
-import com.google.common.util.concurrent.ListenableFuture;
+import static java.util.Collections.emptyList;
+import com.google.common.base.Optional;
import java.util.ArrayList;
+import java.util.Collections;
import java.util.List;
-
+import javax.annotation.Nonnull;
+import javax.annotation.PostConstruct;
+import javax.inject.Inject;
+import javax.inject.Singleton;
+import org.opendaylight.controller.md.sal.binding.api.DataBroker;
import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
import org.opendaylight.genius.datastoreutils.AsyncDataTreeChangeListenerBase;
import org.opendaylight.genius.interfacemanager.globals.InterfaceInfo;
+import org.opendaylight.genius.interfacemanager.interfaces.IInterfaceManager;
+import org.opendaylight.genius.mdsalutil.MDSALUtil;
+import org.opendaylight.infrautils.jobcoordinator.JobCoordinator;
+import org.opendaylight.netvirt.elan.ElanException;
import org.opendaylight.netvirt.elan.utils.ElanConstants;
import org.opendaylight.netvirt.elan.utils.ElanUtils;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.etree.rev160614.EtreeInstance;
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
-import org.opendaylight.genius.interfacemanager.interfaces.IInterfaceManager;
import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.IdManagerService;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.etree.rev160614.EtreeInstance;
import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.ElanDpnInterfaces;
import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.ElanInstances;
import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.rev150602.elan.dpn.interfaces.ElanDpnInterfacesList;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
-public class ElanInstanceManager extends AsyncDataTreeChangeListenerBase<ElanInstance, ElanInstanceManager>
- implements AutoCloseable {
+@Singleton
+public class ElanInstanceManager extends AsyncDataTreeChangeListenerBase<ElanInstance, ElanInstanceManager> {
- private static final Logger logger = LoggerFactory.getLogger(ElanInstanceManager.class);
+ private static final Logger LOG = LoggerFactory.getLogger(ElanInstanceManager.class);
private final DataBroker broker;
private final IdManagerService idManager;
private final IInterfaceManager interfaceManager;
private final ElanInterfaceManager elanInterfaceManager;
- private ElanUtils elanUtils;
+ private final JobCoordinator jobCoordinator;
+ @Inject
public ElanInstanceManager(final DataBroker dataBroker, final IdManagerService managerService,
final ElanInterfaceManager elanInterfaceManager,
- final IInterfaceManager interfaceManager) {
+ final IInterfaceManager interfaceManager, final JobCoordinator jobCoordinator) {
super(ElanInstance.class, ElanInstanceManager.class);
this.broker = dataBroker;
this.idManager = managerService;
this.elanInterfaceManager = elanInterfaceManager;
this.interfaceManager = interfaceManager;
+ this.jobCoordinator = jobCoordinator;
}
- public void setElanUtils(ElanUtils elanUtils) {
- this.elanUtils = elanUtils;
- }
-
+ @Override
+ @PostConstruct
public void init() {
registerListener(LogicalDatastoreType.CONFIGURATION, broker);
}
@Override
protected void remove(InstanceIdentifier<ElanInstance> identifier, ElanInstance deletedElan) {
- logger.trace("Remove ElanInstance - Key: {}, value: {}", identifier, deletedElan);
- List<ListenableFuture<Void>> futures = new ArrayList<>();
+ LOG.trace("Remove ElanInstance - Key: {}, value: {}", identifier, deletedElan);
String elanName = deletedElan.getElanInstanceName();
// check the elan Instance present in the Operational DataStore
- Elan existingElan = elanUtils.getElanByName(elanName);
+ Elan existingElan = ElanUtils.getElanByName(broker, elanName);
long elanTag = deletedElan.getElanTag();
// Cleaning up the existing Elan Instance
if (existingElan != null) {
List<String> elanInterfaces = existingElan.getElanInterfaces();
if (elanInterfaces != null && !elanInterfaces.isEmpty()) {
for (String elanInterfaceName : elanInterfaces) {
- InstanceIdentifier<ElanInterface> elanInterfaceId = elanUtils
+ InstanceIdentifier<ElanInterface> elanInterfaceId = ElanUtils
.getElanInterfaceConfigurationDataPathId(elanInterfaceName);
InterfaceInfo interfaceInfo = interfaceManager.getInterfaceInfo(elanInterfaceName);
- elanInterfaceManager.removeElanInterface(futures, deletedElan, elanInterfaceName,
- interfaceInfo, false);
- elanUtils.delete(broker, LogicalDatastoreType.CONFIGURATION,
+ elanInterfaceManager.removeElanInterface(deletedElan, elanInterfaceName, interfaceInfo);
+ ElanUtils.delete(broker, LogicalDatastoreType.CONFIGURATION,
elanInterfaceId);
}
}
- elanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
- elanUtils.getElanInstanceOperationalDataPath(elanName));
- elanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
+ ElanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
+ ElanUtils.getElanInstanceOperationalDataPath(elanName));
+ Optional<ElanDpnInterfacesList> elanDpnInterfaceList = MDSALUtil.read(broker,
+ LogicalDatastoreType.OPERATIONAL,
+ ElanUtils.getElanDpnOperationDataPath(elanName));
+ if (elanDpnInterfaceList.isPresent()) {
+ ElanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
getElanDpnOperationDataPath(elanName));
- elanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
- elanUtils.getElanInfoEntriesOperationalDataPath(elanTag));
+ }
+ ElanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
+ ElanUtils.getElanInfoEntriesOperationalDataPath(elanTag));
}
+ ElanUtils.removeAndGetElanInterfaces(elanName).forEach(elanInterfaceName -> {
+ jobCoordinator.enqueueJob(ElanUtils.getElanInterfaceJobKey(elanInterfaceName), () -> {
+ WriteTransaction writeConfigTxn = broker.newWriteOnlyTransaction();
+ LOG.info("Deleting the elanInterface present under ConfigDS:{}", elanInterfaceName);
+ ElanUtils.delete(broker, LogicalDatastoreType.CONFIGURATION,
+ ElanUtils.getElanInterfaceConfigurationDataPathId(elanInterfaceName));
+ elanInterfaceManager.unbindService(elanInterfaceName, writeConfigTxn);
+ ElanUtils.removeElanInterfaceToElanInstanceCache(elanName, elanInterfaceName);
+ LOG.info("unbind the Interface:{} service bounded to Elan:{}", elanInterfaceName, elanName);
+ return Collections.singletonList(writeConfigTxn.submit());
+ }, ElanConstants.JOB_MAX_RETRIES);
+ });
// Release tag
- elanUtils.releaseId(idManager, ElanConstants.ELAN_ID_POOL_NAME, elanName);
+ ElanUtils.releaseId(idManager, ElanConstants.ELAN_ID_POOL_NAME, elanName);
if (deletedElan.getAugmentation(EtreeInstance.class) != null) {
removeEtreeInstance(deletedElan);
}
private void removeEtreeInstance(ElanInstance deletedElan) {
// Release leaves tag
- elanUtils.releaseId(idManager, ElanConstants.ELAN_ID_POOL_NAME,
+ ElanUtils.releaseId(idManager, ElanConstants.ELAN_ID_POOL_NAME,
deletedElan.getElanInstanceName() + ElanConstants.LEAVES_POSTFIX);
- elanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
- elanUtils.getElanInfoEntriesOperationalDataPath(
+ ElanUtils.delete(broker, LogicalDatastoreType.OPERATIONAL,
+ ElanUtils.getElanInfoEntriesOperationalDataPath(
deletedElan.getAugmentation(EtreeInstance.class).getEtreeLeafTagVal().getValue()));
}
@Override
protected void update(InstanceIdentifier<ElanInstance> identifier, ElanInstance original, ElanInstance update) {
Long existingElanTag = original.getElanTag();
- if (existingElanTag != null && existingElanTag == update.getElanTag()) {
+ String elanName = update.getElanInstanceName();
+ if (existingElanTag != null && existingElanTag.equals(update.getElanTag())) {
return;
} else if (update.getElanTag() == null) {
// update the elan-Instance with new properties
WriteTransaction tx = broker.newWriteOnlyTransaction();
- elanUtils.updateOperationalDataStore(broker, idManager,
- update, new ArrayList<String>(), tx);
- elanUtils.waitForTransactionToComplete(tx);
+ ElanUtils.updateOperationalDataStore(broker, idManager,
+ update, new ArrayList<>(), tx);
+ ElanUtils.waitForTransactionToComplete(tx);
return;
}
- elanInterfaceManager.handleunprocessedElanInterfaces(update);
+
+ jobCoordinator.enqueueJob(elanName, () -> {
+ try {
+ return elanInterfaceManager.handleunprocessedElanInterfaces(update);
+ } catch (ElanException e) {
+ LOG.error("update() failed for ElanInstance: " + identifier.toString(), e);
+ return emptyList();
+ }
+ }, ElanConstants.JOB_MAX_RETRIES);
+
}
@Override
protected void add(InstanceIdentifier<ElanInstance> identifier, ElanInstance elanInstanceAdded) {
- Elan elanInfo = elanUtils.getElanByName(elanInstanceAdded.getElanInstanceName());
+ String elanInstanceName = elanInstanceAdded.getElanInstanceName();
+ Elan elanInfo = ElanUtils.getElanByName(broker, elanInstanceName);
if (elanInfo == null) {
WriteTransaction tx = broker.newWriteOnlyTransaction();
- elanUtils.updateOperationalDataStore(broker, idManager,
- elanInstanceAdded, new ArrayList<String>(), tx);
- elanUtils.waitForTransactionToComplete(tx);
+ ElanUtils.updateOperationalDataStore(broker, idManager,
+ elanInstanceAdded, new ArrayList<>(), tx);
+ ElanUtils.waitForTransactionToComplete(tx);
}
}
public ElanInstance getElanInstanceByName(String elanInstanceName) {
InstanceIdentifier<ElanInstance> elanIdentifierId = getElanInstanceConfigurationDataPath(elanInstanceName);
- Optional<ElanInstance> elanInstance = elanUtils.read(broker,
- LogicalDatastoreType.CONFIGURATION, elanIdentifierId);
- if (elanInstance.isPresent()) {
- return elanInstance.get();
- }
- return null;
+ return MDSALUtil.read(broker, LogicalDatastoreType.CONFIGURATION, elanIdentifierId).orNull();
}
+ @Nonnull
public List<DpnInterfaces> getElanDPNByName(String elanInstanceName) {
+ return getElanDPNByName(broker, elanInstanceName);
+ }
+
+ @Nonnull
+ public static List<DpnInterfaces> getElanDPNByName(DataBroker dataBroker, String elanInstanceName) {
InstanceIdentifier<ElanDpnInterfacesList> elanIdentifier = getElanDpnOperationDataPath(elanInstanceName);
- Optional<ElanDpnInterfacesList> elanInstance = elanUtils.read(broker,
- LogicalDatastoreType.OPERATIONAL, elanIdentifier);
- if (elanInstance.isPresent()) {
- ElanDpnInterfacesList elanDPNs = elanInstance.get();
- return elanDPNs.getDpnInterfaces();
- }
- return null;
+ return MDSALUtil.read(dataBroker, LogicalDatastoreType.OPERATIONAL, elanIdentifier).toJavaUtil().map(
+ ElanDpnInterfacesList::getDpnInterfaces).orElse(Collections.emptyList());
}
- private InstanceIdentifier<ElanDpnInterfacesList> getElanDpnOperationDataPath(String elanInstanceName) {
+ private static InstanceIdentifier<ElanDpnInterfacesList> getElanDpnOperationDataPath(String elanInstanceName) {
return InstanceIdentifier.builder(ElanDpnInterfaces.class)
.child(ElanDpnInterfacesList.class, new ElanDpnInterfacesListKey(elanInstanceName)).build();
}