Adjust to odlparent 3 Checkstyle settings
[genius.git] / itm / itm-impl / src / main / java / org / opendaylight / genius / itm / confighelpers / ItmInternalTunnelDeleteWorker.java
index 9aa98d6b5fed9ee8d4b9a8c7e770184187bf4cb4..79d8eefa9974bf9f5fcdb61d61d9b41635b788b0 100644 (file)
@@ -7,10 +7,11 @@
  */
 package org.opendaylight.genius.itm.confighelpers;
 
+import static java.util.Collections.singletonList;
+
 import com.google.common.base.Optional;
 import com.google.common.util.concurrent.ListenableFuture;
 import java.math.BigInteger;
-import java.util.ArrayList;
 import java.util.Collection;
 import java.util.Collections;
 import java.util.List;
@@ -19,11 +20,12 @@ import java.util.concurrent.Callable;
 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.DataStoreJobCoordinator;
+import org.opendaylight.genius.infra.ManagedNewTransactionRunner;
+import org.opendaylight.genius.infra.ManagedNewTransactionRunnerImpl;
 import org.opendaylight.genius.itm.impl.ItmUtils;
 import org.opendaylight.genius.mdsalutil.interfaces.IMdsalApiManager;
+import org.opendaylight.infrautils.jobcoordinator.JobCoordinator;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.Interface;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.idmanager.rev160406.IdManagerService;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.rev160406.TunnelMonitoringTypeBase;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.rev160406.TunnelMonitoringTypeLldp;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.rev160406.TunnelTypeBase;
@@ -42,25 +44,32 @@ import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
 public class ItmInternalTunnelDeleteWorker {
+
     private static final Logger LOG = LoggerFactory.getLogger(ItmInternalTunnelDeleteWorker.class) ;
 
+    private final DataBroker dataBroker;
+    private final ManagedNewTransactionRunner txRunner;
+    private final JobCoordinator jobCoordinator;
+
+    public ItmInternalTunnelDeleteWorker(DataBroker dataBroker, JobCoordinator jobCoordinator) {
+        this.dataBroker = dataBroker;
+        this.txRunner = new ManagedNewTransactionRunnerImpl(dataBroker);
+        this.jobCoordinator = jobCoordinator;
+    }
+
     @SuppressWarnings("checkstyle:IllegalCatch")
-    public static List<ListenableFuture<Void>> deleteTunnels(DataBroker dataBroker, IdManagerService idManagerService,
-                                                             IMdsalApiManager mdsalManager,
-                                                             List<DPNTEPsInfo> dpnTepsList,
-                                                             List<DPNTEPsInfo> meshedDpnList) {
+    public List<ListenableFuture<Void>> deleteTunnels(IMdsalApiManager mdsalManager, List<DPNTEPsInfo> dpnTepsList,
+                                                      List<DPNTEPsInfo> meshedDpnList) {
         LOG.trace("TEPs to be deleted {} " , dpnTepsList);
-        List<ListenableFuture<Void>> futures = new ArrayList<>();
-        WriteTransaction writeTransaction = dataBroker.newWriteOnlyTransaction();
-        try {
+        return Collections.singletonList(txRunner.callWithNewWriteOnlyTransactionAndSubmit(writeTransaction -> {
             if (dpnTepsList == null || dpnTepsList.size() == 0) {
                 LOG.debug("no vtep to delete");
-                return futures ;
+                return;
             }
 
             if (meshedDpnList == null || meshedDpnList.size() == 0) {
                 LOG.debug("No Meshed Vteps");
-                return futures ;
+                return;
             }
             for (DPNTEPsInfo srcDpn : dpnTepsList) {
                 LOG.trace("Processing srcDpn {}", srcDpn);
@@ -68,7 +77,7 @@ public class ItmInternalTunnelDeleteWorker {
                 List<TunnelEndPoints> meshedEndPtCache = ItmUtils.getTEPsForDpn(srcDpn.getDPNID(), meshedDpnList);
                 if (meshedEndPtCache == null) {
                     LOG.debug("No Tunnel End Point configured for this DPN {}", srcDpn.getDPNID());
-                    continue ;
+                    continue;
                 }
                 LOG.debug("Entries in meshEndPointCache {} for DPN Id{} ", meshedEndPtCache.size(), srcDpn.getDPNID());
                 for (TunnelEndPoints srcTep : srcDpn.getTunnelEndPoints()) {
@@ -93,8 +102,8 @@ public class ItmInternalTunnelDeleteWorker {
                                             // remove all trunk interfaces
                                             LOG.trace("Invoking removeTrunkInterface between source TEP {} , "
                                                     + "Destination TEP {} ", srcTep, dstTep);
-                                            removeTrunkInterface(dataBroker, idManagerService, srcTep, dstTep, srcDpn
-                                                    .getDPNID(), dstDpn.getDPNID(), writeTransaction, futures);
+                                            removeTrunkInterface(srcTep, dstTep, srcDpn.getDPNID(), dstDpn.getDPNID(),
+                                                    writeTransaction);
                                         }
                                     }
                                 }
@@ -173,18 +182,12 @@ public class ItmInternalTunnelDeleteWorker {
                     }
                 }
             }
-            futures.add(writeTransaction.submit());
-        } catch (Exception e1) {
-            LOG.error("exception while deleting tep", e1);
-        }
-        return futures ;
+        }));
     }
 
-    private static void removeTrunkInterface(DataBroker dataBroker, IdManagerService idManagerService,
-                                             TunnelEndPoints srcTep, TunnelEndPoints dstTep, BigInteger srcDpnId,
-                                             BigInteger dstDpnId, WriteTransaction transaction,
-                                             List<ListenableFuture<Void>> futures) {
-        String trunkfwdIfName = ItmUtils.getTrunkInterfaceName(idManagerService, srcTep.getInterfaceName(),
+    private void removeTrunkInterface(TunnelEndPoints srcTep, TunnelEndPoints dstTep,
+                                      BigInteger srcDpnId, BigInteger dstDpnId, WriteTransaction transaction) {
+        String trunkfwdIfName = ItmUtils.getTrunkInterfaceName(srcTep.getInterfaceName(),
                 new String(srcTep.getIpAddress().getValue()),
                 new String(dstTep.getIpAddress().getValue()),
                 srcTep.getTunnelType().getName());
@@ -193,20 +196,20 @@ public class ItmInternalTunnelDeleteWorker {
         LOG.debug(" Removing Trunk Interface Name - {} , Id - {} from Config DS ",
                 trunkfwdIfName, trunkIdentifier) ;
         transaction.delete(LogicalDatastoreType.CONFIGURATION, trunkIdentifier);
-        ItmUtils.itmCache.removeInterface(trunkfwdIfName);
+        ItmUtils.ITM_CACHE.removeInterface(trunkfwdIfName);
         // also update itm-state ds -- Delete the forward tunnel-interface from the tunnel list
         InstanceIdentifier<InternalTunnel> path = InstanceIdentifier.create(TunnelList.class)
                 .child(InternalTunnel.class, new InternalTunnelKey(dstDpnId, srcDpnId, srcTep.getTunnelType()));
         transaction.delete(LogicalDatastoreType.CONFIGURATION,path) ;
-        ItmUtils.itmCache.removeInternalTunnel(trunkfwdIfName);
+        ItmUtils.ITM_CACHE.removeInternalTunnel(trunkfwdIfName);
         // Release the Ids for the forward trunk interface Name
-        ItmUtils.releaseIdForTrunkInterfaceName(idManagerService,srcTep.getInterfaceName(),
+        ItmUtils.releaseIdForTrunkInterfaceName(srcTep.getInterfaceName(),
                 new String(srcTep.getIpAddress().getValue()),
                 new String(dstTep.getIpAddress().getValue()),
                 srcTep.getTunnelType().getName());
-        removeLogicalGroupTunnel(srcDpnId, dstDpnId, dataBroker);
+        removeLogicalGroupTunnel(srcDpnId, dstDpnId);
 
-        String trunkRevIfName = ItmUtils.getTrunkInterfaceName(idManagerService, dstTep.getInterfaceName(),
+        String trunkRevIfName = ItmUtils.getTrunkInterfaceName(dstTep.getInterfaceName(),
                 new String(dstTep.getIpAddress().getValue()),
                 new String(srcTep.getIpAddress().getValue()),
                 srcTep.getTunnelType().getName());
@@ -215,18 +218,18 @@ public class ItmInternalTunnelDeleteWorker {
         LOG.debug(" Removing Trunk Interface Name - {} , Id - {} from Config DS ",
                 trunkRevIfName, trunkIdentifier) ;
         transaction.delete(LogicalDatastoreType.CONFIGURATION, trunkIdentifier);
-        ItmUtils.itmCache.removeInternalTunnel(trunkRevIfName);
+        ItmUtils.ITM_CACHE.removeInternalTunnel(trunkRevIfName);
         // also update itm-state ds -- Delete the reverse tunnel-interface from the tunnel list
         path = InstanceIdentifier.create(TunnelList.class)
                 .child(InternalTunnel.class, new InternalTunnelKey(srcDpnId, dstDpnId, dstTep.getTunnelType()));
         transaction.delete(LogicalDatastoreType.CONFIGURATION,path) ;
 
         // Release the Ids for the reverse trunk interface Name
-        ItmUtils.releaseIdForTrunkInterfaceName(idManagerService, dstTep.getInterfaceName(),
+        ItmUtils.releaseIdForTrunkInterfaceName(dstTep.getInterfaceName(),
                 new String(dstTep.getIpAddress().getValue()),
                 new String(srcTep.getIpAddress().getValue()),
                 dstTep.getTunnelType().getName());
-        removeLogicalGroupTunnel(dstDpnId, srcDpnId, dataBroker);
+        removeLogicalGroupTunnel(dstDpnId, srcDpnId);
     }
 
     private static boolean checkIfTrunkExists(BigInteger srcDpnId, BigInteger dstDpnId,
@@ -236,17 +239,15 @@ public class ItmInternalTunnelDeleteWorker {
         return ItmUtils.read(LogicalDatastoreType.CONFIGURATION,path, dataBroker).isPresent();
     }
 
-    private static void removeLogicalGroupTunnel(BigInteger srcDpnId, BigInteger dstDpnId,
-                                                 DataBroker dataBroker) {
+    private void removeLogicalGroupTunnel(BigInteger srcDpnId, BigInteger dstDpnId) {
         boolean tunnelAggregationEnabled = ItmTunnelAggregationHelper.isTunnelAggregationEnabled();
         if (!tunnelAggregationEnabled) {
             return;
         }
         String logicTunnelName = ItmUtils.getLogicalTunnelGroupName(srcDpnId, dstDpnId);
-        DataStoreJobCoordinator coordinator = DataStoreJobCoordinator.getInstance();
         ItmTunnelAggregationDeleteWorker addWorker =
                 new ItmTunnelAggregationDeleteWorker(logicTunnelName, srcDpnId, dstDpnId, dataBroker);
-        coordinator.enqueueJob(logicTunnelName, addWorker);
+        jobCoordinator.enqueueJob(logicTunnelName, addWorker);
     }
 
     private static class ItmTunnelAggregationDeleteWorker implements Callable<List<ListenableFuture<Void>>> {
@@ -254,18 +255,18 @@ public class ItmInternalTunnelDeleteWorker {
         private final String logicTunnelName;
         private final BigInteger srcDpnId;
         private final BigInteger dstDpnId;
-        private final DataBroker dataBroker;
+        private final ManagedNewTransactionRunner txRunner;
 
         ItmTunnelAggregationDeleteWorker(String groupName, BigInteger srcDpnId, BigInteger dstDpnId, DataBroker db) {
             this.logicTunnelName = groupName;
             this.srcDpnId = srcDpnId;
             this.dstDpnId = dstDpnId;
-            this.dataBroker = db;
+            this.txRunner = new ManagedNewTransactionRunnerImpl(db);
         }
 
         @Override
         public List<ListenableFuture<Void>> call() throws Exception {
-            Collection<InternalTunnel> tunnels = ItmUtils.itmCache.getAllInternalTunnel();
+            Collection<InternalTunnel> tunnels = ItmUtils.ITM_CACHE.getAllInternalTunnel();
             if (tunnels == null) {
                 return Collections.emptyList();
             }
@@ -284,18 +285,18 @@ public class ItmInternalTunnelDeleteWorker {
                 }
             }
             if (emptyTunnelGroup && foundLogicGroupIface) {
-                WriteTransaction tx = dataBroker.newWriteOnlyTransaction();
-                LOG.debug("MULTIPLE_VxLAN_TUNNELS: remove the logical tunnel group {} because a last tunnel"
-                    + " interface on srcDpnId {} dstDpnId {} is removed", logicTunnelName, srcDpnId, dstDpnId);
-                InstanceIdentifier<Interface> trunkIdentifier = ItmUtils.buildId(logicTunnelName);
-                tx.delete(LogicalDatastoreType.CONFIGURATION, trunkIdentifier);
-                ItmUtils.itmCache.removeInterface(logicTunnelName);
-                InstanceIdentifier<InternalTunnel> path = InstanceIdentifier.create(TunnelList.class)
-                        .child(InternalTunnel.class,
-                                new InternalTunnelKey(dstDpnId, srcDpnId, TunnelTypeLogicalGroup.class));
-                tx.delete(LogicalDatastoreType.CONFIGURATION, path);
-                ItmUtils.itmCache.removeInternalTunnel(logicTunnelName);
-                return Collections.singletonList(tx.submit());
+                return singletonList(txRunner.callWithNewWriteOnlyTransactionAndSubmit(tx -> {
+                    LOG.debug("MULTIPLE_VxLAN_TUNNELS: remove the logical tunnel group {} because a last tunnel"
+                        + " interface on srcDpnId {} dstDpnId {} is removed", logicTunnelName, srcDpnId, dstDpnId);
+                    InstanceIdentifier<Interface> trunkIdentifier = ItmUtils.buildId(logicTunnelName);
+                    tx.delete(LogicalDatastoreType.CONFIGURATION, trunkIdentifier);
+                    ItmUtils.ITM_CACHE.removeInterface(logicTunnelName);
+                    InstanceIdentifier<InternalTunnel> path = InstanceIdentifier.create(TunnelList.class)
+                            .child(InternalTunnel.class,
+                                    new InternalTunnelKey(dstDpnId, srcDpnId, TunnelTypeLogicalGroup.class));
+                    tx.delete(LogicalDatastoreType.CONFIGURATION, path);
+                    ItmUtils.ITM_CACHE.removeInternalTunnel(logicTunnelName);
+                }));
             } else if (!emptyTunnelGroup) {
                 LOG.debug("MULTIPLE_VxLAN_TUNNELS: not last tunnel in logical tunnel group {}", logicTunnelName);
             }