Remove unused parameters 28/63228/18
authorStephen Kitt <skitt@redhat.com>
Mon, 18 Sep 2017 09:43:35 +0000 (11:43 +0200)
committerSam Hague <shague@redhat.com>
Sun, 25 Feb 2018 05:05:06 +0000 (00:05 -0500)
This patch removes unused parameters (except where they’re obviously
WIP).

Change-Id: I9e9e741c4c10df347ccb95099d7554ad53dd241f
Signed-off-by: Stephen Kitt <skitt@redhat.com>
Signed-off-by: Sam Hague <shague@redhat.com>
106 files changed:
aclservice/impl/src/main/java/org/opendaylight/netvirt/aclservice/EgressAclServiceImpl.java
aclservice/impl/src/main/java/org/opendaylight/netvirt/aclservice/IngressAclServiceImpl.java
aclservice/impl/src/main/java/org/opendaylight/netvirt/aclservice/utils/AclServiceUtils.java
bgpmanager/api/src/main/java/org/opendaylight/netvirt/bgpmanager/api/IBgpManager.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpConfigurationManager.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpManager.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/BgpUtil.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/ConfigureBgpCli.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/FibDSWriter.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/VtyshCli.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/oam/BgpAlarmBroadcaster.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/oam/BgpAlarmBroadcasterMBean.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/thrift/client/BgpRouter.java
bgpmanager/impl/src/main/java/org/opendaylight/netvirt/bgpmanager/thrift/server/BgpThriftService.java
bgpmanager/impl/src/test/java/org/opendaylight/netvirt/bgpmanager/test/BgpManagerTest.java
cloud-servicechain/api/src/main/java/org/opendaylight/netvirt/cloudservicechain/api/ICloudServiceChain.java
cloud-servicechain/impl/src/main/java/org/opendaylight/netvirt/cloudservicechain/CloudServiceChainProvider.java
coe/impl/src/main/java/org/opendaylight/netvirt/coe/listeners/PodListener.java
coe/impl/src/main/java/org/opendaylight/netvirt/coe/utils/CoeUtils.java
dhcpservice/impl/src/main/java/org/opendaylight/netvirt/dhcpservice/DhcpExternalTunnelManager.java
dhcpservice/impl/src/main/java/org/opendaylight/netvirt/dhcpservice/DhcpPktHandler.java
elanmanager/api/src/main/java/org/opendaylight/netvirt/elanmanager/api/IElanService.java
elanmanager/api/src/main/java/org/opendaylight/netvirt/elanmanager/api/IEtreeService.java
elanmanager/api/src/main/java/org/opendaylight/netvirt/elanmanager/api/IL2gwService.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/cli/ElanInterfaceDelete.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/cli/StaticMacAdd.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/cli/StaticMacDelete.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/cli/etree/EtreeInterfaceDelete.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/cli/l2gw/L2GwValidateCli.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/diagstatus/ElanDiagStatusProvider.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/internal/ElanBridgeManager.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/internal/ElanDpnToTransportZoneListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/internal/ElanServiceProvider.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/internal/VpnDpnToTransportZoneListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/HwvtepHAUtil.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/handlers/ConfigNodeUpdatedHandler.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/handlers/HAEventHandler.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/handlers/IHAEventHandler.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/handlers/OpNodeUpdatedHandler.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/listeners/HAConfigNodeListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/listeners/HAOpClusteredListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/listeners/HAOpNodeListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/ha/listeners/HwvtepNodeBaseListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/jobs/DisAssociateHwvtepFromElanJob.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/jobs/LogicalSwitchAddedJob.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/listeners/ChildListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/listeners/HwvtepRemoteMcastMacListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/listeners/HwvtepTerminationPointListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/listeners/LocalUcastMacListener.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/utils/ElanL2GatewayMulticastUtils.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/utils/ElanL2GatewayUtils.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/utils/L2GatewayConnectionUtils.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/l2gw/utils/L2gwServiceProvider.java
elanmanager/impl/src/main/java/org/opendaylight/netvirt/elan/utils/ElanDmacUtils.java
fibmanager/api/src/main/java/org/opendaylight/netvirt/fibmanager/api/IFibManager.java
fibmanager/impl/src/main/java/org/opendaylight/netvirt/fibmanager/BgpRouteVrfEntryHandler.java
fibmanager/impl/src/main/java/org/opendaylight/netvirt/fibmanager/FibManagerImpl.java
fibmanager/impl/src/main/java/org/opendaylight/netvirt/fibmanager/VrfEntryListener.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/AbstractSnatService.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/ConntrackBasedSnatService.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/EvpnDnatFlowProgrammer.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/ExternalNetworksChangeListener.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/ExternalRoutersListener.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/FloatingIPListener.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/NaptSwitchHA.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/NatEvpnUtil.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/NatTunnelInterfaceStateListener.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/NatUtil.java
natservice/impl/src/main/java/org/opendaylight/netvirt/natservice/internal/VpnFloatingIpHandler.java
neutronvpn/impl/src/main/java/org/opendaylight/netvirt/neutronvpn/NeutronPortChangeListener.java
neutronvpn/impl/src/main/java/org/opendaylight/netvirt/neutronvpn/NeutronvpnNatManager.java
neutronvpn/impl/src/main/java/org/opendaylight/netvirt/neutronvpn/l2gw/L2GatewayListener.java
policyservice/impl/src/main/java/org/opendaylight/netvirt/policyservice/listeners/UnderlayNetworkDpnListener.java
sfc/translator/src/main/java/org/opendaylight/netvirt/sfc/translator/DelegatingDataTreeListener.java
sfc/translator/src/main/java/org/opendaylight/netvirt/sfc/translator/INeutronSfcDataProcessor.java
sfc/translator/src/main/java/org/opendaylight/netvirt/sfc/translator/flowclassifier/NeutronFlowClassifierListener.java
sfc/translator/src/main/java/org/opendaylight/netvirt/sfc/translator/portchain/NeutronPortChainListener.java
sfc/translator/src/main/java/org/opendaylight/netvirt/sfc/translator/portchain/NeutronPortPairGroupListener.java
sfc/translator/src/main/java/org/opendaylight/netvirt/sfc/translator/portchain/NeutronPortPairListener.java
statistics/impl/src/main/java/org/opendaylight/netvirt/statistics/CounterRetriever.java
statistics/impl/src/main/java/org/opendaylight/netvirt/statistics/CountersServiceUtils.java
statistics/impl/src/main/java/org/opendaylight/netvirt/statistics/StatisticsImpl.java
vpnmanager/api/src/main/java/org/opendaylight/netvirt/vpnmanager/api/IVpnManager.java
vpnmanager/api/src/main/java/org/opendaylight/netvirt/vpnmanager/api/intervpnlink/IVpnLinkService.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/AlivenessMonitorUtils.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/ArpMonitorStartTask.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/ArpMonitoringHandler.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/CentralizedSwitchChangeListener.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/SubnetRoutePacketInHandler.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/TransactionUtil.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/TunnelInterfaceStateListener.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnInstanceListener.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnInterfaceManager.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnManagerImpl.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnOpStatusListener.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnRpcServiceImpl.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnSubnetRouteHandler.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/VpnUtil.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/arp/responder/ArpResponderHandler.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/intervpnlink/IVpnLinkServiceImpl.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/intervpnlink/InterVpnLinkNodeAddTask.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/intervpnlink/InterVpnLinkUtil.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/populator/impl/L3vpnOverMplsGrePopulator.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/populator/impl/L3vpnOverVxlanPopulator.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/populator/impl/L3vpnPopulator.java
vpnmanager/impl/src/main/java/org/opendaylight/netvirt/vpnmanager/populator/intfc/VpnPopulator.java

index 4aa4881d2f1a34edf86a13e678bba93ada8b1acd..5444fead7bf251173072b0ecb587fa90cbb62513 100644 (file)
@@ -137,7 +137,7 @@ public class EgressAclServiceImpl extends AbstractAclServiceImpl {
     protected void programRemoteAclTableFlow(BigInteger dpId, Integer aclTag, AllowedAddressPairs aap,
             int addOrRemove) {
         List<MatchInfoBase> flowMatches = new ArrayList<>();
-        flowMatches.addAll(AclServiceUtils.buildIpAndDstServiceMatch(aclTag, aap, dataBroker));
+        flowMatches.addAll(AclServiceUtils.buildIpAndDstServiceMatch(aclTag, aap));
 
         List<InstructionInfo> instructions = AclServiceOFFlowBuilder.getGotoInstructionInfo(getAclCommitterTable());
         String flowNameAdded = "Acl_Filter_Egress_" + String.valueOf(aap.getIpAddress().getValue()) + "_" + aclTag;
index c73eeb1e0e177891ce28e48d8902a2e5c61a825c..6bacbc81d965911d0572fb7a09d75e59a2c6bb39 100644 (file)
@@ -127,8 +127,8 @@ public class IngressAclServiceImpl extends AbstractAclServiceImpl {
         BigInteger dpid = port.getDpId();
         int lportTag = port.getLPortTag();
         if (action == Action.ADD || action == Action.REMOVE) {
-            ingressAclDhcpAllowServerTraffic(dpid, lportTag, addOrRemove, AclConstants.PROTO_PREFIX_MATCH_PRIORITY);
-            ingressAclDhcpv6AllowServerTraffic(dpid, lportTag, addOrRemove, AclConstants.PROTO_PREFIX_MATCH_PRIORITY);
+            ingressAclDhcpAllowServerTraffic(dpid, lportTag, addOrRemove);
+            ingressAclDhcpv6AllowServerTraffic(dpid, lportTag, addOrRemove);
             ingressAclIcmpv6AllowedTraffic(dpid, lportTag, addOrRemove);
 
             programArpRule(dpid, lportTag, addOrRemove);
@@ -162,7 +162,7 @@ public class IngressAclServiceImpl extends AbstractAclServiceImpl {
     protected void programRemoteAclTableFlow(BigInteger dpId, Integer aclTag, AllowedAddressPairs aap,
             int addOrRemove) {
         List<MatchInfoBase> flowMatches = new ArrayList<>();
-        flowMatches.addAll(AclServiceUtils.buildIpAndSrcServiceMatch(aclTag, aap, dataBroker));
+        flowMatches.addAll(AclServiceUtils.buildIpAndSrcServiceMatch(aclTag, aap));
 
         List<InstructionInfo> instructions = AclServiceOFFlowBuilder.getGotoInstructionInfo(getAclCommitterTable());
         String flowNameAdded = "Acl_Filter_Ingress_" + String.valueOf(aap.getIpAddress().getValue()) + "_" + aclTag;
@@ -178,10 +178,8 @@ public class IngressAclServiceImpl extends AbstractAclServiceImpl {
      * @param dpId the dpid
      * @param lportTag the lport tag
      * @param addOrRemove is write or delete
-     * @param protoPortMatchPriority the priority
      */
-    protected void ingressAclDhcpAllowServerTraffic(BigInteger dpId, int lportTag, int addOrRemove,
-            int protoPortMatchPriority) {
+    protected void ingressAclDhcpAllowServerTraffic(BigInteger dpId, int lportTag, int addOrRemove) {
         final List<MatchInfoBase> matches = AclServiceUtils.buildDhcpMatches(AclConstants.DHCP_SERVER_PORT_IPV4,
                 AclConstants.DHCP_CLIENT_PORT_IPV4, lportTag, serviceMode);
         List<InstructionInfo> instructions = getDispatcherTableResubmitInstructions();
@@ -198,10 +196,8 @@ public class IngressAclServiceImpl extends AbstractAclServiceImpl {
      * @param dpId the dpid
      * @param lportTag the lport tag
      * @param addOrRemove is write or delete
-     * @param protoPortMatchPriority the priority
      */
-    protected void ingressAclDhcpv6AllowServerTraffic(BigInteger dpId, int lportTag, int addOrRemove,
-            Integer protoPortMatchPriority) {
+    protected void ingressAclDhcpv6AllowServerTraffic(BigInteger dpId, int lportTag, int addOrRemove) {
         final List<MatchInfoBase> matches = AclServiceUtils.buildDhcpV6Matches(AclConstants.DHCP_SERVER_PORT_IPV6,
                 AclConstants.DHCP_CLIENT_PORT_IPV6, lportTag, serviceMode);
         List<InstructionInfo> instructions = getDispatcherTableResubmitInstructions();
index 4f8b7c5a68a316861d5bda4b7ea8c4d5114846a1..00fe3e6354921b8b4bf2b462b22a850c42c8d029 100644 (file)
@@ -1027,8 +1027,7 @@ public final class AclServiceUtils {
         deleteIdPool(AclConstants.ACL_TAG_POOL_NAME);
     }
 
-    public static List<? extends MatchInfoBase> buildIpAndSrcServiceMatch(Integer aclTag, AllowedAddressPairs aap,
-            DataBroker dataBroker) {
+    public static List<? extends MatchInfoBase> buildIpAndSrcServiceMatch(Integer aclTag, AllowedAddressPairs aap) {
         List<MatchInfoBase> flowMatches = new ArrayList<>();
         flowMatches.add(buildRemoteAclTagMetadataMatch(aclTag));
         if (aap.getIpAddress().getIpAddress() != null) {
@@ -1061,8 +1060,7 @@ public final class AclServiceUtils {
         return flowMatches;
     }
 
-    public static List<? extends MatchInfoBase> buildIpAndDstServiceMatch(Integer aclTag, AllowedAddressPairs aap,
-            DataBroker dataBroker) {
+    public static List<? extends MatchInfoBase> buildIpAndDstServiceMatch(Integer aclTag, AllowedAddressPairs aap) {
         List<MatchInfoBase> flowMatches = new ArrayList<>();
         flowMatches.add(buildRemoteAclTagMetadataMatch(aclTag));
 
index 41b3374309c2a6cc426e47c3732148593fa654c1..5c00826267e18d971e2d7e404338e434b51b3d92 100644 (file)
@@ -76,7 +76,7 @@ public interface IBgpManager {
 
     String getDCGwIP();
 
-    void sendNotificationEvent(String pfx, int code, int subcode);
+    void sendNotificationEvent(int code, int subcode);
 
     void setQbgprestartTS(long qbgprestartTS);
 
index b03a46b4bd9fe0f8954b80548af1a6dd16216320..92632f352536f288527259e14146918fd9b13dc9 100755 (executable)
@@ -1207,8 +1207,8 @@ public class BgpConfigurationManager {
                     List<AddressFamiliesVrf> vrfAddrFamilyList = vrfs.getAddressFamiliesVrf();
                     for (AddressFamiliesVrf vrfAddrFamily : vrfAddrFamilyList) {
                         /*add to br the new vrfs arguments*/
-                        br.addVrf(BgpUtil.getLayerType(vrfAddrFamily), rd, vrfs.getImportRts(), vrfs.getExportRts(),
-                                vrfAddrFamily.getAfi(), vrfAddrFamily.getSafi());
+                        br.addVrf(BgpUtil.getLayerType(vrfAddrFamily), rd, vrfs.getImportRts(), vrfs.getExportRts()
+                        );
                     }
                     /*add to br the vrfs contained in mapNewAdFamily*/
                     List<AddressFamiliesVrf> vrfAddrFamilyListFromMap = mapNewAdFamily.get(rd);
@@ -1221,8 +1221,8 @@ public class BgpConfigurationManager {
                             mapNewAdFamily.remove(rd);
                         } else  if (adf != null) {
 
-                            br.addVrf(BgpUtil.getLayerType(adf), rd, vrfs.getImportRts(), vrfs.getExportRts(),
-                                    adf.getAfi(), adf.getSafi());
+                            br.addVrf(BgpUtil.getLayerType(adf), rd, vrfs.getImportRts(), vrfs.getExportRts()
+                            );
                             // remove AddressFamiliesVrf which was already added to BGP
                             vrfAddrFamilyListFromMap.remove(adf);
                             if (vrfAddrFamilyListFromMap.isEmpty()) {
@@ -1329,7 +1329,7 @@ public class BgpConfigurationManager {
                     try {
                         LOG.debug("call addVRf rd {} afi {} safi {}", rd, adfvrf.getAfi(), adfvrf.getSafi());
                         br.addVrf(BgpUtil.getLayerType(adfvrf), rd, newval.getImportRts(),
-                                newval.getExportRts(), adfvrf.getAfi(), adfvrf.getSafi());
+                                newval.getExportRts());
                     } catch (TException | BgpRouterException e) {
                         LOG.error("{} Add received exception; {}", YANG_OBJ, ADD_WARN, e);
                     }
@@ -1731,8 +1731,6 @@ public class BgpConfigurationManager {
                            prefix,
                            plen,
                            nexthop,
-                           update.getEthtag(),
-                           update.getEsi(),
                            update.getMacaddress(),
                            label,
                            l2label,
@@ -1780,7 +1778,7 @@ public class BgpConfigurationManager {
      */
 
     public void onUpdatePushRoute(protocol_type protocolType, String rd, String prefix, int plen, String nextHop,
-            int ethtag, String esi, String macaddress, int label, int l2label, String routermac, af_afi afi) {
+                                  String macaddress, int label, int l2label, String routermac, af_afi afi) {
         boolean addroute = false;
         boolean macupdate = false;
         long l3vni = 0L;
@@ -1838,7 +1836,7 @@ public class BgpConfigurationManager {
                     rd, prefix, nextHop, label, afi);
             // TODO: modify addFibEntryToDS signature
             List<String> nextHopList = Collections.singletonList(nextHop);
-            fibDSWriter.addFibEntryToDS(rd, macaddress, prefix + "/" + plen, nextHopList, encapType, label, l3vni,
+            fibDSWriter.addFibEntryToDS(rd, prefix + "/" + plen, nextHopList, encapType, label, l3vni,
                                         routermac, RouteOrigin.BGP);
             LOG.info("ADD: Added Fib entry rd {} prefix {} nexthop {} label {}", rd, prefix, nextHop, label);
             String vpnName = bgpUtil.getVpnNameFromRd(rd);
@@ -2191,7 +2189,7 @@ public class BgpConfigurationManager {
             for (AddressFamiliesVrf adf : vrf.getAddressFamiliesVrf()) {
                 try {
                     br.addVrf(BgpUtil.getLayerType(adf), vrf.getRd(), vrf.getImportRts(),
-                            vrf.getExportRts(), adf.getAfi(), adf.getSafi());
+                            vrf.getExportRts());
                 } catch (TException | BgpRouterException e) {
                     LOG.error("Replay:addVrf() received exception", e);
                 }
@@ -2282,11 +2280,11 @@ public class BgpConfigurationManager {
     }
 
     private <T extends DataObject> void update(InstanceIdentifier<T> iid, T dto) {
-        bgpUtil.update(LogicalDatastoreType.CONFIGURATION, iid, dto);
+        bgpUtil.update(iid, dto);
     }
 
     private <T extends DataObject> void delete(InstanceIdentifier<T> iid) {
-        bgpUtil.delete(LogicalDatastoreType.CONFIGURATION, iid);
+        bgpUtil.delete(iid);
     }
 
     public void startConfig(String bgpHost, int thriftPort) {
index 36f3031f4aa5a7acdf8ed98befe2388f697de5e1..929b2456ce5f9281e3f3a2970a45ffc3f32ae142 100644 (file)
@@ -112,7 +112,7 @@ public class BgpManager implements AutoCloseable, IBgpManager {
     public void addPrefix(String rd, String macAddress, String prefix, List<String> nextHopList,
                           VrfEntry.EncapType encapType, int vpnLabel, long l3vni,
                           String gatewayMac, RouteOrigin origin) {
-        fibDSWriter.addFibEntryToDS(rd, macAddress, prefix, nextHopList,
+        fibDSWriter.addFibEntryToDS(rd, prefix, nextHopList,
                 encapType, vpnLabel, l3vni, gatewayMac, origin);
         bcm.addPrefix(rd, macAddress, prefix, nextHopList,
                 encapType, vpnLabel, l3vni, 0 /*l2vni*/, gatewayMac);
@@ -220,7 +220,7 @@ public class BgpManager implements AutoCloseable, IBgpManager {
     @Override
     // This method doesn't actually do any real work currently but may at some point so suppress FindBugs violation.
     @SuppressFBWarnings("UC_USELESS_VOID_METHOD")
-    public synchronized void sendNotificationEvent(String pfx, int code, int subcode) {
+    public synchronized void sendNotificationEvent(int code, int subcode) {
         if (code != BgpConstants.BGP_NOTIFY_CEASE_CODE) {
             // CEASE Notifications alone have to be reported to the CBA.
             // Silently return here. No need to log because tons
index c6cce539d1fa74aa07c48ca7652949419ede3af3..d3c1bf583a5027ae8396415bed531d288623074a 100755 (executable)
@@ -121,8 +121,7 @@ public class BgpUtil implements AutoCloseable {
         return retValue;
     }
 
-    public <T extends DataObject> void update(final LogicalDatastoreType datastoreType,
-            final InstanceIdentifier<T> path, final T data) {
+    public <T extends DataObject> void update(final InstanceIdentifier<T> path, final T data) {
         ActionableResource actResource = new ActionableResourceImpl(path.toString());
         actResource.setAction(ActionableResource.UPDATE);
         actResource.setInstanceIdentifier(path);
@@ -130,8 +129,7 @@ public class BgpUtil implements AutoCloseable {
         bgpResourcesBufferQ.add(actResource);
     }
 
-    public <T extends DataObject> void write(final LogicalDatastoreType datastoreType, final InstanceIdentifier<T> path,
-            final T data) {
+    public <T extends DataObject> void write(final InstanceIdentifier<T> path, final T data) {
         ActionableResource actResource = new ActionableResourceImpl(path.toString());
         actResource.setAction(ActionableResource.CREATE);
         actResource.setInstanceIdentifier(path);
@@ -139,8 +137,7 @@ public class BgpUtil implements AutoCloseable {
         bgpResourcesBufferQ.add(actResource);
     }
 
-    public <T extends DataObject> void delete(final LogicalDatastoreType datastoreType,
-            final InstanceIdentifier<T> path) {
+    public <T extends DataObject> void delete(final InstanceIdentifier<T> path) {
         ActionableResource actResource = new ActionableResourceImpl(path.toString());
         actResource.setAction(ActionableResource.DELETE);
         actResource.setInstanceIdentifier(path);
@@ -225,7 +222,7 @@ public class BgpUtil implements AutoCloseable {
         ExternalTepsKey externalTepsKey = externalTepsId.firstKeyOf(ExternalTeps.class);
         externalTepsBuilder.setKey(externalTepsKey);
         externalTepsBuilder.setTepIp(externalTepsKey.getTepIp());
-        update(LogicalDatastoreType.CONFIGURATION, externalTepsId, externalTepsBuilder.build());
+        update(externalTepsId, externalTepsBuilder.build());
     }
 
     public void deleteTepFromElanInstance(String rd, String tepIp) {
@@ -240,7 +237,7 @@ public class BgpUtil implements AutoCloseable {
         }
         LOG.debug("Deleting tepIp {} from elan {}", tepIp, elanName);
         InstanceIdentifier<ExternalTeps> externalTepsId = getExternalTepsIdentifier(elanName, tepIp);
-        delete(LogicalDatastoreType.CONFIGURATION, externalTepsId);
+        delete(externalTepsId);
     }
 
     private static InstanceIdentifier<ExternalTeps> getExternalTepsIdentifier(String elanInstanceName, String tepIp) {
@@ -290,6 +287,6 @@ public class BgpUtil implements AutoCloseable {
                    InstanceIdentifier.builder(FibEntries.class)
                    .child(VrfTables.class, new VrfTablesKey(rd))
                    .child(VrfEntry.class, new VrfEntryKey(vrfEntry.getDestPrefix())).build();
-        delete(LogicalDatastoreType.CONFIGURATION, vrfEntryId);
+        delete(vrfEntryId);
     }
 }
index a605b3e25de50c1f1c7af685f4dc2bc0c781d9b2..b73a6c64c59b35ab718b5dc8d5cde233029f04dd 100644 (file)
@@ -367,7 +367,7 @@ public class ConfigureBgpCli extends OsgiCommandSupport {
 
     protected void addRoute() {
         bgpConfigurationManager.onUpdatePushRoute(protocol_type.PROTOCOL_EVPN, rd, prefix,
-                0, nexthop, 0, null, mac, l3vni, l2vni, null, null);
+                0, nexthop, mac, l3vni, l2vni, null, null);
     }
 
     protected void deleteRoute() {
index f102fd6c274b41a32356e7bd0a84322053e542d4..f2eea91ef243ab997242018956bae0c26702c2f7 100644 (file)
@@ -41,20 +41,18 @@ import org.slf4j.LoggerFactory;
 @Singleton
 public class FibDSWriter {
     private static final Logger LOG = LoggerFactory.getLogger(FibDSWriter.class);
-    private final DataBroker dataBroker;
     private final SingleTransactionDataBroker singleTxDB;
     private final BgpUtil bgpUtil;
 
     @Inject
     public FibDSWriter(final DataBroker dataBroker, final BgpUtil bgpUtil) {
-        this.dataBroker = dataBroker;
         this.bgpUtil = bgpUtil;
         this.singleTxDB = new SingleTransactionDataBroker(dataBroker);
     }
 
-    public synchronized void addFibEntryToDS(String rd, String macAddress, String prefix, List<String> nextHopList,
-                                             VrfEntry.EncapType encapType, int label, long l3vni,
-                                             String gatewayMacAddress, RouteOrigin origin) {
+    public synchronized void addFibEntryToDS(String rd, String prefix, List<String> nextHopList,
+            VrfEntry.EncapType encapType, int label, long l3vni,
+            String gatewayMacAddress, RouteOrigin origin) {
         if (rd == null || rd.isEmpty()) {
             LOG.error("Prefix {} not associated with vpn", prefix);
             return;
@@ -76,9 +74,9 @@ public class FibDSWriter {
                         .child(VrfEntry.class, new VrfEntryKey(prefix)).build();
 
         VrfEntryBuilder vrfEntryBuilder = new VrfEntryBuilder().setDestPrefix(prefix).setOrigin(origin.getValue());
-        buildVpnEncapSpecificInfo(vrfEntryBuilder, encapType, label, l3vni, macAddress,
+        buildVpnEncapSpecificInfo(vrfEntryBuilder, encapType, label, l3vni,
                 gatewayMacAddress, nextHopList);
-        bgpUtil.update(LogicalDatastoreType.CONFIGURATION, vrfEntryId, vrfEntryBuilder.build());
+        bgpUtil.update(vrfEntryId, vrfEntryBuilder.build());
     }
 
     public void addMacEntryToDS(String rd, String macAddress, String prefix,
@@ -98,7 +96,7 @@ public class FibDSWriter {
         }
 
         MacVrfEntryBuilder macEntryBuilder = new MacVrfEntryBuilder().setOrigin(origin.getValue());
-        buildVpnEncapSpecificInfo(macEntryBuilder, encapType, l2vni, macAddress,
+        buildVpnEncapSpecificInfo(macEntryBuilder, encapType, l2vni,
                 gatewayMacAddress, nextHopList);
         macEntryBuilder.setMac(macAddress);
         macEntryBuilder.setDestPrefix(prefix);
@@ -106,11 +104,11 @@ public class FibDSWriter {
                 InstanceIdentifier.builder(FibEntries.class)
                         .child(VrfTables.class, new VrfTablesKey(rd))
                         .child(MacVrfEntry.class, new MacVrfEntryKey(macAddress)).build();
-        bgpUtil.update(LogicalDatastoreType.CONFIGURATION, macEntryId, macEntryBuilder.build());
+        bgpUtil.update(macEntryId, macEntryBuilder.build());
     }
 
     private static void buildVpnEncapSpecificInfo(VrfEntryBuilder builder,
-            VrfEntry.EncapType encapType, long label, long l3vni, String macAddress,
+            VrfEntry.EncapType encapType, long label, long l3vni,
             String gatewayMac, List<String> nextHopList) {
         if (!encapType.equals(VrfEntry.EncapType.Mplsgre)) {
             builder.setL3vni(l3vni);
@@ -125,7 +123,7 @@ public class FibDSWriter {
     }
 
     private static void buildVpnEncapSpecificInfo(MacVrfEntryBuilder builder,
-                                                  VrfEntry.EncapType encapType, long l2vni, String macAddress,
+                                                  VrfEntry.EncapType encapType, long l2vni,
                                                   String gatewayMac, List<String> nextHopList) {
         builder.setEncapType(encapType);
         builder.setGatewayMacAddress(gatewayMac);
@@ -148,7 +146,7 @@ public class FibDSWriter {
                 InstanceIdentifier.builder(FibEntries.class).child(VrfTables.class, new VrfTablesKey(rd)).child(
                         VrfEntry.class, new VrfEntryKey(prefix));
         InstanceIdentifier<VrfEntry> vrfEntryId = idBuilder.build();
-        bgpUtil.delete(LogicalDatastoreType.CONFIGURATION, vrfEntryId);
+        bgpUtil.delete(vrfEntryId);
 
     }
 
@@ -164,7 +162,7 @@ public class FibDSWriter {
                 InstanceIdentifier.builder(FibEntries.class).child(VrfTables.class, new VrfTablesKey(rd)).child(
                         MacVrfEntry.class, new MacVrfEntryKey(macAddress));
         InstanceIdentifier<MacVrfEntry> macEntryId = idBuilder.build();
-        bgpUtil.delete(LogicalDatastoreType.CONFIGURATION, macEntryId);
+        bgpUtil.delete(macEntryId);
 
     }
 
@@ -187,7 +185,7 @@ public class FibDSWriter {
                     existingVrfEntry.toJavaUtil().map(VrfEntry::getRoutePaths).orElse(Collections.emptyList());
             if (routePaths.size() == 1) {
                 if (routePaths.get(0).getNexthopAddress().equals(nextHop)) {
-                    bgpUtil.delete(LogicalDatastoreType.CONFIGURATION, vrfEntryId);
+                    bgpUtil.delete(vrfEntryId);
                 }
             } else {
                 routePaths.stream()
@@ -197,7 +195,7 @@ public class FibDSWriter {
                     .ifPresent(nh -> {
                         InstanceIdentifier<RoutePaths> routePathId =
                                 FibHelper.buildRoutePathId(rd, prefix, nextHop);
-                        bgpUtil.delete(LogicalDatastoreType.CONFIGURATION, routePathId);
+                        bgpUtil.delete(routePathId);
                     });
             }
         } catch (ReadFailedException e) {
@@ -262,7 +260,7 @@ public class FibDSWriter {
                 InstanceIdentifier.builder(FibEntries.class).child(VrfTables.class, new VrfTablesKey(rd));
         InstanceIdentifier<VrfTables> vrfTableId = idBuilder.build();
 
-        bgpUtil.delete(LogicalDatastoreType.CONFIGURATION, vrfTableId);
+        bgpUtil.delete(vrfTableId);
 
     }
 }
index ca24351cf51ccba4ee36d69a29c4cdde49abcdb0..27106315c7b33a20467687d2a8ff00bca2f9674b 100644 (file)
@@ -96,7 +96,7 @@ public class VtyshCli extends OsgiCommandSupport {
         switch (handlerModule) {
             case BGPD:
                 try {
-                    handleCommand(firstArg, cmd);
+                    handleCommand(cmd);
                 } catch (IOException ioe) {
                     session.getConsole().println("IOException thrown.");
                 }
@@ -115,7 +115,7 @@ public class VtyshCli extends OsgiCommandSupport {
         return serverName;
     }
 
-    public void handleCommand(String arg, String command) throws IOException {
+    public void handleCommand(String command) throws IOException {
         char[] cbuf = new char[10];
         Socket socket;
         PrintWriter outToSocket;
index 7dde3eb0b7195bce7f75b277c22713b8016b4737..d87632a4c8703c02b52d685aaec7746f8d753d9e 100644 (file)
@@ -59,7 +59,7 @@ public class BgpAlarmBroadcaster extends NotificationBroadcasterSupport
     }
 
     @Override
-    public void sendBgpAlarmInfo(String pfx, int code, int subcode) {
+    public void sendBgpAlarmInfo(String pfx, int subcode) {
         BgpAlarmErrorCodes userAlarm = BgpAlarmErrorCodes.checkErrorSubcode(subcode);
         ArrayList<String> arrayList = new ArrayList<>();
         arrayList.add(userAlarm.getAlarmType());
index da98b15e67b7f307379fb1d632dc322e80d0bc99..e813f52fedd5ba3406a0480cadded56b251bd9d5 100644 (file)
@@ -14,6 +14,6 @@ package org.opendaylight.netvirt.bgpmanager.oam;
  */
 public interface BgpAlarmBroadcasterMBean {
 
-    void sendBgpAlarmInfo(String pfx, int code , int subcode);
+    void sendBgpAlarmInfo(String pfx, int subcode);
 
 }
index 116f427aea60de7fed7464737d431ff27d8db47d..b0a0f8b96815128aaaeb46c674f8989727df9907 100644 (file)
@@ -399,8 +399,8 @@ public final class BgpRouter {
         dispatch(bop);
     }
 
-    public synchronized void addVrf(LayerType layerType, String rd, List<String> irts, List<String> erts,
-            long afi, long safi) throws TException, BgpRouterException {
+    public synchronized void addVrf(LayerType layerType, String rd, List<String> irts, List<String> erts)
+            throws TException, BgpRouterException {
         bop.thriftLayerType = layerType == LayerType.LAYER2 ? layer_type.LAYER_2 : layer_type.LAYER_3;
         bop.type = Optype.VRF;
         bop.add = true;
index 53139bad1dad9f9dfe4444054aa29df1a23049c4..4f31e1c73ea3c0dbd4a9521d4c2b3d8462853a40 100644 (file)
@@ -149,8 +149,6 @@ public class BgpThriftService {
                     prefix,
                     plen,
                     nexthop,
-                    ethtag,
-                    esi,
                     macaddress,
                     l3label,
                     l2label,
@@ -190,7 +188,7 @@ public class BgpThriftService {
         @Override
         public void onNotificationSendEvent(String prefix, byte errCode,
                 byte errSubcode) {
-            bgpManager.sendNotificationEvent(prefix, errCode, errSubcode);
+            bgpManager.sendNotificationEvent((int) errCode, (int) errSubcode);
         }
     }
 
index b013d962f7e4875e85c7dfc2303d12e792a23ce1..d9d7e691b35ad605e8eb7237e6ef22161f6c6f52 100644 (file)
@@ -43,7 +43,7 @@ public class BgpManagerTest extends AbstractDataBrokerTest {
         List<String> nexthop = Collections.singletonList("100.100.100.100");
         int label = 1234;
 
-        bgpFibWriter.addFibEntryToDS(rd, null /*macAddress*/, prefix, nexthop,
+        bgpFibWriter.addFibEntryToDS(rd,  /*macAddress*/ prefix, nexthop,
                 VrfEntry.EncapType.Mplsgre, label, 0 /*l3vni*/, null /*gatewayMacAddress*/, RouteOrigin.LOCAL);
         //assertEquals(1, fibManager.getDataChgCount());
         assertEquals(1, 1);
@@ -56,7 +56,7 @@ public class BgpManagerTest extends AbstractDataBrokerTest {
         String prefix = "10.10.10.10/32";
         int label = 1234;
         try {
-            bgpFibWriter.addFibEntryToDS(rd, null /*macAddress*/, prefix, null,
+            bgpFibWriter.addFibEntryToDS(rd,  /*macAddress*/ prefix, null,
                     VrfEntry.EncapType.Mplsgre, label, 0 /*l3vni*/, null /*gatewayMacAddress*/, RouteOrigin.CONNECTED);
             assertEquals(1,0); //The code is not launching NullPointerException
         } catch (NullPointerException e) {
index 92b88a8c40c64705cc1cb63f556f220763b0bded..b29fc7ac56cdeddd56f89404d97ccc00d3ca9650 100755 (executable)
@@ -67,13 +67,10 @@ public interface ICloudServiceChain {
      *     retrieve the VNI and the elanTag.
      * @param scfTag ServiceChainForwarding Tag
      * @param elanLportTag LPortTag of the Elan Pseudo Port
-     * @param isLastServiceChain Only considered in removal operations. States
-     *     if there are no more ServiceChains using the ElanPseudoPort as
-     *     ingress port.
      * @param addOrRemove States if the flows must be added or removed
      */
     void programElanScfPipeline(String elanName, short tableId, long scfTag, int elanLportTag,
-                                boolean isLastServiceChain, int addOrRemove);
+                                int addOrRemove);
 
     // TODO: To be removed when sdnc is changed so that it calls the following API instead:
     void programElanScfPipeline(String elanName, short tableId, int scfTag, int elanLportTag, int addOrRemove);
index 90372753468c06a092b93097477c5bb9cc607cbb..3fafb5f6900b54a35e25e0f55aacdd0a5204dee2 100755 (executable)
@@ -54,7 +54,7 @@ public class CloudServiceChainProvider implements ICloudServiceChain {
 
     @Override
     public void programElanScfPipeline(String elanName, short tableId, long scfTag, int elanLportTag,
-                                       boolean isLastServiceChain, int addOrRemove) {
+                                       int addOrRemove) {
         LOG.info("ELAN Service chaining :programElanScfPipeline [Started] {} {} {} {} {}",
                  elanName, tableId, scfTag, elanLportTag, addOrRemove);
         elanServiceChainHandler.programElanScfPipeline(elanName, tableId, scfTag, elanLportTag, addOrRemove);
index 04f27e909148b00d3c6a89443399cf27bf387b00..48f631cb47d3d16bccd4fc20a0852e49a2fe6fdf 100644 (file)
@@ -97,13 +97,13 @@ public class PodListener implements DataTreeChangeListener<Pods> {
                 remove(podsBefore, podInterfaceBefore);
                 break;
             case SUBTREE_MODIFIED:
-                update(pods, podsBefore, podInterfaceBefore, podInterfaceAfter);
+                update(pods, podsBefore, podInterfaceAfter);
                 break;
             case WRITE:
                 if (podInterfaceBefore == null) {
                     add(pods, podInterfaceAfter);
                 } else {
-                    update(pods, podsBefore, podInterfaceBefore, podInterfaceAfter);
+                    update(pods, podsBefore, podInterfaceAfter);
                 }
                 break;
             default:
@@ -123,7 +123,7 @@ public class PodListener implements DataTreeChangeListener<Pods> {
         jobCoordinator.enqueueJob(pods.getName(), new PodConfigAddWorker(txRunner, pods, podInterface));
     }
 
-    private void update(Pods podsAfter, Pods podsBefore, Interface podInterfaceBefore, Interface podInterfaceAfter) {
+    private void update(Pods podsAfter, Pods podsBefore, Interface podInterfaceAfter) {
         LOG.trace("Pod updated before :{}, after :{}",podsBefore, podsAfter);
         if (!Objects.equals(podsAfter.getNetworkNS(), podsBefore.getNetworkNS())
                 || !Objects.equals(podsAfter.getHostIpAddress(), podsBefore.getHostIpAddress())) {
@@ -168,9 +168,9 @@ public class PodListener implements DataTreeChangeListener<Pods> {
                 ElanInstance elanInstance = CoeUtils.createElanInstanceForTheFirstPodInTheNetwork(
                         pods.getNetworkNS(), nodeIp, podInterface, tx);
                 LOG.info("interface creation for pod {}", interfaceName);
-                String portInterfaceName = CoeUtils.createOfPortInterface(interfaceName, podInterface, tx);
+                String portInterfaceName = CoeUtils.createOfPortInterface(interfaceName, tx);
                 LOG.debug("Creating ELAN Interface for pod {}", interfaceName);
-                CoeUtils.createElanInterface(podInterface, portInterfaceName,
+                CoeUtils.createElanInterface(portInterfaceName,
                         elanInstance.getElanInstanceName(), tx);
             }));
         }
index cba2b59c4111f2d46813d3d9eb5472ff3d887adb..475b6d923d59f25bce718f7e4bf0e3aee189064b 100644 (file)
@@ -95,9 +95,7 @@ public final class CoeUtils {
         return elanInstanceBuilder.build();
     }
 
-    public static void createElanInterface(org.opendaylight.yang.gen.v1.urn.opendaylight.coe
-                                                   .northbound.pod.rev170611.pod_attributes.Interface podInterface,
-                                           String elanInterfaceName, String elanInstanceName,
+    public static void createElanInterface(String elanInterfaceName, String elanInstanceName,
                                            WriteTransaction wrtConfigTxn) {
         InstanceIdentifier<ElanInterface> id = InstanceIdentifier.builder(ElanInterfaces.class).child(ElanInterface
                 .class, new ElanInterfaceKey(elanInterfaceName)).build();
@@ -116,8 +114,6 @@ public final class CoeUtils {
     }
 
     public static String createOfPortInterface(String interfaceName,
-                                               org.opendaylight.yang.gen.v1.urn.opendaylight.coe
-                                                       .northbound.pod.rev170611.pod_attributes.Interface podInterface,
                                                WriteTransaction wrtConfigTxn) {
         org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.interfaces.rev140508.interfaces.Interface inf =
                 buildInterface(interfaceName);
index 134529fda1f209e9c64ff00df3bd34e41bf6e9f1..38dbfbe95aa55afad86e7659385586bb43eb12af 100644 (file)
@@ -392,14 +392,14 @@ public class DhcpExternalTunnelManager {
                     }
                 }
                 for (Pair<IpAddress, String> pair : setOfTunnelIpElanNamePairs) {
-                    updateCacheAndInstallNewFlows(dpnId, listOfDpns, pair, tx);
+                    updateCacheAndInstallNewFlows(listOfDpns, pair, tx);
                 }
             }), LOG, "Error writing to datastore");
         }
     }
 
-    public void updateCacheAndInstallNewFlows(BigInteger dpnId,
-            List<BigInteger> listOfDpns, Pair<IpAddress, String> pair, WriteTransaction tx) {
+    public void updateCacheAndInstallNewFlows(List<BigInteger> listOfDpns, Pair<IpAddress, String> pair,
+            WriteTransaction tx) {
         BigInteger newDesignatedDpn = chooseDpn(pair.getLeft(), pair.getRight(), listOfDpns);
         if (newDesignatedDpn.equals(DhcpMConstants.INVALID_DPID)) {
             return;
@@ -448,7 +448,7 @@ public class DhcpExternalTunnelManager {
                 elanDpnAvailableFlag = false;
             }
             int size = 0;
-            L2GatewayDevice device = getDeviceFromTunnelIp(elanInstanceName, tunnelIp);
+            L2GatewayDevice device = getDeviceFromTunnelIp(tunnelIp);
             if (device == null) {
                 LOG.trace("Could not find any device for elanInstanceName {} and tunnelIp {}",
                         elanInstanceName, tunnelIp);
@@ -519,7 +519,7 @@ public class DhcpExternalTunnelManager {
 
     public void configureDhcpArpRequestResponseFlow(BigInteger dpnId, String elanInstanceName, boolean addFlow,
                                             IpAddress tunnelIp, String dhcpIpAddress, String dhcpMacAddress) {
-        L2GatewayDevice device = getDeviceFromTunnelIp(elanInstanceName, tunnelIp);
+        L2GatewayDevice device = getDeviceFromTunnelIp(tunnelIp);
         if (device == null) {
             LOG.error("Unable to get L2Device for tunnelIp {} and elanInstanceName {}", tunnelIp,
                     elanInstanceName);
@@ -657,7 +657,7 @@ public class DhcpExternalTunnelManager {
                                     tunnelIpInDpn, subnetDhcpData.get().getPortFixedip(),
                                     subnetDhcpData.get().getPortMacaddress());
                         }
-                        updateCacheAndInstallNewFlows(interfaceDpn, dpns, tunnelElanPair, tx);
+                        updateCacheAndInstallNewFlows(dpns, tunnelElanPair, tx);
                     }
                 }
             }));
@@ -832,7 +832,7 @@ public class DhcpExternalTunnelManager {
             }
 
             LOG.info("Installing remote McastMac");
-            L2GatewayDevice device = getDeviceFromTunnelIp(elanInstanceName, tunnelIp);
+            L2GatewayDevice device = getDeviceFromTunnelIp(tunnelIp);
             if (device == null) {
                 LOG.error("Unable to get L2Device for tunnelIp {} and elanInstanceName {}", tunnelIp,
                     elanInstanceName);
@@ -855,7 +855,7 @@ public class DhcpExternalTunnelManager {
         });
     }
 
-    private L2GatewayDevice getDeviceFromTunnelIp(String elanInstanceName, IpAddress tunnelIp) {
+    private L2GatewayDevice getDeviceFromTunnelIp(IpAddress tunnelIp) {
         Collection<L2GatewayDevice> devices = l2GatewayCache.getAll();
         LOG.trace("In getDeviceFromTunnelIp devices {}", devices);
         for (L2GatewayDevice device : devices) {
index fa83285d13331272acc6b905a9afe92656c49ebb..5153984459e85d412d27358a3b6368c1fcaa5ad1 100644 (file)
@@ -189,7 +189,7 @@ public class DhcpPktHandler implements PacketProcessingListener {
         if (interfacePort != null) {
             dhcpInfo = handleDhcpNeutronPacket(msgType, interfacePort, subnet, serverIp);
         } else if (config.isDhcpDynamicAllocationPoolEnabled()) {
-            dhcpInfo = handleDhcpAllocationPoolPacket(msgType, dhcpPkt, interfaceName, macAddress);
+            dhcpInfo = handleDhcpAllocationPoolPacket(msgType, interfaceName, macAddress);
         }
         DHCP reply = null;
         if (dhcpInfo != null) {
@@ -215,8 +215,7 @@ public class DhcpPktHandler implements PacketProcessingListener {
     }
 
 
-    private DhcpInfo handleDhcpAllocationPoolPacket(byte msgType, DHCP dhcpPkt, String interfaceName,
-            String macAddress) {
+    private DhcpInfo handleDhcpAllocationPoolPacket(byte msgType, String interfaceName, String macAddress) {
         String networkId = dhcpAllocationPoolMgr.getNetworkByPort(interfaceName);
         AllocationPool pool = networkId != null ? dhcpAllocationPoolMgr.getAllocationPoolByNetwork(networkId)
                 : null;
index dda8164f3fc8e732ecdf115e25005e038ae950c1..17d3c1e9edbb7e4bd598bc734bcdd2d3117cfc0d 100644 (file)
@@ -35,12 +35,11 @@ public interface IElanService extends IEtreeService {
     void updateElanInterface(String elanInstanceName, String interfaceName, List<String> updatedStaticMacAddresses,
             String newDescription);
 
-    void deleteElanInterface(String elanInstanceName, String interfaceName);
+    void deleteElanInterface(String interfaceName);
 
-    void addStaticMacAddress(String elanInstanceName, String interfaceName, String macAddress);
+    void addStaticMacAddress(String interfaceName, String macAddress);
 
-    void deleteStaticMacAddress(String elanInstanceName, String interfaceName, String macAddress)
-            ;
+    void deleteStaticMacAddress(String interfaceName, String macAddress);
 
     Collection<MacEntry> getElanMacTable(String elanInstanceName);
 
index 9e44ffae9266d74d812a532194fc496bce3a94d3..5327760682334da8323ff115f8d251094b26fec4 100644 (file)
@@ -13,7 +13,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.etree.rev16061
 
 public interface IEtreeService {
 
-    void deleteEtreeInterface(String elanInstanceName, String interfaceName);
+    void deleteEtreeInterface(String interfaceName);
 
     boolean deleteEtreeInstance(String elanInstanceName);
 
index deb6c2ccf543ace131513edf60011799848109fa..598b730643a054383fa060aa3007f2280c4fe421 100644 (file)
@@ -11,7 +11,6 @@ package org.opendaylight.netvirt.elanmanager.api;
 import java.util.List;
 import java.util.Set;
 
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.netvirt.neutronvpn.api.l2gw.L2GatewayDevice;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.rev130715.IpAddress;
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.Uuid;
@@ -30,5 +29,5 @@ public interface IL2gwService {
                                 String l2GwDeviceName,
                                 L2gateway l2Gateway);
 
-    List<L2gatewayConnection> getAssociatedL2GwConnections(DataBroker broker, Set<Uuid> l2GatewayIds);
+    List<L2gatewayConnection> getAssociatedL2GwConnections(Set<Uuid> l2GatewayIds);
 }
index e317f682064584ffc13350ba38ccf97a48efa675..7cbf0f68bbefb5c4675e3c66d9faa279f23fc399 100644 (file)
@@ -31,7 +31,7 @@ public class ElanInterfaceDelete  extends OsgiCommandSupport {
     @Override
     protected Object doExecute() {
         LOG.debug("Deleting ElanInterface command" + "\t" + elanName + "\t" + interfaceName + "\t");
-        elanProvider.deleteElanInterface(elanName, interfaceName);
+        elanProvider.deleteElanInterface(interfaceName);
         return null;
     }
 }
index 8a92cd70961e2f3951b1e4cd106f24235767c8c0..f18a6a37aaad5762851b50f7777da7cf4292734f 100644 (file)
@@ -35,7 +35,7 @@ public class StaticMacAdd extends OsgiCommandSupport {
     protected Object doExecute() {
         LOG.debug("Executing create ElanInterface command" + "\t" + elanName + "\t" + interfaceName + "\t"
                 + staticMacAddress + "\t");
-        elanProvider.addStaticMacAddress(elanName, interfaceName, staticMacAddress);
+        elanProvider.addStaticMacAddress(interfaceName, staticMacAddress);
         return null;
     }
 }
index 4236c30a06f98cf8395c5cbe196517d2e1c08aa4..eaa487ead0d16f80ae6f963c69c3512c06d084c0 100644 (file)
@@ -35,7 +35,7 @@ public class StaticMacDelete  extends OsgiCommandSupport {
     protected Object doExecute() {
         LOG.debug("Executing create ElanInterface command" + "\t" + elanName + "\t" + interfaceName + "\t"
                 + staticMacAddress + "\t");
-        elanProvider.deleteStaticMacAddress(elanName, interfaceName, staticMacAddress);
+        elanProvider.deleteStaticMacAddress(interfaceName, staticMacAddress);
         return null;
     }
 }
index bc2bd6dc9a850347988f9be7a3678de945c8e9c9..b4a7a27cbe2093966fc3fbc03d2754b1f4f862a2 100644 (file)
@@ -39,7 +39,7 @@ public class EtreeInterfaceDelete extends OsgiCommandSupport {
             session.getConsole()
                     .println("Etree interface doesn't exist or isn't configured as etree: " + interfaceName);
         }
-        elanProvider.deleteEtreeInterface(etreeName, interfaceName);
+        elanProvider.deleteEtreeInterface(interfaceName);
         session.getConsole().println("Deleted the Etree interface succesfully");
         return null;
     }
index da91352340c4c0d469421e2c0ba90b1eb6b86c36..897e1546c8a9b7f755ccb34f9363a7544ecc6e4d 100644 (file)
@@ -397,7 +397,7 @@ public class L2GwValidateCli extends OsgiCommandSupport {
 
                 isValid = verfiyLogicalSwitch(logicalSwitchName, nodeIid);
                 if (isValid) {
-                    isValid = verifyMcastMac(logicalSwitchName, l2GatewayDevice, nodeIid);
+                    isValid = verifyMcastMac(logicalSwitchName, nodeIid);
                     verifyVlanBindings(nodeIid, logicalSwitchName, device, l2gatewayConnection.getSegmentId());
                     L2GatewayDevice elanL2gatewayDevice = ElanL2GwCacheUtils
                             .getL2GatewayDeviceFromCache(logicalSwitchName, nodeId.getValue());
@@ -457,7 +457,6 @@ public class L2GwValidateCli extends OsgiCommandSupport {
     }
 
     private boolean verifyMcastMac(String logicalSwitchName,
-                                   L2GatewayDevice l2GatewayDevice,
                                    InstanceIdentifier<Node> nodeIid) {
         NodeId nodeId = nodeIid.firstKeyOf(Node.class).getNodeId();
 
index 8aa6cc6c2fcdb7e206b36a68e68ec2bab84d24ef..1b76d6a6943ff0fd4e6d35274e485c67780454b3 100644 (file)
@@ -14,7 +14,6 @@ import org.opendaylight.infrautils.diagstatus.DiagStatusService;
 import org.opendaylight.infrautils.diagstatus.ServiceDescriptor;
 import org.opendaylight.infrautils.diagstatus.ServiceState;
 import org.opendaylight.infrautils.diagstatus.ServiceStatusProvider;
-import org.opendaylight.netvirt.elan.internal.ElanServiceProvider;
 import org.opendaylight.netvirt.elan.utils.ElanConstants;
 import org.ops4j.pax.cdi.api.OsgiServiceProvider;
 import org.slf4j.Logger;
@@ -33,8 +32,7 @@ public class ElanDiagStatusProvider implements ServiceStatusProvider {
     private volatile ServiceDescriptor serviceDescriptor;
 
     @Inject
-    public ElanDiagStatusProvider(final ElanServiceProvider elanServiceProvider,
-                                  final DiagStatusService diagStatusService) {
+    public ElanDiagStatusProvider(final DiagStatusService diagStatusService) {
         this.diagStatusService = diagStatusService;
         diagStatusService.register(ElanConstants.ELAN_SERVICE_NAME);
         serviceDescriptor = new ServiceDescriptor(ElanConstants.ELAN_SERVICE_NAME, ServiceState.OPERATIONAL,
index 05035f4c7365d744638aa99e3a48cb7ffbaaed6f..ce331b1261528cb10558b30371bc413b9238f47d 100644 (file)
@@ -21,7 +21,6 @@ import java.util.Optional;
 import java.util.Random;
 import javax.inject.Inject;
 import javax.inject.Singleton;
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.genius.interfacemanager.globals.IfmConstants;
 import org.opendaylight.genius.interfacemanager.interfaces.IInterfaceManager;
@@ -84,14 +83,13 @@ public class ElanBridgeManager implements IElanBridgeManager {
 
     /**
      * Construct a new ElanBridgeManager.
-     * @param dataBroker DataBroker
      * @param elanConfig the elan configuration
      * @param interfaceManager InterfaceManager
      * @param southboundUtils southboutUtils
      * @param mdsalUtils mdsalUtils
      */
     @Inject
-    public ElanBridgeManager(DataBroker dataBroker, ElanConfig elanConfig, IInterfaceManager interfaceManager,
+    public ElanBridgeManager(ElanConfig elanConfig, IInterfaceManager interfaceManager,
             SouthboundUtils southboundUtils, MdsalUtils mdsalUtils) {
         //TODO: ClusterAware!!!??
         this.mdsalUtils = mdsalUtils;
index 498acc848b9c40ec9bd4c61498bacf67010d525f..baf8711d9fcec50f303ee8e94b53fcc4c73ac72e 100644 (file)
@@ -14,7 +14,6 @@ import javax.inject.Singleton;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.genius.datastoreutils.AsyncDataTreeChangeListenerBase;
-import org.opendaylight.genius.interfacemanager.interfaces.IInterfaceManager;
 import org.opendaylight.netvirt.elan.cache.ElanInstanceCache;
 import org.opendaylight.netvirt.elan.utils.ElanUtils;
 import org.opendaylight.netvirt.elan.utils.TransportZoneNotificationUtil;
@@ -37,7 +36,7 @@ public class ElanDpnToTransportZoneListener
     private final ElanInstanceCache elanInstanceCache;
 
     @Inject
-    public ElanDpnToTransportZoneListener(final DataBroker dbx, final IInterfaceManager interfaceManager,
+    public ElanDpnToTransportZoneListener(final DataBroker dbx,
             final ElanConfig elanConfig, final TransportZoneNotificationUtil tznu,
             final ElanInstanceCache elanInstanceCache) {
         useTransportZone = elanConfig.isAutoConfigTransportZones();
index 785cb0110db98031d48bdddeb938cb581adbc54f..94b582d6d2073bbdcc1342d71d4b3e519e35b733 100644 (file)
@@ -105,7 +105,6 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
     public ElanServiceProvider(IdManagerService idManager, IInterfaceManager interfaceManager,
                                ElanBridgeManager bridgeMgr,
                                DataBroker dataBroker,
-                               ElanInterfaceManager elanInterfaceManager,
                                ElanUtils elanUtils,
                                EntityOwnershipService entityOwnershipService,
                                SouthboundUtils southboundUtils, ElanInstanceCache elanInstanceCache,
@@ -330,13 +329,13 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
     }
 
     @Override
-    public void deleteEtreeInterface(String elanInstanceName, String interfaceName) {
-        deleteElanInterface(elanInstanceName, interfaceName);
+    public void deleteEtreeInterface(String interfaceName) {
+        deleteElanInterface(interfaceName);
         LOG.debug("deleting the Etree Interface {}", interfaceName);
     }
 
     @Override
-    public void deleteElanInterface(String elanInstanceName, String interfaceName) {
+    public void deleteElanInterface(String interfaceName) {
         Optional<ElanInterface> existingElanInterface = elanInterfaceCache.get(interfaceName);
         if (existingElanInterface.isPresent()) {
             ElanUtils.delete(broker, LogicalDatastoreType.CONFIGURATION,
@@ -346,7 +345,7 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
     }
 
     @Override
-    public void addStaticMacAddress(String elanInstanceName, String interfaceName, String macAddress) {
+    public void addStaticMacAddress(String interfaceName, String macAddress) {
         Optional<ElanInterface> existingElanInterface = elanInterfaceCache.get(interfaceName);
         if (existingElanInterface.isPresent()) {
             StaticMacEntriesBuilder staticMacEntriesBuilder = new StaticMacEntriesBuilder();
@@ -359,7 +358,7 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
     }
 
     @Override
-    public void deleteStaticMacAddress(String elanInstanceName, String interfaceName, String macAddress) {
+    public void deleteStaticMacAddress(String interfaceName, String macAddress) {
         Optional<ElanInterface> existingElanInterface = elanInterfaceCache.get(interfaceName);
         if (existingElanInterface.isPresent()) {
             InstanceIdentifier<StaticMacEntries> staticMacEntriesIdentifier =
@@ -404,7 +403,7 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
             if (elanInterfaceMac.getMacEntry() != null && elanInterfaceMac.getMacEntry().size() > 0) {
                 List<MacEntry> macEntries = elanInterfaceMac.getMacEntry();
                 for (MacEntry macEntry : macEntries) {
-                    deleteStaticMacAddress(elanInstanceName, elanInterface, macEntry.getMacAddress().getValue());
+                    deleteStaticMacAddress(elanInterface, macEntry.getMacAddress().getValue());
                 }
             }
         }
@@ -510,7 +509,7 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
         String intfName = providerIntfName + IfmConstants.OF_URI_SEPARATOR + elanInstance.getSegmentationId();
         Interface memberIntf = interfaceManager.getInterfaceInfoFromConfigDataStore(intfName);
         if (memberIntf != null) {
-            deleteElanInterface(elanInstance.getElanInstanceName(), intfName);
+            deleteElanInterface(intfName);
             deleteIetfInterface(intfName);
             LOG.debug("delete vlan prv intf {} in elan {}, dpID {}", intfName,
                     elanInstance.getElanInstanceName(), dpnId);
@@ -535,7 +534,7 @@ public class ElanServiceProvider extends AbstractLifecycle implements IElanServi
                 if (shouldDeleteTrunk(trunkInterfaceName, elanInterface)) {
                     deleteIetfInterface(trunkInterfaceName);
                 }
-                deleteElanInterface(elanInstanceName, elanInterface);
+                deleteElanInterface(elanInterface);
             }
         }
     }
index 6f3d41c6d9df513b0639c1ffb2e74b7fd13e2878..e8b57ad179cfdf79300842175f0f15110836d35e 100644 (file)
@@ -14,7 +14,6 @@ import javax.inject.Singleton;
 import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.common.api.data.LogicalDatastoreType;
 import org.opendaylight.genius.datastoreutils.AsyncDataTreeChangeListenerBase;
-import org.opendaylight.genius.interfacemanager.interfaces.IInterfaceManager;
 import org.opendaylight.netvirt.elan.utils.TransportZoneNotificationUtil;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.elan.config.rev150710.ElanConfig;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.VpnInstanceOpData;
@@ -34,7 +33,7 @@ public class VpnDpnToTransportZoneListener
     private final Boolean useTransportZone;
 
     @Inject
-    public VpnDpnToTransportZoneListener(final DataBroker dbx, final IInterfaceManager interfaceManager,
+    public VpnDpnToTransportZoneListener(final DataBroker dbx,
             final ElanConfig elanConfig, final TransportZoneNotificationUtil tznu) {
         useTransportZone = elanConfig.isAutoConfigTransportZones();
         transportZoneNotificationUtil = tznu;
index 88db72b857638e6cd806d02a144e765ab89e0f07..d0fc9fa14cef5ee7be0f3a72de888bfdec172a36 100644 (file)
@@ -29,7 +29,6 @@ import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.inet.types.
 import org.opendaylight.yang.gen.v1.urn.ietf.params.xml.ns.yang.ietf.yang.types.rev130715.Uuid;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepGlobalAugmentation;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepGlobalAugmentationBuilder;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepGlobalRef;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepLogicalSwitchRef;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepNodeName;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.params.xml.ns.yang.ovsdb.hwvtep.rev150901.HwvtepPhysicalLocatorRef;
@@ -221,18 +220,6 @@ public final class HwvtepHAUtil {
         return false;
     }
 
-    public static void mergeManagedByNode(Node psNode,
-                                          PhysicalSwitchAugmentationBuilder builder,
-                                          InstanceIdentifier<Node> haNodePath,
-                                          InstanceIdentifier<Node> haPsPath, NodeId haPSNodeId) {
-        PhysicalSwitchAugmentation psAugmentation = psNode.getAugmentation(PhysicalSwitchAugmentation.class);
-        builder.setManagedBy(new HwvtepGlobalRef(haNodePath));
-        builder.setHwvtepNodeName(psAugmentation.getHwvtepNodeName());
-        builder.setHwvtepNodeDescription(psAugmentation.getHwvtepNodeDescription());
-        builder.setTunnelIps(psAugmentation.getTunnelIps());
-        builder.setPhysicalSwitchUuid(getUUid(psAugmentation.getHwvtepNodeName().getValue()));
-    }
-
     public static Node getOriginal(DataObjectModification<Node> mod) {
         Node node = null;
         switch (mod.getModificationType()) {
index 2c925321cdd5ec744b0977e0f7d72ee8ae1a51fd..b890bc1dcea7f71832d666801f90e6b0b23b7282 100644 (file)
@@ -27,15 +27,11 @@ public class ConfigNodeUpdatedHandler {
     /**
      * Copy updated data from HA node to child node of config data tree.
      *
-     * @param haUpdated HA node updated
-     * @param haOriginal HA node original
      * @param haChildNodeId HA child node which needs to be updated
      * @param mod the data object modification
      * @param tx Transaction
      */
-    public void copyHAGlobalUpdateToChild(Node haUpdated,
-                                          Node haOriginal,
-                                          InstanceIdentifier<Node> haChildNodeId,
+    public void copyHAGlobalUpdateToChild(InstanceIdentifier<Node> haChildNodeId,
                                           DataObjectModification<Node> mod,
                                           ReadWriteTransaction tx) {
         globalAugmentationMerger.mergeConfigUpdate(haChildNodeId,
@@ -46,15 +42,11 @@ public class ConfigNodeUpdatedHandler {
     /**
      * Copy HA ps node update to HA child ps node of config data tree.
      *
-     * @param haUpdated HA node updated
-     * @param haOriginal HA node original
      * @param haChildNodeId HA child node which needs to be updated
      * @param mod the data object modification
      * @param tx Transaction
      */
-    public void copyHAPSUpdateToChild(Node haUpdated,
-                                      Node haOriginal,
-                                      InstanceIdentifier<Node> haChildNodeId,
+    public void copyHAPSUpdateToChild(InstanceIdentifier<Node> haChildNodeId,
                                       DataObjectModification<Node> mod,
                                       ReadWriteTransaction tx) {
 
index 7ce513d704f2a7ab21af5fbc07803e5c83bc62de..0f9e7b59fc6278f2632b369dd782009fd3c8a8d1 100644 (file)
@@ -59,51 +59,44 @@ public class HAEventHandler implements IHAEventHandler {
     }
 
     @Override
-    public void copyChildGlobalOpUpdateToHAParent(Node updatedSrcNode,
-                                                  Node origSrcNode,
-                                                  InstanceIdentifier<Node> haPath,
+    public void copyChildGlobalOpUpdateToHAParent(InstanceIdentifier<Node> haPath,
                                                   DataObjectModification<Node> mod,
                                                   ReadWriteTransaction tx) {
         if (haPath == null) {
             return;
         }
-        opNodeUpdatedHandler.copyChildGlobalOpUpdateToHAParent(updatedSrcNode, origSrcNode, haPath, mod, tx);
+        opNodeUpdatedHandler.copyChildGlobalOpUpdateToHAParent(haPath, mod, tx);
     }
 
     @Override
     public void copyChildPsOpUpdateToHAParent(Node updatedSrcPSNode,
-                                              Node origSrcPSNode,
                                               InstanceIdentifier<Node> haPath,
                                               DataObjectModification<Node> mod,
                                               ReadWriteTransaction tx) {
         if (haPath == null) {
             return;
         }
-        opNodeUpdatedHandler.copyChildPsOpUpdateToHAParent(updatedSrcPSNode, origSrcPSNode, haPath, mod, tx);
+        opNodeUpdatedHandler.copyChildPsOpUpdateToHAParent(updatedSrcPSNode, haPath, mod, tx);
     }
 
     @Override
-    public void copyHAPSUpdateToChild(Node haUpdated,
-                                      Node haOriginal,
-                                      InstanceIdentifier<Node> haChildNodeId,
+    public void copyHAPSUpdateToChild(InstanceIdentifier<Node> haChildNodeId,
                                       DataObjectModification<Node> mod,
                                       ReadWriteTransaction tx) {
         if (haChildNodeId == null) {
             return;
         }
-        configNodeUpdatedHandler.copyHAPSUpdateToChild(haUpdated, haOriginal, haChildNodeId, mod, tx);
+        configNodeUpdatedHandler.copyHAPSUpdateToChild(haChildNodeId, mod, tx);
     }
 
     @Override
-    public void copyHAGlobalUpdateToChild(Node haUpdated,
-                                          Node haOriginal,
-                                          InstanceIdentifier<Node> haChildNodeId,
+    public void copyHAGlobalUpdateToChild(InstanceIdentifier<Node> haChildNodeId,
                                           DataObjectModification<Node> mod,
                                           ReadWriteTransaction tx) {
         if (haChildNodeId == null) {
             return;
         }
-        configNodeUpdatedHandler.copyHAGlobalUpdateToChild(haUpdated, haOriginal, haChildNodeId, mod, tx);
+        configNodeUpdatedHandler.copyHAGlobalUpdateToChild(haChildNodeId, mod, tx);
     }
 
 }
index 8ac0bd57363f7eb5f314b95b4c6c51f999caade0..1a04d70f9aa7a24a434271482e177cb495addb90 100644 (file)
@@ -34,28 +34,21 @@ public interface IHAEventHandler {
             throws ReadFailedException, ExecutionException, InterruptedException;
 
 
-    void copyChildGlobalOpUpdateToHAParent(Node updatedSrcNode,
-                                           Node origSrcNode,
-                                           InstanceIdentifier<Node> haPath,
+    void copyChildGlobalOpUpdateToHAParent(InstanceIdentifier<Node> haPath,
                                            DataObjectModification<Node> mod,
                                            ReadWriteTransaction tx);
 
     void copyChildPsOpUpdateToHAParent(Node updatedSrcPSNode,
-                                       Node origSrcPSNode,
                                        InstanceIdentifier<Node> haPath,
                                        DataObjectModification<Node> mod,
                                        ReadWriteTransaction tx);
 
-    void copyHAPSUpdateToChild(Node haUpdated,
-                               Node haOriginal,
-                               InstanceIdentifier<Node> haChildPath,
+    void copyHAPSUpdateToChild(InstanceIdentifier<Node> haChildPath,
                                DataObjectModification<Node> mod,
                                ReadWriteTransaction tx)
             ;
 
-    void copyHAGlobalUpdateToChild(Node haUpdated,
-                                   Node haOriginal,
-                                   InstanceIdentifier<Node> haChildPath,
+    void copyHAGlobalUpdateToChild(InstanceIdentifier<Node> haChildPath,
                                    DataObjectModification<Node> mod,
                                    ReadWriteTransaction tx)
             ;
index 846bc3aad96504d2388a66389702b84fbc13653b..2c215f49f3afe7dfea750fb5bd431ea63970eca6 100644 (file)
@@ -30,13 +30,11 @@ public class OpNodeUpdatedHandler {
      * Copy HA ps node update to HA child ps node of operational data tree.
      *
      * @param updatedSrcPSNode Updated HA child ps node
-     * @param origSrcPSNode Original HA ps node
      * @param haPath HA node path
      * @param mod the data object modification
      * @param tx Transaction
      */
     public void copyChildPsOpUpdateToHAParent(Node updatedSrcPSNode,
-                                              Node origSrcPSNode,
                                               InstanceIdentifier<Node> haPath,
                                               DataObjectModification<Node> mod,
                                               ReadWriteTransaction tx) {
@@ -51,15 +49,11 @@ public class OpNodeUpdatedHandler {
     /**
      * Copy updated data from HA node to child node of operational data tree.
      *
-     * @param updatedSrcNode Updated HA child node
-     * @param origSrcNode Original HA node
      * @param haPath HA node path
      * @param mod the data object modification
      * @param tx Transaction
      */
-    public void copyChildGlobalOpUpdateToHAParent(Node updatedSrcNode,
-                                                  Node origSrcNode,
-                                                  InstanceIdentifier<Node> haPath,
+    public void copyChildGlobalOpUpdateToHAParent(InstanceIdentifier<Node> haPath,
                                                   DataObjectModification<Node> mod,
                                                   ReadWriteTransaction tx) {
 
index cc8b903388b3b53d5694c50715cc7f3f4132972c..622e5e7ed6b9cee89ed1f327930a24cd4a6d22a2 100644 (file)
@@ -61,16 +61,15 @@ public class HAConfigNodeListener extends HwvtepNodeBaseListener {
     }
 
     @Override
-    void onPsNodeUpdate(InstanceIdentifier<Node> key,
-                        Node haPSUpdated,
-                        Node haPSOriginal,
-                        DataObjectModification<Node> mod,
-                        ReadWriteTransaction tx) throws InterruptedException, ExecutionException, ReadFailedException {
+    void onPsNodeUpdate(Node haPSUpdated,
+            Node haPSOriginal,
+            DataObjectModification<Node> mod,
+            ReadWriteTransaction tx) throws InterruptedException, ExecutionException, ReadFailedException {
         //copy the ps node data to children
         String psId = haPSUpdated.getNodeId().getValue();
         Set<InstanceIdentifier<Node>> childSwitchIds = HwvtepHAUtil.getPSChildrenIdsForHAPSNode(psId);
         for (InstanceIdentifier<Node> childSwitchId : childSwitchIds) {
-            haEventHandler.copyHAPSUpdateToChild(haPSUpdated, haPSOriginal, childSwitchId, mod, tx);
+            haEventHandler.copyHAPSUpdateToChild(childSwitchId, mod, tx);
         }
     }
 
@@ -83,7 +82,7 @@ public class HAConfigNodeListener extends HwvtepNodeBaseListener {
             throws InterruptedException, ExecutionException, ReadFailedException {
         Set<InstanceIdentifier<Node>> childNodeIds = hwvtepHACache.getChildrenForHANode(key);
         for (InstanceIdentifier<Node> haChildNodeId : childNodeIds) {
-            haEventHandler.copyHAGlobalUpdateToChild(haUpdated, haOriginal, haChildNodeId, mod, tx);
+            haEventHandler.copyHAGlobalUpdateToChild(haChildNodeId, mod, tx);
         }
     }
 
index 0eb974628266b3d1a7583b022c2ec6eb9952b1b1..33033e4500d26d283fdc45a03540d9f97ce98003 100644 (file)
@@ -107,7 +107,7 @@ public class HAOpClusteredListener extends HwvtepNodeBaseListener implements Clu
                             DataObjectModification<Node> mod,
                             ReadWriteTransaction tx) {
         boolean wasHAChild = hwvtepHACache.isHAEnabledDevice(childPath);
-        addToHACacheIfBecameHAChild(childPath, updatedChildNode, beforeChildNode, tx);
+        addToHACacheIfBecameHAChild(childPath, updatedChildNode, beforeChildNode);
         boolean isHAChild = hwvtepHACache.isHAEnabledDevice(childPath);
 
 
@@ -133,12 +133,10 @@ public class HAOpClusteredListener extends HwvtepNodeBaseListener implements Clu
      * @param childPath HA child path which got converted to HA node
      * @param updatedChildNode updated Child node
      * @param beforeChildNode non-ha node before updated to HA node
-     * @param tx Transaction
      */
     public static void addToHACacheIfBecameHAChild(InstanceIdentifier<Node> childPath,
                                                    Node updatedChildNode,
-                                                   Node beforeChildNode,
-                                                   ReadWriteTransaction tx) {
+                                                   Node beforeChildNode) {
         HwvtepGlobalAugmentation updatedAugmentaion = updatedChildNode.getAugmentation(HwvtepGlobalAugmentation.class);
         HwvtepGlobalAugmentation beforeAugmentaion = null;
         if (beforeChildNode != null) {
index 08bb93cf512b200bad6b66108dcfd7abf98464bb..f2fcb9f549651d0ec76ca56f7ac46a8b52474a5a 100644 (file)
@@ -84,7 +84,7 @@ public class HAOpNodeListener extends HwvtepNodeBaseListener {
         } catch (ReadFailedException e) {
             LOG.error("Failed to read nodes {} , {} ", childGlobalPath, haNodePath);
         }
-        readAndCopyChildPsOpToParent(childGlobalPath, childNode, haNodePath, tx);
+        readAndCopyChildPsOpToParent(childNode, tx);
     }
 
     //Update on global node has been taken care by HAListeners as per perf improvement
@@ -99,11 +99,11 @@ public class HAOpNodeListener extends HwvtepNodeBaseListener {
         if (!Strings.isNullOrEmpty(oldHAId)) { //was already ha child
             InstanceIdentifier<Node> haPath = hwvtepHACache.getParent(childGlobalPath);
             LOG.debug("Copy oper update from child {} to parent {}", childGlobalPath, haPath);
-            haEventHandler.copyChildGlobalOpUpdateToHAParent(updatedChildNode, originalChildNode, haPath, mod, tx);
+            haEventHandler.copyChildGlobalOpUpdateToHAParent(haPath, mod, tx);
             return;//TODO handle unha case
         }
 
-        HAOpClusteredListener.addToHACacheIfBecameHAChild(childGlobalPath, updatedChildNode, originalChildNode, tx);
+        HAOpClusteredListener.addToHACacheIfBecameHAChild(childGlobalPath, updatedChildNode, originalChildNode);
         if (IS_NOT_HA_CHILD.test(childGlobalPath)) {
             return;
         }
@@ -163,17 +163,16 @@ public class HAOpNodeListener extends HwvtepNodeBaseListener {
     }
 
     @Override
-    void onPsNodeUpdate(InstanceIdentifier<Node> childPsPath,
-                        Node updatedChildPSNode,
-                        Node originalChildPSNode,
-                        DataObjectModification<Node> mod,
-                        ReadWriteTransaction tx) throws ReadFailedException {
+    void onPsNodeUpdate(Node updatedChildPSNode,
+            Node originalChildPSNode,
+            DataObjectModification<Node> mod,
+            ReadWriteTransaction tx) throws ReadFailedException {
         InstanceIdentifier<Node> childGlobalPath = HwvtepHAUtil.getGlobalNodePathFromPSNode(updatedChildPSNode);
         if (IS_NOT_HA_CHILD.test(childGlobalPath)) {
             return;
         }
         InstanceIdentifier<Node> haGlobalPath = hwvtepHACache.getParent(childGlobalPath);
-        haEventHandler.copyChildPsOpUpdateToHAParent(updatedChildPSNode, originalChildPSNode, haGlobalPath, mod, tx);
+        haEventHandler.copyChildPsOpUpdateToHAParent(updatedChildPSNode, haGlobalPath, mod, tx);
     }
 
     @Override
@@ -204,10 +203,7 @@ public class HAOpNodeListener extends HwvtepNodeBaseListener {
         }
     }
 
-    private void readAndCopyChildPsOpToParent(InstanceIdentifier<Node> childGlobalPath,
-                                              Node childNode,
-                                              InstanceIdentifier<Node> haNodePath,
-                                              ReadWriteTransaction tx) {
+    private void readAndCopyChildPsOpToParent(Node childNode, ReadWriteTransaction tx) {
         String childGlobalNodeId = childNode.getNodeId().getValue();
         List<InstanceIdentifier> childPsIids = new ArrayList<>();
         HwvtepGlobalAugmentation hwvtepGlobalAugmentation = childNode.getAugmentation(HwvtepGlobalAugmentation.class);
index 1e8ba0fcbbf44d458f0f61a3be9b1e812a8b4650..ba7581a18ddaecc2743028f3b5e0e1740844602a 100644 (file)
@@ -88,7 +88,7 @@ public abstract class HwvtepNodeBaseListener implements DataTreeChangeListener<N
                 if (!nodeId.contains(HwvtepHAUtil.PHYSICALSWITCH)) {
                     onGlobalNodeUpdate(key, updated, original, mod, tx);
                 } else {
-                    onPsNodeUpdate(key, updated, original, mod, tx);
+                    onPsNodeUpdate(updated, original, mod, tx);
                 }
             }
         }
@@ -180,7 +180,7 @@ public abstract class HwvtepNodeBaseListener implements DataTreeChangeListener<N
 
     }
 
-    void onPsNodeUpdate(InstanceIdentifier<Node> key, Node updated, Node original,
+    void onPsNodeUpdate(Node updated, Node original,
                         DataObjectModification<Node> mod, ReadWriteTransaction tx)
             throws ReadFailedException, InterruptedException, ExecutionException {
 
index 85d3488260fd9fc7270cb084bcc286826272d752..946b350556f48c501a05d8607837d7a84bc9b752 100644 (file)
@@ -85,7 +85,7 @@ public class DisAssociateHwvtepFromElanJob implements Callable<List<ListenableFu
 
             LOG.info("delete mcast mac for {} {}", elanName, strHwvtepNodeId);
             futures.addAll(elanL2GatewayMulticastUtils.handleMcastForElanL2GwDeviceDelete(this.elanName,
-                    elanInstanceSupplier.get(), l2GatewayDevice));
+                    l2GatewayDevice));
 
             LOG.info("delete local ucast macs {} {}", elanName, strHwvtepNodeId);
             elanL2GatewayUtils.deleteL2GwDeviceUcastLocalMacsFromElan(l2GatewayDevice, elanName);
index 6868ffe137d82d327b42774c6e11f7c194bcb44f..4d6b4aa06a53a4eeb0a6d8fddeb9728e24ec79f8 100644 (file)
@@ -74,7 +74,7 @@ public class LogicalSwitchAddedJob implements Callable<List<ListenableFuture<Voi
             new NodeId(elanL2GwDevice.getHwvtepNodeId()), logicalSwitchName, physicalDevice, defaultVlanId));
         LOG.info("creating mast mac entries for {} {}", logicalSwitchName, elanL2GwDevice.getHwvtepNodeId());
         futures.add(elanL2GatewayMulticastUtils.handleMcastForElanL2GwDeviceAdd(logicalSwitchName,
-                elanInstanceSupplier.get(), elanL2GwDevice));
+                elanL2GwDevice));
         futures.add(elanL2GatewayUtils.installElanMacsInL2GatewayDevice(
                 logicalSwitchName, elanL2GwDevice));
         return futures;
index 4da3880e563576fb4f1754586fcf5b60621d9216..1985830b296f47abb9b76a6b3da3184efc62d85e 100644 (file)
@@ -84,11 +84,10 @@ public abstract class ChildListener<P extends DataObject, C extends DataObject,
 
     /**
      * Group of data for bulk child update cases.
-     * @param childIid - Subtree Node iid
      * @param child - Subtree Node data
      * @return - Group value
      */
-    protected abstract G getGroup(InstanceIdentifier<C> childIid, C child);
+    protected abstract G getGroup(C child);
 
     /**
      * Process the added/updated/Deleted subtree data.
@@ -190,12 +189,12 @@ public abstract class ChildListener<P extends DataObject, C extends DataObject,
             switch (modificationType) {
                 case WRITE:
                 case SUBTREE_MODIFIED:
-                    group = getGroup(childIid, dataAfter);
+                    group = getGroup(dataAfter);
                     updatedMacsGrouped.computeIfAbsent(group, (grp) -> new ConcurrentHashMap<>());
                     updatedMacsGrouped.get(group).put(childIid, dataAfter);
                     break;
                 case DELETE:
-                    group = getGroup(childIid, dataBefore);
+                    group = getGroup(dataBefore);
                     deletedMacsGrouped.computeIfAbsent(group, (grp) -> new ConcurrentHashMap<>());
                     deletedMacsGrouped.get(group).put(childIid, dataBefore);
                     break;
index 126004f2e0c818ee6f9d1d4999074c64fa335eba..a1dc1daf9b81f068cca07a5b07c018a3d5bed7b3 100644 (file)
@@ -151,7 +151,7 @@ public class HwvtepRemoteMcastMacListener
     @SuppressWarnings("checkstyle:IllegalCatch") // TODO remove when using AutoCloseables
     void runTask() {
         try {
-            String jobKey = ElanL2GatewayUtils.getL2GatewayConnectionJobKey(nodeId.getValue(), nodeId.getValue());
+            String jobKey = ElanL2GatewayUtils.getL2GatewayConnectionJobKey(nodeId.getValue());
             jobCoordinator.enqueueJob(jobKey, taskToRun,
                     SystemPropertyReader.getDataStoreJobCoordinatorMaxRetries());
         } finally {
index a62dd54def9a37d58935db6b3b2ab95c0bdffd9e..44a56c6ad4b4f21e792d134124e0d4ec0ca9a8e3 100644 (file)
@@ -99,7 +99,7 @@ public class HwvtepTerminationPointListener
         if (portAugmentation != null) {
             final NodeId nodeId = identifier.firstIdentifierOf(Node.class).firstKeyOf(Node.class).getNodeId();
             elanClusterUtils.runOnlyInOwnerNode(HwvtepSouthboundConstants.ELAN_ENTITY_NAME,
-                () -> handlePortAdded(portAugmentation, add, nodeId));
+                () -> handlePortAdded(add, nodeId));
             return;
         }
 
@@ -118,8 +118,7 @@ public class HwvtepTerminationPointListener
         return this;
     }
 
-    private List<ListenableFuture<Void>> handlePortAdded(HwvtepPhysicalPortAugmentation portAugmentation,
-                                                         TerminationPoint portAdded, NodeId psNodeId) {
+    private List<ListenableFuture<Void>> handlePortAdded(TerminationPoint portAdded, NodeId psNodeId) {
         Node psNode = HwvtepUtils.getHwVtepNode(broker, LogicalDatastoreType.OPERATIONAL, psNodeId);
         if (psNode != null) {
             String psName = psNode.getAugmentation(PhysicalSwitchAugmentation.class).getHwvtepNodeName().getValue();
index 273c257e92e971a08ea03e71ca96307f53225754..831d81dd0f6453efd75ad09b7e59052dbc9a2209 100644 (file)
@@ -100,8 +100,7 @@ public class LocalUcastMacListener extends ChildListener<Node, LocalUcastMacs, S
     }
 
     @Override
-    protected String getGroup(final InstanceIdentifier<LocalUcastMacs> childIid,
-                              final LocalUcastMacs localUcastMacs) {
+    protected String getGroup(final LocalUcastMacs localUcastMacs) {
         return getElanName(localUcastMacs);
     }
 
index 2f0fe43fcf057ddc57b97a4a3b1ee0cab3475b93..5886cba7afec39a8adf47bacdd3e834d4d70ab82 100644 (file)
@@ -22,7 +22,6 @@ import javax.annotation.Nonnull;
 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.controller.md.sal.common.api.data.ReadFailedException;
 import org.opendaylight.genius.datastoreutils.SingleTransactionDataBroker;
@@ -112,17 +111,15 @@ public class ElanL2GatewayMulticastUtils {
     /**
      * Handle mcast for elan l2 gw device add.
      * @param elanName the elan name
-     * @param elanInstance elanInstace
      * @param device the device
      * @return the listenable future
      */
-    public ListenableFuture<Void> handleMcastForElanL2GwDeviceAdd(String elanName, ElanInstance elanInstance,
-            L2GatewayDevice device) {
+    public ListenableFuture<Void> handleMcastForElanL2GwDeviceAdd(String elanName, L2GatewayDevice device) {
         InstanceIdentifier<ExternalTeps> tepPath = buildExternalTepPath(elanName, device.getTunnelIp());
         JdkFutures.addErrorLogging(txRunner.callWithNewWriteOnlyTransactionAndSubmit(tx -> {
             tx.put(LogicalDatastoreType.CONFIGURATION, tepPath, buildExternalTeps(device));
         }), LOG, "Failed to write to config external tep {}", tepPath);
-        return updateMcastMacsForAllElanDevices(elanName, elanInstance, device, true/* updateThisDevice */);
+        return updateMcastMacsForAllElanDevices(elanName, device, true/* updateThisDevice */);
     }
 
     public static InstanceIdentifier<ExternalTeps> buildExternalTepPath(String elan, IpAddress tepIp) {
@@ -147,7 +144,7 @@ public class ElanL2GatewayMulticastUtils {
     public ListenableFuture<Void> updateRemoteMcastMacOnElanL2GwDevices(String elanName) {
         return txRunner.callWithNewWriteOnlyTransactionAndSubmit(tx -> {
             for (L2GatewayDevice device : ElanL2GwCacheUtils.getInvolvedL2GwDevices(elanName).values()) {
-                prepareRemoteMcastMacUpdateOnDevice(tx, elanName, device);
+                prepareRemoteMcastMacUpdateOnDevice(elanName, device);
             }
         });
     }
@@ -168,17 +165,17 @@ public class ElanL2GatewayMulticastUtils {
      */
     public ListenableFuture<Void> updateRemoteMcastMacOnElanL2GwDevice(String elanName, L2GatewayDevice device) {
         return txRunner.callWithNewWriteOnlyTransactionAndSubmit(
-            tx -> prepareRemoteMcastMacUpdateOnDevice(tx, elanName, device));
+            tx -> prepareRemoteMcastMacUpdateOnDevice(elanName, device));
     }
 
-    public void prepareRemoteMcastMacUpdateOnDevice(WriteTransaction transaction,String elanName,
-                                                           L2GatewayDevice device) {
+    public void prepareRemoteMcastMacUpdateOnDevice(String elanName,
+            L2GatewayDevice device) {
         ConcurrentMap<String, L2GatewayDevice> elanL2gwDevices = ElanL2GwCacheUtils
                 .getInvolvedL2GwDevices(elanName);
         List<DpnInterfaces> dpns = elanUtils.getElanDPNByName(elanName);
         List<IpAddress> dpnsTepIps = getAllTepIpsOfDpns(device, dpns);
         List<IpAddress> l2GwDevicesTepIps = getAllTepIpsOfL2GwDevices(elanL2gwDevices);
-        preapareRemoteMcastMacEntry(transaction, elanName, device, dpnsTepIps, l2GwDevicesTepIps);
+        preapareRemoteMcastMacEntry(elanName, device, dpnsTepIps, l2GwDevicesTepIps);
     }
 
     /**
@@ -188,16 +185,14 @@ public class ElanL2GatewayMulticastUtils {
      *
      * @param elanName
      *            the elan name
-     * @param elanInstance
-     *            the ElanInstance
      * @param device
      *            the device
      * @param updateThisDevice
      *            the update this device
      * @return the listenable future
      */
-    private ListenableFuture<Void> updateMcastMacsForAllElanDevices(String elanName, ElanInstance elanInstance,
-            L2GatewayDevice device, boolean updateThisDevice) {
+    private ListenableFuture<Void> updateMcastMacsForAllElanDevices(String elanName, L2GatewayDevice device,
+                                                                    boolean updateThisDevice) {
 
         SettableFuture<Void> ft = SettableFuture.create();
         ft.set(null);
@@ -216,14 +211,14 @@ public class ElanL2GatewayMulticastUtils {
 
         return txRunner.callWithNewWriteOnlyTransactionAndSubmit(tx -> {
             if (updateThisDevice) {
-                preapareRemoteMcastMacEntry(tx, elanName, device, dpnsTepIps, l2GwDevicesTepIps);
+                preapareRemoteMcastMacEntry(elanName, device, dpnsTepIps, l2GwDevicesTepIps);
             }
 
             // TODO: Need to revisit below logic as logical switches might not be
             // present to configure RemoteMcastMac entry
             for (L2GatewayDevice otherDevice : devices.values()) {
                 if (!otherDevice.getDeviceName().equals(device.getDeviceName())) {
-                    preapareRemoteMcastMacEntry(tx, elanName, otherDevice, dpnsTepIps, l2GwDevicesTepIps);
+                    preapareRemoteMcastMacEntry(elanName, otherDevice, dpnsTepIps, l2GwDevicesTepIps);
                 }
             }
         });
@@ -425,8 +420,6 @@ public class ElanL2GatewayMulticastUtils {
     /**
      * Update remote mcast mac.
      *
-     * @param transaction
-     *            the transaction
      * @param elanName
      *            the elan name
      * @param device
@@ -437,9 +430,9 @@ public class ElanL2GatewayMulticastUtils {
      *            the l2 gw devices tep ips
      * @return the write transaction
      */
-    private void preapareRemoteMcastMacEntry(WriteTransaction transaction, String elanName,
-                                                    L2GatewayDevice device, List<IpAddress> dpnsTepIps,
-                                                    List<IpAddress> l2GwDevicesTepIps) {
+    private void preapareRemoteMcastMacEntry(String elanName,
+                                             L2GatewayDevice device, List<IpAddress> dpnsTepIps,
+                                             List<IpAddress> l2GwDevicesTepIps) {
         NodeId nodeId = new NodeId(device.getHwvtepNodeId());
 
         ArrayList<IpAddress> remoteTepIps = new ArrayList<>(l2GwDevicesTepIps);
@@ -473,7 +466,7 @@ public class ElanL2GatewayMulticastUtils {
             remoteTepIps.add(dhcpDesignatedSwitchTepIp);
         }
         String logicalSwitchName = ElanL2GatewayUtils.getLogicalSwitchFromElan(elanName);
-        putRemoteMcastMac(transaction, nodeId, logicalSwitchName, remoteTepIps);
+        putRemoteMcastMac(nodeId, logicalSwitchName, remoteTepIps);
         LOG.info("Adding RemoteMcastMac for node: {} with physical locators: {}", device.getHwvtepNodeId(),
                 remoteTepIps);
     }
@@ -481,8 +474,6 @@ public class ElanL2GatewayMulticastUtils {
     /**
      * Put remote mcast mac in config DS.
      *
-     * @param transaction
-     *            the transaction
      * @param nodeId
      *            the node id
      * @param logicalSwitchName
@@ -490,7 +481,7 @@ public class ElanL2GatewayMulticastUtils {
      * @param tepIps
      *            the tep ips
      */
-    private static void putRemoteMcastMac(WriteTransaction transaction, NodeId nodeId, String logicalSwitchName,
+    private static void putRemoteMcastMac(NodeId nodeId, String logicalSwitchName,
             ArrayList<IpAddress> tepIps) {
         List<LocatorSet> locators = new ArrayList<>();
         for (IpAddress tepIp : tepIps) {
@@ -557,20 +548,18 @@ public class ElanL2GatewayMulticastUtils {
      *
      * @param elanName
      *            the elan instance name
-     * @param elanInstance
-     *            the ElanInstance
      * @param l2GatewayDevice
      *            the l2 gateway device
      * @return the listenable future
      */
     public List<ListenableFuture<Void>> handleMcastForElanL2GwDeviceDelete(String elanName,
-            ElanInstance elanInstance, L2GatewayDevice l2GatewayDevice) {
+                                                                           L2GatewayDevice l2GatewayDevice) {
         ListenableFuture<Void> deleteTepFuture = txRunner.callWithNewWriteOnlyTransactionAndSubmit(tx -> {
             tx.delete(LogicalDatastoreType.CONFIGURATION,
                     buildExternalTepPath(elanName, l2GatewayDevice.getTunnelIp()));
         });
         ListenableFuture<Void> updateMcastMacsFuture = updateMcastMacsForAllElanDevices(
-                elanName, elanInstance, l2GatewayDevice, false/* updateThisDevice */);
+                elanName, l2GatewayDevice, false/* updateThisDevice */);
         ListenableFuture<Void> deleteRemoteMcastMacFuture = deleteRemoteMcastMac(
                 new NodeId(l2GatewayDevice.getHwvtepNodeId()), elanName);
         return Arrays.asList(updateMcastMacsFuture, deleteRemoteMcastMacFuture, deleteTepFuture);
index d6a39be726e22217e6878b89fc9f7c2907d5142a..5373249cdded4ede814009594bc63327b9c6d197 100644 (file)
@@ -650,7 +650,7 @@ public class ElanL2GatewayUtils {
 
         List<RemoteUcastMacs> lstL2GatewayDevicesMacs = getOtherDevicesMacs(elanName, l2GatewayDevice, hwVtepNodeId,
                 logicalSwitchName);
-        List<RemoteUcastMacs> lstElanMacTableEntries = getElanMacTableEntriesMacs(elanName, l2GatewayDevice,
+        List<RemoteUcastMacs> lstElanMacTableEntries = getElanMacTableEntriesMacs(elanName,
                 hwVtepNodeId, logicalSwitchName);
 
         List<RemoteUcastMacs> lstRemoteUcastMacs = new ArrayList<>(lstL2GatewayDevicesMacs);
@@ -725,8 +725,6 @@ public class ElanL2GatewayUtils {
      *
      * @param elanName
      *            the elan name
-     * @param l2GatewayDeviceToBeConfigured
-     *            the l2 gateway device to be configured
      * @param hwVtepNodeId
      *            the hw vtep node id
      * @param logicalSwitchName
@@ -734,7 +732,7 @@ public class ElanL2GatewayUtils {
      * @return the elan mac table entries as remote ucast macs
      */
     public List<RemoteUcastMacs> getElanMacTableEntriesMacs(String elanName,
-            L2GatewayDevice l2GatewayDeviceToBeConfigured, NodeId hwVtepNodeId, String logicalSwitchName) {
+            NodeId hwVtepNodeId, String logicalSwitchName) {
         List<RemoteUcastMacs> lstRemoteUcastMacs = new ArrayList<>();
 
         MacTable macTable = ElanUtils.getElanMacTable(broker, elanName);
@@ -925,13 +923,11 @@ public class ElanL2GatewayUtils {
     /**
      * Gets the l2 gateway connection job key.
      *
-     * @param nodeId
-     *            the node id
      * @param logicalSwitchName
      *            the logical switch name
      * @return the l2 gateway connection job key
      */
-    public static String getL2GatewayConnectionJobKey(String nodeId, String logicalSwitchName) {
+    public static String getL2GatewayConnectionJobKey(String logicalSwitchName) {
         return logicalSwitchName;
     }
 
index 24cd393778bd53d3932b95b0a76c8f8d60b7bed1..c21dbac829b7807c9f6f52b21d86adbf191a4d12 100644 (file)
@@ -31,7 +31,6 @@ import org.opendaylight.genius.utils.hwvtep.HwvtepSouthboundUtils;
 import org.opendaylight.genius.utils.hwvtep.HwvtepUtils;
 import org.opendaylight.infrautils.jobcoordinator.JobCoordinator;
 import org.opendaylight.netvirt.elan.cache.ElanInstanceCache;
-import org.opendaylight.netvirt.elan.internal.ElanInstanceManager;
 import org.opendaylight.netvirt.elan.l2gw.ha.listeners.HAOpClusteredListener;
 import org.opendaylight.netvirt.elan.l2gw.jobs.AssociateHwvtepToElanJob;
 import org.opendaylight.netvirt.elan.l2gw.jobs.DisAssociateHwvtepFromElanJob;
@@ -64,7 +63,6 @@ public class L2GatewayConnectionUtils implements AutoCloseable {
     private static final Logger LOG = LoggerFactory.getLogger(L2GatewayConnectionUtils.class);
 
     private final DataBroker broker;
-    private final ElanInstanceManager elanInstanceManager;
     private final ElanL2GatewayUtils elanL2GatewayUtils;
     private final ElanClusterUtils elanClusterUtils;
     private final ElanL2GatewayMulticastUtils elanL2GatewayMulticastUtils;
@@ -75,13 +73,12 @@ public class L2GatewayConnectionUtils implements AutoCloseable {
     private final HAOpClusteredListener haOpClusteredListener;
 
     @Inject
-    public L2GatewayConnectionUtils(DataBroker dataBroker, ElanInstanceManager elanInstanceManager,
+    public L2GatewayConnectionUtils(DataBroker dataBroker,
             ElanClusterUtils elanClusterUtils, ElanL2GatewayUtils elanL2GatewayUtils,
             JobCoordinator jobCoordinator, ElanL2GatewayMulticastUtils elanL2GatewayMulticastUtils,
             L2GatewayCache l2GatewayCache, HAOpClusteredListener haOpClusteredListener,
             ElanInstanceCache elanInstanceCache) {
         this.broker = dataBroker;
-        this.elanInstanceManager = elanInstanceManager;
         this.elanL2GatewayUtils = elanL2GatewayUtils;
         this.elanClusterUtils = elanClusterUtils;
         this.elanL2GatewayMulticastUtils = elanL2GatewayMulticastUtils;
index 1ae9495368fd098bfdfd3c5f23de15f7046246b9..1d1c1c72a1b811c6b797e20e4fd534363b3818ab 100644 (file)
@@ -85,7 +85,7 @@ public class L2gwServiceProvider extends AbstractLifecycle implements IL2gwServi
     }
 
     @Override
-    public List<L2gatewayConnection> getAssociatedL2GwConnections(DataBroker broker, Set<Uuid> l2GatewayIds) {
+    public List<L2gatewayConnection> getAssociatedL2GwConnections(Set<Uuid> l2GatewayIds) {
         return L2GatewayConnectionUtils.getAssociatedL2GwConnections(dataBroker, l2GatewayIds);
     }
 
index 4f2a97b685d09e8875c92845d145f01e233dd5fd..8b508be52c5b40adf5cc81603dc646087a9fde57 100644 (file)
@@ -329,7 +329,7 @@ public class ElanDmacUtils {
     }
 
     private ListenableFuture<Void> buildEtreeDmacFlowDropIfPacketComingFromTunnelwithBatch(
-            BigInteger dpnId, String extDeviceNodeId, Long elanTag, String macAddress,EtreeLeafTagName etreeLeafTag) {
+            BigInteger dpnId, String extDeviceNodeId, String macAddress, EtreeLeafTagName etreeLeafTag) {
         if (etreeLeafTag != null) {
             Flow dropFlow = buildDmacFlowDropIfPacketComingFromTunnel(dpnId, extDeviceNodeId,
                     etreeLeafTag.getEtreeLeafTag().getValue(), macAddress);
@@ -365,7 +365,7 @@ public class ElanDmacUtils {
         if (etreeLeafTag != null) {
             return Lists.newArrayList(
                 buildEtreeDmacFlowDropIfPacketComingFromTunnelwithBatch(
-                        dpnId, extDeviceNodeId, elanTag, macAddress, etreeLeafTag),
+                        dpnId, extDeviceNodeId, macAddress, etreeLeafTag),
                 buildEtreeDmacFlowForExternalRemoteMacWithBatch(
                         dpnId, extDeviceNodeId, vni, macAddress, displayName, interfaceName, etreeLeafTag));
         }
index b9566ed44d8feda79761b648d56f3685cd6beab4..c65587727767dbbadd50838da5c05749f263dc65 100644 (file)
@@ -13,7 +13,6 @@ import com.google.common.util.concurrent.FutureCallback;
 import java.math.BigInteger;
 import java.util.List;
 
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.fibmanager.rev150330.RouterInterface;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.fibmanager.rev150330.vrfentries.VrfEntry;
@@ -51,22 +50,22 @@ public interface IFibManager {
                                 String destTepIp,
                                 long label);
 
-    void addOrUpdateFibEntry(DataBroker broker, String rd, String macAddress, String prefix, List<String> nextHopList,
+    void addOrUpdateFibEntry(String rd, String macAddress, String prefix, List<String> nextHopList,
                              VrfEntry.EncapType encapType, long label, long l3vni, String gwMacAddress,
                              String parentVpnRd, RouteOrigin origin, WriteTransaction writeConfigTxn);
 
-    void addFibEntryForRouterInterface(DataBroker broker, String rd, String prefix,
+    void addFibEntryForRouterInterface(String rd, String prefix,
                                        RouterInterface routerInterface, long label, WriteTransaction writeConfigTxn);
 
-    void removeOrUpdateFibEntry(DataBroker broker, String rd, String prefix, String nextHopToRemove,
+    void removeOrUpdateFibEntry(String rd, String prefix, String nextHopToRemove,
                                 WriteTransaction writeConfigTxn);
 
-    void removeFibEntry(DataBroker broker, String rd, String prefix, WriteTransaction writeConfigTxn);
+    void removeFibEntry(String rd, String prefix, WriteTransaction writeConfigTxn);
 
-    void updateRoutePathForFibEntry(DataBroker broker, String rd, String prefix, String nextHop,
-                        long label, boolean nextHopAdd, WriteTransaction writeConfigTxn);
+    void updateRoutePathForFibEntry(String rd, String prefix, String nextHop,
+                                    long label, boolean nextHopAdd, WriteTransaction writeConfigTxn);
 
-    void removeVrfTable(DataBroker broker, String rd, WriteTransaction writeConfigTxn);
+    void removeVrfTable(String rd, WriteTransaction writeConfigTxn);
 
     void removeInterVPNLinkRouteFlows(String interVpnLinkName,
                                       boolean isVpnFirstEndPoint,
index f7527a304d1179942f9e857a9bd16ba741a2d444..4e786feaf3f34ec1f1624640cbd34414c5bd8eb1 100644 (file)
@@ -370,7 +370,7 @@ public class BgpRouteVrfEntryHandler extends BaseVrfEntryHandler
     }
 
     public Consumer<? super VrfEntry> getConsumerForDeletingRemoteFib(
-            final BigInteger dpnId, final long vpnId, final String rd,
+            final BigInteger dpnId, final long vpnId,
             final String remoteNextHopIp, final Optional<VrfTables> vrfTable,
             WriteTransaction writeCfgTxn, List<SubTransaction> subTxns) {
         return vrfEntry -> vrfEntry.getRoutePaths().stream()
index e87486950a0e02b07837ac0b0e4262752a58832e..71ff1f7ef4c4fe7f9a8c187db952a5ae05d2cf9f 100755 (executable)
@@ -15,7 +15,6 @@ import java.util.List;
 import javax.inject.Inject;
 import javax.inject.Singleton;
 import org.opendaylight.controller.config.api.osgi.WaitingServiceTracker;
-import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.controller.md.sal.binding.api.WriteTransaction;
 import org.opendaylight.netvirt.fibmanager.api.IFibManager;
 import org.opendaylight.netvirt.fibmanager.api.RouteOrigin;
@@ -121,40 +120,40 @@ public class FibManagerImpl implements IFibManager {
     }
 
     @Override
-    public void addOrUpdateFibEntry(DataBroker broker, String rd, String macAddress, String prefix,
-                                    List<String> nextHopList, VrfEntry.EncapType encapType, long label,
-                                    long l3vni, String gwMacAddress, String parentVpnRd, RouteOrigin origin,
-                                    WriteTransaction writeConfigTxn) {
+    public void addOrUpdateFibEntry(String rd, String macAddress, String prefix,
+            List<String> nextHopList, VrfEntry.EncapType encapType, long label,
+            long l3vni, String gwMacAddress, String parentVpnRd, RouteOrigin origin,
+            WriteTransaction writeConfigTxn) {
         fibUtil.addOrUpdateFibEntry(rd, macAddress, prefix, nextHopList , encapType, label, l3vni, gwMacAddress,
                 parentVpnRd, origin, writeConfigTxn);
     }
 
     @Override
-    public void addFibEntryForRouterInterface(DataBroker broker, String rd, String prefix,
-                                              RouterInterface routerInterface, long label,
-                                              WriteTransaction writeConfigTxn) {
+    public void addFibEntryForRouterInterface(String rd, String prefix,
+            RouterInterface routerInterface, long label,
+            WriteTransaction writeConfigTxn) {
         fibUtil.addFibEntryForRouterInterface(rd, prefix, routerInterface, label, writeConfigTxn);
     }
 
     @Override
-    public void removeOrUpdateFibEntry(DataBroker broker, String rd, String prefix,
-                                       String nextHopToRemove, WriteTransaction writeConfigTxn) {
+    public void removeOrUpdateFibEntry(String rd, String prefix,
+            String nextHopToRemove, WriteTransaction writeConfigTxn) {
         fibUtil.removeOrUpdateFibEntry(rd, prefix, nextHopToRemove, writeConfigTxn);
     }
 
     @Override
-    public void removeFibEntry(DataBroker broker, String rd, String prefix, WriteTransaction writeConfigTxn) {
+    public void removeFibEntry(String rd, String prefix, WriteTransaction writeConfigTxn) {
         fibUtil.removeFibEntry(rd, prefix, writeConfigTxn);
     }
 
     @Override
-    public void updateRoutePathForFibEntry(DataBroker broker, String rd, String prefix, String nextHop,
-                               long label, boolean nextHopAdd, WriteTransaction writeConfigTxn) {
+    public void updateRoutePathForFibEntry(String rd, String prefix, String nextHop,
+            long label, boolean nextHopAdd, WriteTransaction writeConfigTxn) {
         fibUtil.updateRoutePathForFibEntry(rd, prefix, nextHop, label, nextHopAdd, writeConfigTxn);
     }
 
     @Override
-    public void removeVrfTable(DataBroker broker, String rd, WriteTransaction writeConfigTxn) {
+    public void removeVrfTable(String rd, WriteTransaction writeConfigTxn) {
         fibUtil.removeVrfTable(rd, writeConfigTxn);
     }
 
index b2b30951b42253f5dd88f89f6030bcffefb9f96d..12848837c3bf8f406638d278455706dd54cb6371 100755 (executable)
@@ -1835,7 +1835,7 @@ public class VrfEntryListener extends AsyncDataTreeChangeListenerBase<VrfEntry,
                         WriteTransaction writeCfgTxn = dataBroker.newWriteOnlyTransaction();
                         vrfTable.get().getVrfEntry().stream()
                             .filter(vrfEntry -> RouteOrigin.value(vrfEntry.getOrigin()) == RouteOrigin.BGP)
-                            .forEach(bgpRouteVrfEntryHandler.getConsumerForDeletingRemoteFib(dpnId, vpnId, rd,
+                            .forEach(bgpRouteVrfEntryHandler.getConsumerForDeletingRemoteFib(dpnId, vpnId,
                                 remoteNextHopIp, vrfTable, writeCfgTxn, txnObjects));
                         futures.add(writeCfgTxn.submit());
                     }
index 0476f789ba5fa14a60eb3a28737e76aa1b782977..59bac165c18aa50e14f019e14ac979f50266b7f9 100644 (file)
@@ -147,8 +147,8 @@ public abstract class AbstractSnatService implements SnatServiceListener {
         if (addOrRemove == NwConstants.ADD_FLOW) {
             extSubnetId = NatUtil.getExternalSubnetVpnId(dataBroker,externalSubnetId);
         }
-        installInboundFibEntry(dpnId, externalIp, routerName, routerId, extSubnetId, addOrRemove);
-        installInboundTerminatingServiceTblEntry(dpnId, routerId, routerName, externalIp, extSubnetId, addOrRemove);
+        installInboundFibEntry(dpnId, externalIp, routerId, extSubnetId, addOrRemove);
+        installInboundTerminatingServiceTblEntry(dpnId, routerId, extSubnetId, addOrRemove);
     }
 
     protected void installSnatCommonEntriesForNonNaptSwitch(Routers routers, BigInteger primarySwitchId,
@@ -179,7 +179,7 @@ public abstract class AbstractSnatService implements SnatServiceListener {
     protected abstract void installSnatSpecificEntriesForNonNaptSwitch(Routers routers, BigInteger dpnId,
             int addOrRemove);
 
-    protected void installInboundFibEntry(BigInteger dpnId, String externalIp, String routerName, Long routerId,
+    protected void installInboundFibEntry(BigInteger dpnId, String externalIp, Long routerId,
             long extSubnetId, int addOrRemove) {
         List<MatchInfo> matches = new ArrayList<>();
         matches.add(MatchEthernetType.IPV4);
@@ -265,8 +265,8 @@ public abstract class AbstractSnatService implements SnatServiceListener {
                 NwConstants.COOKIE_SNAT_TABLE, matches, instructions, addOrRemove);
     }
 
-    protected void installInboundTerminatingServiceTblEntry(BigInteger dpnId, Long  routerId, String routerName,
-            String externalIp,  long extSubnetId, int addOrRemove) {
+    protected void installInboundTerminatingServiceTblEntry(BigInteger dpnId, Long routerId,
+            long extSubnetId, int addOrRemove) {
         //Install the tunnel table entry in NAPT switch for inbound traffic to SNAP IP from a non a NAPT switch.
         LOG.info("installInboundTerminatingServiceTblEntry : creating entry for Terminating Service Table "
                 + "for switch {}, routerId {}", dpnId, routerId);
index f49b810f27e0d329482db4f956a4318d840575e6..84f3daea78abc61ef9960dd88b43d91d5604f20f 100644 (file)
@@ -89,11 +89,10 @@ public abstract class ConntrackBasedSnatService extends AbstractSnatService {
             extSubnetId = NatUtil.getExternalSubnetVpnId(getDataBroker(),externalSubnetId);
         }
         createOutboundTblEntry(dpnId, routerId, externalIp, elanId, extGwMacAddress, addOrRemove);
-        installNaptPfibFlow(routers, dpnId, routerId, routerName, extSubnetId, addOrRemove);
+        installNaptPfibFlow(routers, dpnId, routerId, extSubnetId, addOrRemove);
 
         //Install Inbound NAT entries
-        Long extNetId = NatUtil.getVpnId(getDataBroker(), routers.getNetworkId().getValue());
-        installInboundEntry(dpnId, routerId, routerName, extNetId, externalIp, elanId, extSubnetId, addOrRemove);
+        installInboundEntry(dpnId, routerId, externalIp, elanId, extSubnetId, addOrRemove);
         installNaptPfibEntry(dpnId, routerId, addOrRemove);
 
     }
@@ -203,7 +202,7 @@ public abstract class ConntrackBasedSnatService extends AbstractSnatService {
     }
 
     protected void installNaptPfibFlow(Routers routers, BigInteger dpnId, long routerId,
-            String routerName, long extSubnetId, int addOrRemove) {
+            long extSubnetId, int addOrRemove) {
         Long extNetId = NatUtil.getVpnId(getDataBroker(), routers.getNetworkId().getValue());
         LOG.info("installNaptPfibFlow : dpId {}, extNetId {}", dpnId, extNetId);
         List<MatchInfoBase> matches = new ArrayList<>();
@@ -230,8 +229,8 @@ public abstract class ConntrackBasedSnatService extends AbstractSnatService {
                 flowRef, NwConstants.COOKIE_SNAT_TABLE, matches, instructions, addOrRemove);
     }
 
-    protected void installInboundEntry(BigInteger dpnId, long routerId, String routerName, Long extNetId,
-            String externalIp, int elanId, long extSubnetId, int addOrRemove) {
+    protected void installInboundEntry(BigInteger dpnId, long routerId, String externalIp, int elanId, long extSubnetId,
+            int addOrRemove) {
         LOG.info("installInboundEntry : dpId {} and routerId {}", dpnId, routerId);
         List<MatchInfoBase> matches = new ArrayList<>();
         matches.add(MatchEthernetType.IPV4);
index 64a26cc65f6aaadc7faefca9ccd62f827fffa498..21d5c1f57cf8a0204f051e6f79ac8ad14db1841f 100644 (file)
@@ -252,7 +252,7 @@ public class EvpnDnatFlowProgrammer {
 
     public void onRemoveFloatingIp(final BigInteger dpnId, final String vpnName, final String externalIp,
                                    final String floatingIpInterface, final String floatingIpPortMacAddress,
-                                   final String routerName, final long routerId, WriteTransaction removeFlowInvTx) {
+                                   final long routerId, WriteTransaction removeFlowInvTx) {
     /*
      *  1) Remove the flow INTERNAL_TUNNEL_TABLE (table=36)-> PDNAT_TABLE (table=25) (SNAT VM on DPN1 is
      *     responding back to FIP VM on DPN2) {SNAT to DNAT traffic on different Hypervisor}
@@ -284,7 +284,7 @@ public class EvpnDnatFlowProgrammer {
         String fibExternalIp = NatUtil.validateAndAddNetworkMask(externalIp);
 
         //Remove Prefix from BGP
-        NatUtil.removePrefixFromBGP(dataBroker, bgpManager, fibManager, rd, fibExternalIp, vpnName, LOG);
+        NatUtil.removePrefixFromBGP(bgpManager, fibManager, rd, fibExternalIp, vpnName, LOG);
 
         //Remove custom FIB routes flow for L3_FIB_TABLE (table=21)-> PDNAT_TABLE (table=25)
         RemoveFibEntryInput input = new RemoveFibEntryInputBuilder().setVpnName(vpnName)
index e405e87bbf51c72cfbb9336fb8c7db39c9a8648b..1dce2ca9955c1b72f39f792a19b4ea861c320cf4 100644 (file)
@@ -145,7 +145,7 @@ public class ExternalNetworksChangeListener
             Collection<String> externalIps = NatUtil.getExternalIpsForRouter(dataBroker,routerId);
             if (natMode == NatMode.Controller) {
                 externalRouterListener.handleDisableSnatInternetVpn(routerUuid.getValue(), routerId, networkUuid,
-                        externalIps, false, original.getVpnid().getValue(), writeFlowInvTx);
+                        externalIps, original.getVpnid().getValue(), writeFlowInvTx);
             }
         }
     }
index e12145dffd343d82fce4189e44433cbc278da4e6..3c23b873753e324f3040906bfe38f1f6ca4087c6 100644 (file)
@@ -265,7 +265,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
                     addOrDelDefFibRouteToSNAT(routerName, routerId, bgpVpnId, bgpVpnUuid, true, writeFlowInvTx);
                     List<ListenableFuture<Void>> futures = new ArrayList<>();
                     // Allocate Primary Napt Switch for this router
-                    BigInteger primarySwitchId = getPrimaryNaptSwitch(routerName, routerId);
+                    BigInteger primarySwitchId = getPrimaryNaptSwitch(routerName);
                     if (primarySwitchId != null && !primarySwitchId.equals(BigInteger.ZERO)) {
                         if (!routers.isEnableSnat()) {
                             LOG.info("add : SNAT is disabled for external router {} ", routerName);
@@ -317,7 +317,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
                 // Handle switches and NAPT switches separately
                 if (!dpnId.equals(primarySwitchId)) {
                     LOG.debug("handleEnableSnat : Handle Ordinary switch");
-                    handleSwitches(dpnId, routerName, routerId, primarySwitchId, writeFlowInvTx);
+                    handleSwitches(dpnId, routerName, routerId, primarySwitchId);
                 } else {
                     LOG.debug("handleEnableSnat : Handle NAPT switch");
                     handlePrimaryNaptSwitch(dpnId, routerName, routerId, writeFlowInvTx);
@@ -340,7 +340,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
         LOG.debug("handleEnableSnat : Exit");
     }
 
-    private BigInteger getPrimaryNaptSwitch(String routerName, long segmentId) {
+    private BigInteger getPrimaryNaptSwitch(String routerName) {
         // Allocate Primary Napt Switch for this router
         BigInteger primarySwitchId = NatUtil.getPrimaryNaptfromRouterName(dataBroker, routerName);
         if (primarySwitchId != null && !primarySwitchId.equals(BigInteger.ZERO)) {
@@ -832,8 +832,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
         }
     }
 
-    protected void handleSwitches(BigInteger dpnId, String routerName, long routerId, BigInteger primarySwitchId,
-            WriteTransaction writeFlowInvTx) {
+    protected void handleSwitches(BigInteger dpnId, String routerName, long routerId, BigInteger primarySwitchId) {
         LOG.debug("handleSwitches : Installing SNAT miss entry in switch {}", dpnId);
         List<ActionInfo> listActionInfoPrimary = new ArrayList<>();
         String ifNamePrimary = getTunnelInterfaceName(dpnId, primarySwitchId);
@@ -1042,7 +1041,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
                         }
                         Routers extRouter = router != null ? router :
                             NatUtil.getRoutersFromConfigDS(dataBroker, routerName);
-                        Uuid externalSubnetId = NatUtil.getExternalSubnetForRouterExternalIp(dataBroker, externalIp,
+                        Uuid externalSubnetId = NatUtil.getExternalSubnetForRouterExternalIp(externalIp,
                                 extRouter);
                         NatUtil.addPrefixToBGP(dataBroker, bgpManager, fibManager, vpnName, rd, externalSubnetId,
                             externalIp, nextHopIp, extRouter.getNetworkId().getValue(), null, label, l3vni,
@@ -1220,7 +1219,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
             if (bgpVpnUuid != null) {
                 bgpVpnId = NatUtil.getVpnId(dataBroker, bgpVpnUuid.getValue());
             }
-            BigInteger dpnId = getPrimaryNaptSwitch(routerName, routerId);
+            BigInteger dpnId = getPrimaryNaptSwitch(routerName);
             if (dpnId == null || dpnId.equals(BigInteger.ZERO)) {
                 // Router has no interface attached
                 return;
@@ -1286,7 +1285,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
                     LOG.debug("update : Start processing of the External IPs removal during the update operation");
                     vpnManager.removeArpResponderFlowsToExternalNetworkIps(routerName,
                             removedExternalIps, original.getExtGwMacAddress(),
-                            dpnId, networkId, null);
+                            dpnId, networkId);
 
                     for (String removedExternalIp : removedExternalIps) {
                     /*
@@ -1736,7 +1735,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
             Collection<Uuid> externalSubnetList = NatUtil.getExternalSubnetIdsFromExternalIps(router.getExternalIps());
             removeNaptFlowsFromActiveSwitch(routerId, routerName, naptSwitchDpnId, networkUuid, vpnName, externalIps,
                     externalSubnetList, removeFlowInvTx, extNwProvType);
-            removeFlowsFromNonActiveSwitches(routerId, routerName, naptSwitchDpnId, networkUuid, removeFlowInvTx);
+            removeFlowsFromNonActiveSwitches(routerId, routerName, naptSwitchDpnId, removeFlowInvTx);
             try {
                 String externalSubnetVpn = null;
                 for (Uuid externalSubnetId : externalSubnetList) {
@@ -1770,7 +1769,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
     // TODO Clean up the exception handling
     @SuppressWarnings("checkstyle:IllegalCatch")
     public void handleDisableSnatInternetVpn(String routerName, long routerId, Uuid networkUuid,
-                                             @Nonnull Collection<String> externalIps, boolean routerFlag,
+                                             @Nonnull Collection<String> externalIps,
                                              String vpnId, WriteTransaction writeFlowInvTx) {
         LOG.debug("handleDisableSnatInternetVpn: Started to process handle disable snat for router {} "
                 + "with internet vpn {}", routerName, vpnId);
@@ -2113,7 +2112,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
     }
 
     public void removeFlowsFromNonActiveSwitches(long routerId, String routerName,
-            BigInteger naptSwitchDpnId, Uuid networkId, WriteTransaction removeFlowInvTx) {
+            BigInteger naptSwitchDpnId, WriteTransaction removeFlowInvTx) {
         LOG.debug("removeFlowsFromNonActiveSwitches : Remove NAPT related flows from non active switches");
 
         // Remove the flows from the other switches which points to the primary and secondary switches
@@ -2425,7 +2424,7 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
         //Inform BGP about the route removal
         LOG.info("clearBgpRoutes : Informing BGP to remove route for externalIP {} of vpn {}", externalIp, vpnName);
         String rd = NatUtil.getVpnRd(dataBroker, vpnName);
-        NatUtil.removePrefixFromBGP(dataBroker, bgpManager, fibManager, rd, externalIp, vpnName, LOG);
+        NatUtil.removePrefixFromBGP(bgpManager, fibManager, rd, externalIp, vpnName, LOG);
     }
 
     private void removeTunnelTableEntry(BigInteger dpnId, long serviceId, WriteTransaction writeFlowInvTx) {
@@ -2538,13 +2537,6 @@ public class ExternalRoutersListener extends AsyncDataTreeChangeListenerBase<Rou
     public void installFlowsWithUpdatedVpnId(BigInteger primarySwitchId, String routerName, long bgpVpnId,
                                              long routerId, boolean isSnatCfgd, WriteTransaction writeFlowInvTx,
                                              ProviderTypes extNwProvType) {
-        installFlowsWithUpdatedVpnId(primarySwitchId, routerName, bgpVpnId, routerId, isSnatCfgd, null, writeFlowInvTx,
-                extNwProvType);
-    }
-
-    public void installFlowsWithUpdatedVpnId(BigInteger primarySwitchId, String routerName, long bgpVpnId,
-                                             long routerId, boolean isSnatCfgd, Routers router,
-                                             WriteTransaction writeFlowInvTx, ProviderTypes extNwProvType) {
         long changedVpnId = bgpVpnId;
         String logMsg = "installFlowsWithUpdatedVpnId : Update the BGP VPN ID {}";
         if (bgpVpnId == NatConstants.INVALID_ID) {
index e1cc57e5c685b81138d5af03e50d0e242d14141e..87839c88725aafa176deddb5b10c27a95adb8b8e 100644 (file)
@@ -277,7 +277,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
 
     }
 
-    private void createDNATTblEntry(BigInteger dpnId, InternalToExternalPortMap mapping, long routerId, long vpnId,
+    private void createDNATTblEntry(BigInteger dpnId, InternalToExternalPortMap mapping, long routerId,
                                     long associatedVpnId, WriteTransaction writeFlowInvTx) {
         FlowEntity preFlowEntity = buildPreDNATFlowEntity(dpnId, mapping, routerId, associatedVpnId);
         if (preFlowEntity == null) {
@@ -478,7 +478,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
             addOrDelDefaultFibRouteForDnat(dpnId, routerName, routerId, writeFlowInvTx, true);
         }
         //Create the DNAT and SNAT table entries
-        createDNATTblEntry(dpnId, mapping, routerId, vpnId, associatedVpnId, writeFlowInvTx);
+        createDNATTblEntry(dpnId, mapping, routerId, associatedVpnId, writeFlowInvTx);
         createSNATTblEntry(dpnId, mapping, vpnId, routerId, associatedVpnId, extNwId, writeFlowInvTx);
         floatingIPHandler.onAddFloatingIp(dpnId, routerName, routerId, extNwId, interfaceName, mapping, writeFlowInvTx);
     }
@@ -513,7 +513,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
             addOrDelDefaultFibRouteForDnat(dpnId, routerName, routerId, writeFlowInvTx, true);
         }
         //Create the DNAT and SNAT table entries
-        createDNATTblEntry(dpnId, mapping, routerId, vpnId, associatedVpnId, writeFlowInvTx);
+        createDNATTblEntry(dpnId, mapping, routerId, associatedVpnId, writeFlowInvTx);
         createSNATTblEntry(dpnId, mapping, vpnId, routerId, associatedVpnId, externalNetworkId, writeFlowInvTx);
         floatingIPHandler.onAddFloatingIp(dpnId, routerName, routerId, externalNetworkId, interfaceName, mapping,
                 writeFlowInvTx);
@@ -612,7 +612,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
         if (provType == ProviderTypes.VXLAN) {
             floatingIPHandler.onRemoveFloatingIp(dpnId, routerName, routerId, extNwId, mapping,
                     NatConstants.DEFAULT_L3VNI_VALUE, removeFlowInvTx);
-            removeOperationalDS(routerName, interfaceName, internalIp, externalIp);
+            removeOperationalDS(routerName, interfaceName, internalIp);
             return;
         }
         long label = getOperationalIpMapping(routerName, interfaceName, internalIp);
@@ -623,7 +623,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
         }
         floatingIPHandler.onRemoveFloatingIp(dpnId, routerName, routerId, extNwId, mapping, (int) label,
                 removeFlowInvTx);
-        removeOperationalDS(routerName, interfaceName, internalIp, externalIp);
+        removeOperationalDS(routerName, interfaceName, internalIp);
     }
 
     void removeNATFlowEntries(BigInteger dpnId, String interfaceName, String vpnName, String routerName,
@@ -660,7 +660,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
         if (provType == ProviderTypes.VXLAN) {
             floatingIPHandler.cleanupFibEntries(dpnId, vpnName, externalIp, NatConstants.DEFAULT_L3VNI_VALUE,
                     removeFlowInvTx, provType);
-            removeOperationalDS(routerName, interfaceName, internalIp, externalIp);
+            removeOperationalDS(routerName, interfaceName, internalIp);
             return;
         }
         long label = getOperationalIpMapping(routerName, interfaceName, internalIp);
@@ -670,7 +670,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
             return;
         }
         floatingIPHandler.cleanupFibEntries(dpnId, vpnName, externalIp, label, removeFlowInvTx, provType);
-        removeOperationalDS(routerName, interfaceName, internalIp, externalIp);
+        removeOperationalDS(routerName, interfaceName, internalIp);
     }
 
     protected long getOperationalIpMapping(String routerId, String interfaceName, String internalIp) {
@@ -709,7 +709,7 @@ public class FloatingIPListener extends AsyncDataTreeChangeListenerBase<Internal
         }
     }
 
-    void removeOperationalDS(String routerId, String interfaceName, String internalIp, String externalIp) {
+    void removeOperationalDS(String routerId, String interfaceName, String internalIp) {
         LOG.info("removeOperationalDS : Remove operational DS for floating ip config: {}", internalIp);
         InstanceIdentifier<InternalToExternalPortMap> intExtPortMapId = NatUtil.getIntExtPortMapIdentifier(routerId,
                 interfaceName, internalIp);
index 2fea7e522b9e36658c1f73d4f85e45295bc87eba..9a72205c864e47372587f2374479904c674e10ac 100644 (file)
@@ -43,7 +43,6 @@ import org.opendaylight.genius.mdsalutil.instructions.InstructionGotoTable;
 import org.opendaylight.genius.mdsalutil.interfaces.IMdsalApiManager;
 import org.opendaylight.genius.mdsalutil.matches.MatchEthernetType;
 import org.opendaylight.genius.mdsalutil.matches.MatchMetadata;
-import org.opendaylight.netvirt.bgpmanager.api.IBgpManager;
 import org.opendaylight.netvirt.elanmanager.api.IElanService;
 import org.opendaylight.netvirt.fibmanager.api.IFibManager;
 import org.opendaylight.netvirt.natservice.api.SnatServiceManager;
@@ -61,7 +60,6 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.NodeId;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.Nodes;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.Node;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.inventory.rev130819.nodes.NodeKey;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.fib.rpc.rev160121.FibRpcService;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.fibmanager.rev150330.FibEntries;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.fibmanager.rev150330.fibentries.VrfTables;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.fibmanager.rev150330.fibentries.VrfTablesKey;
@@ -82,7 +80,6 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev16011
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.napt.switches.RouterToNaptSwitch;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.napt.switches.RouterToNaptSwitchBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.natservice.rev160111.napt.switches.RouterToNaptSwitchKey;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.vpn.rpc.rev160201.VpnRpcService;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.slf4j.Logger;
@@ -114,9 +111,6 @@ public class NaptSwitchHA {
                         final OdlInterfaceRpcService interfaceManager,
                         final IdManagerService idManager,
                         final NAPTSwitchSelector naptSwitchSelector,
-                        final IBgpManager bgpManager,
-                        final VpnRpcService vpnService,
-                        final FibRpcService fibService,
                         final IFibManager fibManager,
                         final EvpnNaptSwitchHA evpnNaptSwitchHA,
                         final IElanService elanManager,
@@ -449,7 +443,7 @@ public class NaptSwitchHA {
                             String rd = NatUtil.getVpnRd(dataBroker, vpnName);
                             for (String externalIp : externalIpsCache) {
                                 LOG.debug("isNaptSwitchDown : Removing Fib entry rd {} prefix {}", rd, externalIp);
-                                fibManager.removeFibEntry(dataBroker, rd, externalIp, null);
+                                fibManager.removeFibEntry(rd, externalIp, null);
                             }
                         }
                     } else {
@@ -1056,7 +1050,7 @@ public class NaptSwitchHA {
         Optional<VrfEntry> ent = MDSALUtil.read(dataBroker, LogicalDatastoreType.CONFIGURATION, vrfEntryId);
         if (ent.isPresent()) {
             LOG.debug("removeFibEntry : Removing Fib entry rd {} prefix {}", rd, prefix);
-            fibManager.removeFibEntry(dataBroker, rd, prefix, null);
+            fibManager.removeFibEntry(rd, prefix, null);
         }
     }
 
index dc03f339ac716b86c2a6960f32d6f9ad90791e54..09f2e12d73bef7ce6a35b8e05837690ea436e36b 100644 (file)
@@ -181,7 +181,7 @@ public final class NatEvpnUtil {
             NatUtil.addPrefixToInterface(broker, NatUtil.getVpnId(broker, vpnName), interfaceName, prefix, dpId,
                     null /* subnet-id */, Prefixes.PrefixCue.Nat);
 
-            fibManager.addOrUpdateFibEntry(broker, rd, null /*macAddress*/, prefix,
+            fibManager.addOrUpdateFibEntry(rd, null /*macAddress*/, prefix,
                     Collections.singletonList(nextHopIp), VrfEntry.EncapType.Vxlan, NatConstants.DEFAULT_LABEL_VALUE,
                     l3Vni, gwMacAddress, null /* parent-vpn-rd */, origin, writeTx);
             /* Publish to Bgp only if its an INTERNET VPN */
index dbe45a3ddbc28447347923c10ce5e784f16d0bf9..a5f5b485e5d76e3d42bfe3dc584086c399b8ad72 100644 (file)
@@ -479,7 +479,7 @@ public class NatTunnelInterfaceStateListener
             if (extNwProvType == null) {
                 return false;
             }
-            hndlTepDelForDnatInEachRtr(router, routerId, srcDpnId, extNwProvType, writeFlowInvTx);
+            hndlTepDelForDnatInEachRtr(router, routerId, srcDpnId, extNwProvType);
             LOG.debug("handleTepDelForAllRtrs :  TEP DEL : SNAT -> Withdrawing and Advertising routes for router {} ",
                 router.getRouter());
             hndlTepDelForSnatInEachRtr(router, routerId, srcDpnId, tunnelType, srcTepIp, destTepIp,
@@ -577,7 +577,7 @@ public class NatTunnelInterfaceStateListener
 
             LOG.debug("hndlTepAddOnNonNaptSwitch : SNAT -> Install the group which forward packet to the tunnel port "
                 + "for the NAPT switch {} and the flow 26 which forwards to group", primaryDpnId);
-            externalRouterListner.handleSwitches(srcDpnId, routerName, routerId, primaryDpnId, writeFlowInvTx);
+            externalRouterListner.handleSwitches(srcDpnId, routerName, routerId, primaryDpnId);
         } else {
             LOG.debug("hndlTepAddOnNonNaptSwitch : SNAT -> External BGP VPN (Private BGP) associated to router {}",
                     routerId);
@@ -674,7 +674,7 @@ public class NatTunnelInterfaceStateListener
         for (String externalIp : externalIps) {
             String rd = NatUtil.getVpnRd(dataBroker, externalVpnName);
             LOG.debug("hndlTepAddOnNaptSwitch : Removing Fib entry rd {} prefix {}", rd, externalIp);
-            fibManager.removeFibEntry(dataBroker, rd, externalIp, null);
+            fibManager.removeFibEntry(rd, externalIp, null);
         }
 
         /*
@@ -735,8 +735,7 @@ public class NatTunnelInterfaceStateListener
                 if (NatUtil.isOpenStackVniSemanticsEnforcedForGreAndVxlan(elanManager, extNwProvType)) {
                     l3vni = NatOverVxlanUtil.getInternetVpnVni(idManager, externalVpnName, l3vni).longValue();
                 }
-                Uuid externalSubnetId = NatUtil.getExternalSubnetForRouterExternalIp(dataBroker, externalIp,
-                        router);
+                Uuid externalSubnetId = NatUtil.getExternalSubnetForRouterExternalIp(externalIp, router);
                 NatUtil.addPrefixToBGP(dataBroker, bgpManager, fibManager, externalVpnName, rd, externalSubnetId,
                         fibExternalIp, nextHopIp, networkId.getValue(), null /* mac-address */, label, l3vni,
                         RouteOrigin.STATIC, srcDpnId);
@@ -1016,7 +1015,7 @@ public class NatTunnelInterfaceStateListener
     }
 
     private void hndlTepDelForDnatInEachRtr(RoutersList router, long routerId, BigInteger tepDeletedDpnId,
-            ProviderTypes extNwProvType, WriteTransaction writeFlowInvTx) {
+            ProviderTypes extNwProvType) {
         //DNAT : Withdraw the routes from the BGP
         String routerName = router.getRouter();
 
@@ -1078,7 +1077,7 @@ public class NatTunnelInterfaceStateListener
                 LOG.debug("hndlTepDelForDnatInEachRtr : DNAT -> Withdrawing the FIB route to the floating IP {} "
                     + "configured for the port: {}",
                     externalIp, interfaceName);
-                NatUtil.removePrefixFromBGP(dataBroker, bgpManager, fibManager, rd, externalIp, vpnName, LOG);
+                NatUtil.removePrefixFromBGP(bgpManager, fibManager, rd, externalIp, vpnName, LOG);
                 long serviceId = 0;
                 if (extNwProvType == ProviderTypes.VXLAN) {
                     serviceId = l3Vni;
index eb76cc4b3948f792d3ae8d250602707aaa6ff46d..118feb0e72ae93eb6a4da013e56d8dbb0b972ad4 100644 (file)
@@ -623,7 +623,8 @@ public final class NatUtil {
                                       String macAddress,
                                       long label,
                                       long l3vni,
-                                      RouteOrigin origin, BigInteger dpId) {
+                                      RouteOrigin origin,
+                                      BigInteger dpId) {
         try {
             LOG.info("addPrefixToBGP : Adding Fib entry rd {} prefix {} nextHop {} label {}", rd,
                     prefix, nextHopIp, label);
@@ -635,7 +636,7 @@ public final class NatUtil {
 
             addPrefixToInterface(broker, getVpnId(broker, vpnName), null /*interfaceName*/,prefix, dpId,
                     subnetId, Prefixes.PrefixCue.Nat);
-            fibManager.addOrUpdateFibEntry(broker, rd, macAddress, prefix,
+            fibManager.addOrUpdateFibEntry(rd, macAddress, prefix,
                     Collections.singletonList(nextHopIp), VrfEntry.EncapType.Mplsgre, (int)label, l3vni /*l3vni*/,
                     null /*gatewayMacAddress*/, parentVpnRd, origin, null /*writeTxn*/);
             if (rd != null && !rd.equalsIgnoreCase(vpnName)) {
@@ -754,11 +755,11 @@ public final class NatUtil {
 
     // TODO Clean up the exception handling
     @SuppressWarnings("checkstyle:IllegalCatch")
-    public static void removePrefixFromBGP(DataBroker broker, IBgpManager bgpManager, IFibManager fibManager,
+    public static void removePrefixFromBGP(IBgpManager bgpManager, IFibManager fibManager,
                                            String rd, String prefix, String vpnName, Logger log) {
         try {
             LOG.debug("removePrefixFromBGP: Removing Fib entry rd {} prefix {}", rd, prefix);
-            fibManager.removeFibEntry(broker, rd, prefix, null);
+            fibManager.removeFibEntry(rd, prefix, null);
             if (rd != null && !rd.equalsIgnoreCase(vpnName)) {
                 bgpManager.withdrawPrefix(rd, prefix);
             }
@@ -1662,7 +1663,7 @@ public final class NatUtil {
 
     protected static long getExternalSubnetVpnIdForRouterExternalIp(DataBroker dataBroker, String externalIpAddress,
             Routers router) {
-        Uuid externalSubnetId = NatUtil.getExternalSubnetForRouterExternalIp(dataBroker, externalIpAddress, router);
+        Uuid externalSubnetId = NatUtil.getExternalSubnetForRouterExternalIp(externalIpAddress, router);
         if (externalSubnetId != null) {
             return NatUtil.getExternalSubnetVpnId(dataBroker,externalSubnetId);
         }
@@ -1670,8 +1671,7 @@ public final class NatUtil {
         return NatConstants.INVALID_ID;
     }
 
-    protected static Uuid getExternalSubnetForRouterExternalIp(DataBroker dataBroker, String externalIpAddress,
-            Routers router) {
+    protected static Uuid getExternalSubnetForRouterExternalIp(String externalIpAddress, Routers router) {
         externalIpAddress = validateAndAddNetworkMask(externalIpAddress);
         List<ExternalIps> externalIps = router.getExternalIps();
         for (ExternalIps extIp : externalIps) {
@@ -1917,7 +1917,7 @@ public final class NatUtil {
                     router.getNetworkId(), subnetVpnName.getValue(), writeTx);
             vpnManager.removeArpResponderFlowsToExternalNetworkIps(router.getRouterName(), externalIpsSting,
                     router.getExtGwMacAddress(), primarySwitchId,
-                    router.getNetworkId(), writeTx);
+                    router.getNetworkId());
         }
         writeTx.submit();
     }
index 2d75f714f16f3dd27b86517f5bea526ba553f000..47f2e78127cfde8a51f0cbab33b3b916e26a6053 100644 (file)
@@ -313,7 +313,7 @@ public class VpnFloatingIpHandler implements FloatingIPHandler {
             vpnManager.removeSubnetMacFromVpnInstance(networkVpnName, subnetId.getValue(), floatingIpPortMacAddress,
                     dpnId, tx);
             vpnManager.removeArpResponderFlowsToExternalNetworkIps(routerUuid, Collections.singletonList(externalIp),
-                    floatingIpPortMacAddress, dpnId, networkId, tx);
+                    floatingIpPortMacAddress, dpnId, networkId);
         }), LOG, "onRemoveFloatingIp");
 
         removeFromFloatingIpPortInfo(floatingIpId);
@@ -324,7 +324,7 @@ public class VpnFloatingIpHandler implements FloatingIPHandler {
         if (provType == ProviderTypes.VXLAN) {
             Uuid floatingIpInterface = NatEvpnUtil.getFloatingIpInterfaceIdFromFloatingIpId(dataBroker, floatingIpId);
             evpnDnatFlowProgrammer.onRemoveFloatingIp(dpnId, vpnName, externalIp, floatingIpInterface.getValue(),
-                    floatingIpPortMacAddress, routerUuid, routerId, removeFlowInvTx);
+                    floatingIpPortMacAddress, routerId, removeFlowInvTx);
             return;
         }
         cleanupFibEntries(dpnId, vpnName, externalIp, label, removeFlowInvTx, provType);
@@ -336,7 +336,7 @@ public class VpnFloatingIpHandler implements FloatingIPHandler {
         //Remove Prefix from BGP
         String rd = NatUtil.getVpnRd(dataBroker, vpnName);
         String fibExternalIp = NatUtil.validateAndAddNetworkMask(externalIp);
-        NatUtil.removePrefixFromBGP(dataBroker, bgpManager, fibManager, rd, fibExternalIp, vpnName, LOG);
+        NatUtil.removePrefixFromBGP(bgpManager, fibManager, rd, fibExternalIp, vpnName, LOG);
 
         //Remove custom FIB routes
 
index 99e709e96e3ec3300344b0a191bcff06ddb86d22..7de7a8b1bef39752d8ee6e2c6c69dbb5404ae217 100644 (file)
@@ -243,7 +243,7 @@ public class NeutronPortChangeListener extends AsyncDataTreeChangeListenerBase<P
                                     interfaceIdentifier);
                     if (optionalInf.isPresent()) {
                         InterfaceBuilder interfaceBuilder = new InterfaceBuilder(optionalInf.get());
-                        InterfaceAcl infAcl = handlePortSecurityUpdated(dataBroker, original, update,
+                        InterfaceAcl infAcl = handlePortSecurityUpdated(original, update,
                                 origSecurityEnabled, updatedSecurityEnabled, interfaceBuilder).build();
                         interfaceBuilder.addAugmentation(InterfaceAcl.class, infAcl);
                         LOG.info("update: Of-port-interface updation for port {}", portName);
@@ -755,7 +755,7 @@ public class NeutronPortChangeListener extends AsyncDataTreeChangeListenerBase<P
             })));
     }
 
-    private static InterfaceAclBuilder handlePortSecurityUpdated(DataBroker dataBroker, Port portOriginal,
+    private static InterfaceAclBuilder handlePortSecurityUpdated(Port portOriginal,
             Port portUpdated, boolean origSecurityEnabled, boolean updatedSecurityEnabled,
             InterfaceBuilder interfaceBuilder) {
         InterfaceAclBuilder interfaceAclBuilder = null;
index c6a28bd5159258bc908fe0824d9979a2225106e3..bd6e8a1d10a45a309e5758f28f7a683f14b2fb98 100644 (file)
@@ -363,7 +363,7 @@ public class NeutronvpnNatManager implements AutoCloseable {
         Uuid routerId = update.getUuid();
 
         // Remove the router to the ExtRouters list
-        removeExternalRouter(origExtNetId, update);
+        removeExternalRouter(update);
 
         //Remove router entry from floating-ip-info list
         removeRouterFromFloatingIpInfo(update, dataBroker);
@@ -484,7 +484,7 @@ public class NeutronvpnNatManager implements AutoCloseable {
 
     // TODO Clean up the exception handling
     @SuppressWarnings("checkstyle:IllegalCatch")
-    private void removeExternalRouter(Uuid extNetId, Router update) {
+    private void removeExternalRouter(Router update) {
         Uuid routerId = update.getUuid();
 
         InstanceIdentifier<Routers> routersIdentifier = NeutronvpnUtils.buildExtRoutersIdentifier(routerId);
index dfcdbeaaa7945089950f881d44de726a063f1ca1..50e86c26f7c963b17d8e7eca81b5dcdf6f853b13 100644 (file)
@@ -123,7 +123,7 @@ public class L2GatewayListener extends AsyncClusteredDataTreeChangeListenerBase<
     protected void update(InstanceIdentifier<L2gateway> identifier, L2gateway original, L2gateway update) {
         LOG.trace("Updating L2gateway : key: {}, original value={}, update value={}", identifier, original, update);
         List<L2gatewayConnection> connections = l2gwService.getAssociatedL2GwConnections(
-                dataBroker, Sets.newHashSet(update.getUuid()));
+                Sets.newHashSet(update.getUuid()));
         if (connections == null) {
             LOG.warn("There are no connections associated with l2 gateway uuid {} name {}",
                     update.getUuid(), update.getName());
index 6314bbeabc2a6ed5649529a4e25e285507775b20..c27a00f27d903f258b57ed4f14f50be157181297 100644 (file)
@@ -124,7 +124,7 @@ public class UnderlayNetworkDpnListener
         addedTunnelInterfaces.removeAll(origTunnelInterfaces);
 
         populatePolicyGroupBucketsToDpn(underlayNetwork, profiles, removedTunnelInterfaces, dpId, NwConstants.DEL_FLOW);
-        populatePolicyGroupsToDpn(underlayNetwork, profiles, addedTunnelInterfaces, dpId, NwConstants.ADD_FLOW);
+        populatePolicyGroupsToDpn(profiles, addedTunnelInterfaces, dpId, NwConstants.ADD_FLOW);
         populatePolicyGroupBucketsToDpn(underlayNetwork, profiles, addedTunnelInterfaces, dpId, NwConstants.ADD_FLOW);
     }
 
@@ -140,13 +140,13 @@ public class UnderlayNetworkDpnListener
             return;
         }
 
-        populatePolicyGroupsToDpn(underlayNetwork, profiles, tunnelInterfaces, dpId, NwConstants.ADD_FLOW);
+        populatePolicyGroupsToDpn(profiles, tunnelInterfaces, dpId, NwConstants.ADD_FLOW);
         populatePolicyGroupBucketsToDpn(underlayNetwork, profiles, tunnelInterfaces, dpId, NwConstants.ADD_FLOW);
-        populatePolicyAclRulesToDpn(underlayNetwork, dpId, profiles, NwConstants.ADD_FLOW);
+        populatePolicyAclRulesToDpn(dpId, profiles, NwConstants.ADD_FLOW);
         populatePolicyRoutesToDpn(profiles, tunnelInterfaces, dpId, NwConstants.ADD_FLOW);
     }
 
-    private void populatePolicyGroupsToDpn(String underlayNetwork, List<PolicyProfile> profiles,
+    private void populatePolicyGroupsToDpn(List<PolicyProfile> profiles,
             List<TunnelInterface> tunnelInterfaces, BigInteger dpId, int addOrRemove) {
         profiles.forEach(profile -> {
             String policyClassifier = profile.getPolicyClassifier();
@@ -176,7 +176,7 @@ public class UnderlayNetworkDpnListener
         });
     }
 
-    private void populatePolicyAclRulesToDpn(String underlayNetwork, BigInteger dpId, List<PolicyProfile> profiles,
+    private void populatePolicyAclRulesToDpn(BigInteger dpId, List<PolicyProfile> profiles,
             int addOrRemove) {
         profiles.forEach(profile -> {
             String policyClassifier = profile.getPolicyClassifier();
index ddd69eef8423100656e4d6fdd5df1e07acd910bf..4acbe10ba4fcd1af385dfa043642b049028b2f82 100644 (file)
@@ -66,16 +66,16 @@ public abstract class DelegatingDataTreeListener<T extends DataObject> implement
                     mod.getModificationType(), key);
             switch (mod.getModificationType()) {
                 case DELETE:
-                    dataProcessor.remove(key, mod.getDataBefore());
+                    dataProcessor.remove(mod.getDataBefore());
                     break;
                 case SUBTREE_MODIFIED:
-                    dataProcessor.update(key, mod.getDataBefore(), mod.getDataAfter());
+                    dataProcessor.update(mod.getDataAfter());
                     break;
                 case WRITE:
                     if (mod.getDataBefore() == null) {
-                        dataProcessor.add(key, mod.getDataAfter());
+                        dataProcessor.add(mod.getDataAfter());
                     } else {
-                        dataProcessor.update(key, mod.getDataBefore(), mod.getDataAfter());
+                        dataProcessor.update(mod.getDataAfter());
                     }
                     break;
                 default:
index 814d141629d939830dd56d5ade30efb750508ee6..2f561720f91f46551c32afc1759d481b71f37add 100644 (file)
@@ -9,7 +9,6 @@
 package org.opendaylight.netvirt.sfc.translator;
 
 import org.opendaylight.yangtools.yang.binding.DataObject;
-import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 
 /**
  * Neutron SFC yang model processor.
@@ -19,28 +18,23 @@ public interface INeutronSfcDataProcessor<D extends DataObject> {
     /**
      * Method removes DataObject which is identified by InstanceIdentifier.
      *
-     * @param identifier - the whole path to DataObject
      * @param del - DataObject for removing
      */
-    void remove(InstanceIdentifier<D> identifier, D del);
+    void remove(D del);
 
     /**
      * Method updates the original DataObject to the update DataObject.
      * Both are identified by same InstanceIdentifier.
      *
-     * @param identifier - the whole path to DataObject
-     * @param original - original DataObject (for update)
-     * @param update - changed DataObject (contain updates)
-     */
-    void update(InstanceIdentifier<D> identifier, D original, D update);
+     * @param update - changed DataObject (contain updates)*/
+    void update(D update);
 
     /**
      * Method adds the DataObject which is identified by InstanceIdentifier
      * to device.
      *
-     * @param identifier - the whole path to new DataObject
      * @param add - new DataObject
      */
-    void add(InstanceIdentifier<D> identifier, D add);
+    void add(D add);
 
 }
index 2cdb1c634a4ef3f1d606a28ad10bd543353d1869..051e9c360771a5fcaf40137dd8d14b4a7fa18553 100644 (file)
@@ -38,11 +38,10 @@ public class NeutronFlowClassifierListener extends DelegatingDataTreeListener<Sf
     /**
      * Method removes Acl respective to SfcFlowClassifier which is identified by InstanceIdentifier.
      *
-     * @param path - the whole path to SfcFlowClassifier
      * @param deletedSfcFlowClassifier        - SfcFlowClassifier for removing
      */
     @Override
-    public void remove(InstanceIdentifier<SfcFlowClassifier> path, SfcFlowClassifier deletedSfcFlowClassifier) {
+    public void remove(SfcFlowClassifier deletedSfcFlowClassifier) {
         Acl aclFlowClassifier = FlowClassifierTranslator.buildAcl(deletedSfcFlowClassifier);
         sfcMdsalHelper.removeAclFlowClassifier(aclFlowClassifier);
     }
@@ -51,14 +50,10 @@ public class NeutronFlowClassifierListener extends DelegatingDataTreeListener<Sf
      * Method updates the original SfcFlowClassifier to the update SfcFlowClassifier.
      * Both are identified by same InstanceIdentifier.
      *
-     * @param path - the whole path to SfcFlowClassifier
-     * @param originalSfcFlowClassifier   - original SfcFlowClassifier (for update)
      * @param updatedSfcFlowClassifier     - changed SfcFlowClassifier (contain updates)
      */
     @Override
-    public void update(InstanceIdentifier<SfcFlowClassifier> path,
-                       SfcFlowClassifier originalSfcFlowClassifier,
-                       SfcFlowClassifier updatedSfcFlowClassifier) {
+    public void update(SfcFlowClassifier updatedSfcFlowClassifier) {
 
         Acl aclFlowClassifier = FlowClassifierTranslator.buildAcl(updatedSfcFlowClassifier);
         sfcMdsalHelper.updateAclFlowClassifier(aclFlowClassifier);
@@ -68,11 +63,10 @@ public class NeutronFlowClassifierListener extends DelegatingDataTreeListener<Sf
      * Method adds the SfcFlowClassifier which is identified by InstanceIdentifier
      * to device.
      *
-     * @param path - the whole path to new SfcFlowClassifier
      * @param sfcFlowClassifier        - new SfcFlowClassifier
      */
     @Override
-    public void add(InstanceIdentifier<SfcFlowClassifier> path, SfcFlowClassifier sfcFlowClassifier) {
+    public void add(SfcFlowClassifier sfcFlowClassifier) {
         // Respective ACL classifier will be written in data store, once the chain is created.
     }
 
index 5329f9ef2d27a5530afba2d1cd5f2db0629dce24..52c437fda58041a447bb75402d277c5f737e28b9 100644 (file)
@@ -65,11 +65,10 @@ public class NeutronPortChainListener extends DelegatingDataTreeListener<PortCha
     /**
      * Method removes PortChain which is identified by InstanceIdentifier.
      *
-     * @param path - the whole path to PortChain
      * @param deletedPortChain        - PortChain for removing
      */
     @Override
-    public void remove(InstanceIdentifier<PortChain> path, PortChain deletedPortChain) {
+    public void remove(PortChain deletedPortChain) {
         if (this.rspService != null) {
             DeleteRenderedPathInput deleteRenderedPathInput =
                     PortChainTranslator.buildDeleteRenderedServicePathInput(PortChainTranslator
@@ -87,12 +86,10 @@ public class NeutronPortChainListener extends DelegatingDataTreeListener<PortCha
      * Method updates the original PortChain to the update PortChain.
      * Both are identified by same InstanceIdentifier.
      *
-     * @param path - the whole path to PortChain
-     * @param originalPortChain   - original PortChain (for update)
      * @param updatePortChain     - changed PortChain (contain updates)
      */
     @Override
-    public void update(InstanceIdentifier<PortChain> path, PortChain originalPortChain, PortChain updatePortChain) {
+    public void update(PortChain updatePortChain) {
         //TODO: Add support for chain update
     }
 
@@ -100,11 +97,10 @@ public class NeutronPortChainListener extends DelegatingDataTreeListener<PortCha
      * Method adds the PortChain which is identified by InstanceIdentifier
      * to device.
      *
-     * @param path - the whole path to new PortChain
      * @param newPortChain        - new PortChain
      */
     @Override
-    public void add(final InstanceIdentifier<PortChain> path, final PortChain newPortChain) {
+    public void add(final PortChain newPortChain) {
         processPortChain(newPortChain);
     }
 
index af76ba5634c7b3ae66c75490878ed3909f927d8f..d967c7f7dc36215e37a33e196431e80f75b02d4c 100644 (file)
@@ -31,11 +31,10 @@ public class NeutronPortPairGroupListener extends DelegatingDataTreeListener<Por
     /**
      * Method removes PortPairGroup which is identified by InstanceIdentifier.
      *
-     * @param path - the whole path to PortPairGroup
      * @param deletedPortPairGroup        - PortPairGroup for removing
      */
     @Override
-    public void remove(InstanceIdentifier<PortPairGroup> path, PortPairGroup deletedPortPairGroup) {
+    public void remove(PortPairGroup deletedPortPairGroup) {
         //NO-OP
     }
 
@@ -43,14 +42,10 @@ public class NeutronPortPairGroupListener extends DelegatingDataTreeListener<Por
      * Method updates the original PortPairGroup to the update PortPairGroup.
      * Both are identified by same InstanceIdentifier.
      *
-     * @param path - the whole path to PortPairGroup
-     * @param originalPortPairGroup   - original PortPairGroup (for update)
      * @param updatePortPairGroup     - changed PortPairGroup (contain updates)
      */
     @Override
-    public void update(InstanceIdentifier<PortPairGroup> path,
-                       PortPairGroup originalPortPairGroup,
-                       PortPairGroup updatePortPairGroup) {
+    public void update(PortPairGroup updatePortPairGroup) {
         //NO-OP
     }
 
@@ -58,11 +53,10 @@ public class NeutronPortPairGroupListener extends DelegatingDataTreeListener<Por
      * Method adds the PortPairGroup which is identified by InstanceIdentifier
      * to device.
      *
-     * @param path - the whole path to new PortPairGroup
      * @param newPortPairGroup        - new PortPairGroup
      */
     @Override
-    public void add(InstanceIdentifier<PortPairGroup> path, PortPairGroup newPortPairGroup) {
+    public void add(PortPairGroup newPortPairGroup) {
         //NO-OP
     }
 }
index 2aa2b7267afbe7a2a73f5ab3b3cdc32d354c40bc..f9006be44a51c695a1d6e83bd6cd4bb0d52abb48 100644 (file)
@@ -43,11 +43,10 @@ public class NeutronPortPairListener extends DelegatingDataTreeListener<PortPair
     /**
      * Method removes PortPair which is identified by InstanceIdentifier.
      *
-     * @param path - the whole path to PortPair
      * @param deletedPortPair        - PortPair for removing
      */
     @Override
-    public void remove(InstanceIdentifier<PortPair> path, PortPair deletedPortPair) {
+    public void remove(PortPair deletedPortPair) {
         LOG.info("Received remove port pair event {}", deletedPortPair);
 
         ServiceFunctionKey sfKey = PortPairTranslator.getSFKey(deletedPortPair);
@@ -67,12 +66,10 @@ public class NeutronPortPairListener extends DelegatingDataTreeListener<PortPair
      * Method updates the original PortPair to the update PortPair.
      * Both are identified by same InstanceIdentifier.
      *
-     * @param path - the whole path to PortPair
-     * @param originalPortPair   - original PortPair (for update)
      * @param updatePortPair     - changed PortPair (contain updates)
      */
     @Override
-    public void update(InstanceIdentifier<PortPair> path, PortPair originalPortPair, PortPair updatePortPair) {
+    public void update(PortPair updatePortPair) {
         //NO-OP
     }
 
@@ -80,11 +77,10 @@ public class NeutronPortPairListener extends DelegatingDataTreeListener<PortPair
      * Method adds the PortPair which is identified by InstanceIdentifier
      * to device.
      *
-     * @param path - the whole path to new PortPair
      * @param newPortPair        - new PortPair
      */
     @Override
-    public void add(InstanceIdentifier<PortPair> path, PortPair newPortPair) {
+    public void add(PortPair newPortPair) {
         //NO-OP
         // Port Pair data written in neutron data store will be used
         // When user will create port chain.
index 9f327379aa33ffb223554f7f9704ccdac4c0396f..fd74fdc23e82c90491081db2d2207dd21b7fe9aa 100644 (file)
@@ -20,7 +20,6 @@ import javax.annotation.PostConstruct;
 import javax.annotation.PreDestroy;
 import javax.inject.Inject;
 import javax.inject.Singleton;
-import org.opendaylight.controller.sal.binding.api.RpcProviderRegistry;
 import org.opendaylight.infrautils.counters.api.OccurenceCounter;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.direct.statistics.rev160511.GetFlowStatisticsInputBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.direct.statistics.rev160511.GetFlowStatisticsOutput;
@@ -52,10 +51,10 @@ public class CounterRetriever {
     private final Counters counters = new Counters();
 
     @Inject
-    public CounterRetriever(final RpcProviderRegistry rpcProviderRegistry,
-            final StatisticsConfig statisticsConfig, final OpendaylightDirectStatisticsService odlDirectStatsService) {
+    public CounterRetriever(final StatisticsConfig statisticsConfig,
+            final OpendaylightDirectStatisticsService odlDirectStatsService) {
         this.odlDirectStatsService = odlDirectStatsService;
-        this.nodeResultTimeout = statisticsConfig.getNodeCounterResultTimeout();
+        nodeResultTimeout = statisticsConfig.getNodeCounterResultTimeout();
     }
 
     @PreDestroy
@@ -195,7 +194,7 @@ public class CounterRetriever {
         return crds;
     }
 
-    public CounterResultDataStructure getSwitchFlowCountersDirect(BigInteger dpId, Match match, short tableId) {
+    public CounterResultDataStructure getSwitchFlowCountersDirect(BigInteger dpId, Match match) {
         NodeRef nodeRef = new NodeRef(InstanceIdentifier.builder(Nodes.class)
                 .child(Node.class, new NodeKey(new NodeId(CountersUtils.getNodeId(dpId)))).toInstance());
         GetFlowStatisticsInputBuilder gfsib = new GetFlowStatisticsInputBuilder();
index 385add4a426d0384b8f5b734358deb8b5e88316a..49734ebae607fc8cd0b14d9400c2900a47f30e44 100644 (file)
@@ -162,11 +162,11 @@ public final class CountersServiceUtils {
     }
 
     public static Flow createFlowOnTable(Match match, int priority, short tableId) {
-        return createFlowOnTable(match, priority, tableId, null, false, null);
+        return createFlowOnTable(match, priority, tableId, null, null);
     }
 
     public static Flow createFlowOnTable(Match match, int priority, short tableId, BigInteger cookie,
-            boolean isCreateFlowIdFromMatch, Integer timeout) {
+            Integer timeout) {
         FlowBuilder fb = new FlowBuilder();
         if (match != null) {
             fb.setMatch(match);
index d90255869e3802c4082647b2a7627f3415c633bd..e26c3ef428884686a69dc15b6c6856dd9065f43b 100644 (file)
@@ -559,9 +559,7 @@ public class StatisticsImpl implements StatisticsService, ICountersInterfaceChan
         List<MatchInfoBase> matches = CountersServiceUtils.getCounterFlowMatch(ecr, lportTag,
                 ElementCountersDirection.valueOf(counterRequest.getTrafficDirection()));
         Match match = MDSALUtil.buildMatches(matches);
-        CounterResultDataStructure counterResultDS = counterRetriever.getSwitchFlowCountersDirect(dpId, match,
-                CountersServiceUtils.getTableId(ecr.getDirection()));
-        return counterResultDS;
+        return counterRetriever.getSwitchFlowCountersDirect(dpId, match);
     }
 
     private void initializeCountrsConfigDataSrore() {
index 5c4f0102e2dc36ef7cb1d78bab8d373103f126a3..1779fa49c468a47c5ab62f517688a4195bedb151 100644 (file)
@@ -28,7 +28,7 @@ public interface IVpnManager {
             String rd, String routerID, int label, RouteOrigin origin);
 
     void addExtraRoute(String vpnName, String destination, String nextHop, String rd, String routerID,
-            int label, Long l3vni, RouteOrigin origin, String intfName, Adjacency operationalAdj,
+            Long l3vni, RouteOrigin origin, String intfName, Adjacency operationalAdj,
             VrfEntry.EncapType encapType, WriteTransaction writeConfigTxn);
 
     void delExtraRoute(String vpnName, String destination, String nextHop, String rd, String routerID);
@@ -74,13 +74,13 @@ public interface IVpnManager {
             BigInteger dpnId, Uuid extNetworkId, WriteTransaction writeTx);
 
     void addArpResponderFlowsToExternalNetworkIps(String id, Collection<String> fixedIps, String routerGwMac,
-            BigInteger dpnId, long vpnId, String extInterfaceName, int lportTag, WriteTransaction writeTx);
+            BigInteger dpnId, String extInterfaceName, int lportTag, WriteTransaction writeTx);
 
     void removeArpResponderFlowsToExternalNetworkIps(String id, Collection<String> fixedIps, String macAddress,
-            BigInteger dpnId, Uuid extNetworkId, WriteTransaction writeTx);
+            BigInteger dpnId, Uuid extNetworkId);
 
     void removeArpResponderFlowsToExternalNetworkIps(String id, Collection<String> fixedIps,
-            BigInteger dpnId, String extInterfaceName, int lportTag, WriteTransaction writeTx);
+            BigInteger dpnId, String extInterfaceName, int lportTag);
 
     void onSubnetAddedToVpn(Subnetmap subnetmap, boolean isBgpVpn, Long elanTag);
 
index 1f3d490ac14d500c83182c58adb45cde93899acf..8b0f372fe44fb6030a165324af43049b7446a806 100755 (executable)
@@ -37,10 +37,9 @@ public interface IVpnLinkService {
      * @param label            Label of the route in the original VPN
      * @param forcedOrigin     By default, origin for leaked routes is INTERVPN, however it is possible to
      *                         provide a different origin if desired.
-     * @param addOrRemove states if the routes must be leaked or withdrawn
      */
     void leakRoute(InterVpnLinkDataComposite interVpnLink, String srcVpnUuid, String dstVpnUuid,
-                   String prefix, Long label, RouteOrigin forcedOrigin, int addOrRemove);
+                   String prefix, Long label, RouteOrigin forcedOrigin);
 
     /**
      * Similar to leakRouteIfNeeded but the only requisite to be met is that
index a401fcb36bed46c32ff878b2ba646e8559b4b6c0..ca2aae5fd86d31ffe866e7a1a8af7a3581c42855 100644 (file)
@@ -41,7 +41,7 @@ import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev160411.monitor.profile.create.input.Profile;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev160411.monitor.profile.create.input.ProfileBuilder;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev160411.monitor.start.input.ConfigBuilder;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.rpcs.rev160406.OdlInterfaceRpcService;
+
 import org.opendaylight.yangtools.yang.common.RpcResult;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -54,9 +54,9 @@ public final class AlivenessMonitorUtils {
     private AlivenessMonitorUtils() { }
 
     public static void startArpMonitoring(MacEntry macEntry, Long arpMonitorProfileId,
-        AlivenessMonitorService alivenessMonitorService, DataBroker dataBroker,
-        OdlInterfaceRpcService interfaceRpc, INeutronVpnManager neutronVpnService,
-        IInterfaceManager interfaceManager) {
+            AlivenessMonitorService alivenessMonitorService, DataBroker dataBroker,
+            INeutronVpnManager neutronVpnService,
+            IInterfaceManager interfaceManager) {
         if (interfaceManager.isExternalInterface(macEntry.getInterfaceName())) {
             LOG.debug("ARP monitoring is currently not supported through external interfaces,"
                     + "skipping ARP monitoring from interface {} for IP {} (last known MAC {})",
@@ -64,14 +64,14 @@ public final class AlivenessMonitorUtils {
             return;
         }
         Optional<IpAddress> gatewayIpOptional =
-            VpnUtil.getGatewayIpAddressFromInterface(macEntry.getInterfaceName(), neutronVpnService, dataBroker);
+            VpnUtil.getGatewayIpAddressFromInterface(macEntry.getInterfaceName(), neutronVpnService);
         if (!gatewayIpOptional.isPresent()) {
             LOG.error("Error while retrieving GatewayIp for interface{}", macEntry.getInterfaceName());
             return;
         }
         final IpAddress gatewayIp = gatewayIpOptional.get();
         Optional<String> gatewayMacOptional = VpnUtil.getGWMacAddressFromInterface(macEntry,
-            gatewayIp, dataBroker, interfaceRpc);
+            gatewayIp, dataBroker);
         if (!gatewayMacOptional.isPresent()) {
             LOG.error("Error while retrieving GatewayMac for interface{}", macEntry.getInterfaceName());
             return;
index 9bbaace99ed2e5e9b7c391a2a46783a3d2ad4c2a..fc309eecf2770724b24447235dcebf6463174bf7 100644 (file)
@@ -14,25 +14,22 @@ import org.opendaylight.controller.md.sal.binding.api.DataBroker;
 import org.opendaylight.genius.interfacemanager.interfaces.IInterfaceManager;
 import org.opendaylight.netvirt.neutronvpn.interfaces.INeutronVpnManager;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.alivenessmonitor.rev160411.AlivenessMonitorService;
-import org.opendaylight.yang.gen.v1.urn.opendaylight.genius.interfacemanager.rpcs.rev160406.OdlInterfaceRpcService;
 
 public class ArpMonitorStartTask implements Callable<List<ListenableFuture<Void>>> {
     private MacEntry macEntry;
     private Long arpMonitorProfileId;
     private DataBroker databroker;
     private AlivenessMonitorService alivenessManager;
-    private OdlInterfaceRpcService interfaceRpc;
     private INeutronVpnManager neutronVpnService;
     private IInterfaceManager interfaceManager;
 
     public ArpMonitorStartTask(MacEntry macEntry, Long profileId, DataBroker databroker,
-        AlivenessMonitorService alivenessManager, OdlInterfaceRpcService interfaceRpc,
-        INeutronVpnManager neutronVpnService, IInterfaceManager interfaceManager) {
+            AlivenessMonitorService alivenessManager,
+            INeutronVpnManager neutronVpnService, IInterfaceManager interfaceManager) {
         this.macEntry = macEntry;
         this.arpMonitorProfileId = profileId;
         this.databroker = databroker;
         this.alivenessManager = alivenessManager;
-        this.interfaceRpc = interfaceRpc;
         this.neutronVpnService = neutronVpnService;
         this.interfaceManager = interfaceManager;
     }
@@ -40,7 +37,7 @@ public class ArpMonitorStartTask implements Callable<List<ListenableFuture<Void>
     @Override
     public List<ListenableFuture<Void>> call() {
         AlivenessMonitorUtils.startArpMonitoring(macEntry, arpMonitorProfileId,
-            alivenessManager, databroker, interfaceRpc, neutronVpnService,
+            alivenessManager, databroker, neutronVpnService,
             interfaceManager);
         return null;
     }
index da9eecb49d8a932de7addf7e147c0c828241e13f..5a2b3c838a90067cf7b9d67dab6b9f1fb289d4c1 100644 (file)
@@ -145,7 +145,7 @@ public class ArpMonitoringHandler
                         value.getCreationTime());
                 jobCoordinator.enqueueJob(buildJobKey(srcInetAddr.toString(), vpnName),
                         new ArpMonitorStartTask(macEntry, arpMonitorProfileId, dataBroker, alivenessManager,
-                                interfaceRpc, neutronVpnService, interfaceManager));
+                                neutronVpnService, interfaceManager));
             } catch (UnknownHostException e) {
                 LOG.error("Error in deserializing packet {} with exception", value, e);
             }
index f5e53a0d4ce01c253081dd89565e41f9b3ec568a..2e0e413d960e01981d0da87321f2ea220cd414ca 100644 (file)
@@ -128,7 +128,7 @@ public class CentralizedSwitchChangeListener
                     subnetVpnName.getValue(), writeTx);
             vpnManager.removeArpResponderFlowsToExternalNetworkIps(routerName,
                     VpnUtil.getIpsListFromExternalIps(router.getExternalIps()),
-                    extGwMacAddress, primarySwitchId, extNetworkId, writeTx);
+                    extGwMacAddress, primarySwitchId, extNetworkId);
         }
     }
 }
index fa5da8580b3e394fef5c73a34ed60d7087a3284f..7eb7a207969f71e1b90878016b1b5179fa25c99f 100644 (file)
@@ -192,7 +192,7 @@ public class SubnetRoutePacketInHandler implements PacketProcessingListener {
             throws InterruptedException, ExecutionException, UnknownHostException {
         String vmVpnInterfaceName = VpnUtil.getVpnInterfaceName(odlInterfaceRpcService, metadata);
         if (isTunnel(vmVpnInterfaceName)) {
-            handlePacketFromTunnelToExternalNetwork(vpnIdVpnInstanceName, vmVpnInterfaceName,
+            handlePacketFromTunnelToExternalNetwork(vpnIdVpnInstanceName,
                     srcIpStr, dstIp, elanTag);
         }
         VpnInterface vmVpnInterface = VpnUtil.getVpnInterface(dataBroker, vmVpnInterfaceName);
@@ -272,7 +272,7 @@ public class SubnetRoutePacketInHandler implements PacketProcessingListener {
         transmitArpPacket(targetSubnetForPacketOut.getNhDpnId(), sourceIp, sourceMac, dstIp, elanTag);
     }
 
-    private void handlePacketFromTunnelToExternalNetwork(String vpnIdVpnInstanceName, String tunnelInterfaceName,
+    private void handlePacketFromTunnelToExternalNetwork(String vpnIdVpnInstanceName,
                         String srcIpStr, byte[] dstIp, long elanTag)
                                 throws UnknownHostException {
         String routerId = VpnUtil.getAssociatedExternalRouter(dataBroker, srcIpStr);
index 80c87a33b61528b7242eafc1cb29467a31f8dfbd..1fc965c6b8971ef513e989b2e9a6edabaf386eda 100644 (file)
@@ -62,7 +62,7 @@ public final class TransactionUtil {
     }
 
     public static <T extends DataObject> void syncWrite(DataBroker dataBroker, LogicalDatastoreType datastoreType,
-            InstanceIdentifier<T> path, T data, FutureCallback<Void> callback) {
+            InstanceIdentifier<T> path, T data) {
         WriteTransaction tx = dataBroker.newWriteOnlyTransaction();
         tx.put(datastoreType, path, data, WriteTransaction.CREATE_MISSING_PARENTS);
         try {
index db3d126fc647f4787b55d7e9317fe1e9eec47d18..f016512bd7d9e56162ed3964fded8010dcf6bd58 100644 (file)
@@ -135,7 +135,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
         if (isGreTunnel(del)) {
             programDcGwLoadBalancingGroup(del, NwConstants.DEL_FLOW);
         }
-        handleTunnelEventForDPN(del, UpdateRouteAction.WITHDRAW_ROUTE, TunnelAction.TUNNEL_EP_DELETE);
+        handleTunnelEventForDPN(del, TunnelAction.TUNNEL_EP_DELETE);
     }
 
     @Override
@@ -166,7 +166,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
         }
         WriteTransaction writeConfigTxn = dataBroker.newWriteOnlyTransaction();
         if (tunOpStatus == TunnelOperStatus.Up) {
-            handleTunnelEventForDPN(update, UpdateRouteAction.ADVERTISE_ROUTE, TunnelAction.TUNNEL_EP_ADD);
+            handleTunnelEventForDPN(update, TunnelAction.TUNNEL_EP_ADD);
         } else {
             vpnInstanceOpData.stream().filter(opData -> {
                 if (opData.getVpnToDpnList() == null) {
@@ -185,7 +185,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
                     List<RoutePaths> routePaths = vrfEntry.getRoutePaths();
                     routePaths.forEach(routePath -> {
                         if (routePath.getNexthopAddress().equals(srcTepIp)) {
-                            fibManager.updateRoutePathForFibEntry(dataBroker, opData.getVrfId(),
+                            fibManager.updateRoutePathForFibEntry(opData.getVrfId(),
                                     destPrefix.getDestPrefix(), srcTepIp, routePath.getLabel(),
                                     false, writeConfigTxn);
                         }
@@ -214,7 +214,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
         LOG.info("add: ITM Tunnel ,type {} ,added between src: {} and dest: {}",
                 fibManager.getTransportTypeStr(add.getTransportType().toString()),
                 add.getSrcInfo().getTepDeviceId(), add.getDstInfo().getTepDeviceId());
-        handleTunnelEventForDPN(add, UpdateRouteAction.ADVERTISE_ROUTE, TunnelAction.TUNNEL_EP_ADD);
+        handleTunnelEventForDPN(add, TunnelAction.TUNNEL_EP_ADD);
     }
 
     public enum TunnelEventProcessingMethod {
@@ -233,8 +233,8 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
 
     // TODO Clean up the exception handling
     @SuppressWarnings("checkstyle:IllegalCatch")
-    private void handleTunnelEventForDPN(StateTunnelList stateTunnelList, UpdateRouteAction action,
-                                         TunnelAction tunnelAction) {
+    private void handleTunnelEventForDPN(StateTunnelList stateTunnelList,
+            TunnelAction tunnelAction) {
         final BigInteger srcDpnId = new BigInteger(stateTunnelList.getSrcInfo().getTepDeviceId());
         final String srcTepIp = String.valueOf(stateTunnelList.getSrcInfo().getTepIp().getValue());
         String destTepIp = String.valueOf(stateTunnelList.getDstInfo().getTepIp().getValue());
@@ -340,7 +340,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
                 if (vpnInterface != null && !vpnInterface.isScheduledForRemove()) {
                     listVpnName.addAll(VpnHelper
                         .getVpnInterfaceVpnInstanceNamesString(vpnInterface.getVpnInstanceNames()));
-                    handleTunnelEventForDPNVpn(stateTunnelList, action, vpnIdRdMap,
+                    handleTunnelEventForDPNVpn(stateTunnelList, vpnIdRdMap,
                             tunnelAction, isTepDeletedOnDpn,
                             subnetList, TunnelEventProcessingMethod.POPULATESUBNETS,
                             vpnInterface);
@@ -361,7 +361,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
                 VpnInterface vpnInterface =
                         VpnUtil.getConfiguredVpnInterface(dataBroker, intfName);
                 if (vpnInterface != null) {
-                    handleTunnelEventForDPNVpn(stateTunnelList, action, vpnIdRdMap,
+                    handleTunnelEventForDPNVpn(stateTunnelList, vpnIdRdMap,
                             tunnelAction, isTepDeletedOnDpn,
                             subnetList, TunnelEventProcessingMethod.MANAGEREMOTEROUTES,
                             vpnInterface);
@@ -402,7 +402,7 @@ public class TunnelInterfaceStateListener extends AsyncDataTreeChangeListenerBas
 
     // TODO Clean up the exception handling
     @SuppressWarnings("checkstyle:IllegalCatch")
-    private void handleTunnelEventForDPNVpn(StateTunnelList stateTunnelList, UpdateRouteAction action,
+    private void handleTunnelEventForDPNVpn(StateTunnelList stateTunnelList,
                                             Map<Long, String> vpnIdRdMap, TunnelAction tunnelAction,
                                             boolean isTepDeletedOnDpn, List<Uuid> subnetList,
                                             TunnelEventProcessingMethod method,
index 8494196faaf3f1e5a4dff5a86aecb328c496ec45..816a821779f6d49ff1c7d9fe82986caeb48461fc 100644 (file)
@@ -154,20 +154,17 @@ public class VpnInstanceListener extends AsyncDataTreeChangeListenerBase<VpnInst
     protected void add(final InstanceIdentifier<VpnInstance> identifier, final VpnInstance value) {
         LOG.trace("{} add: Add VPN event key: {}, value: {}", LOGGING_PREFIX_ADD, identifier, value);
         final String vpnName = value.getVpnInstanceName();
-        jobCoordinator.enqueueJob("VPN-" + vpnName, new AddVpnInstanceWorker(idManager, dataBroker, value),
+        jobCoordinator.enqueueJob("VPN-" + vpnName, new AddVpnInstanceWorker(dataBroker, value),
                 SystemPropertyReader.getDataStoreJobCoordinatorMaxRetries());
     }
 
     private class AddVpnInstanceWorker implements Callable<List<ListenableFuture<Void>>> {
         private final Logger log = LoggerFactory.getLogger(AddVpnInstanceWorker.class);
-        IdManagerService idManager;
         VpnInstance vpnInstance;
         DataBroker broker;
 
-        AddVpnInstanceWorker(IdManagerService idManager,
-            DataBroker broker,
-            VpnInstance value) {
-            this.idManager = idManager;
+        AddVpnInstanceWorker(DataBroker broker,
+                VpnInstance value) {
             this.broker = broker;
             this.vpnInstance = value;
         }
@@ -221,7 +218,7 @@ public class VpnInstanceListener extends AsyncDataTreeChangeListenerBase<VpnInst
         } else {
             TransactionUtil.syncWrite(dataBroker, LogicalDatastoreType.CONFIGURATION,
                 VpnOperDsUtils.getVpnInstanceToVpnIdIdentifier(vpnInstanceName),
-                vpnInstanceToVpnId, TransactionUtil.DEFAULT_CALLBACK);
+                vpnInstanceToVpnId);
         }
 
         VpnIds vpnIdToVpnInstance = VpnUtil.getVpnIdToVpnInstance(vpnId, value.getVpnInstanceName(),
@@ -234,7 +231,7 @@ public class VpnInstanceListener extends AsyncDataTreeChangeListenerBase<VpnInst
         } else {
             TransactionUtil.syncWrite(dataBroker, LogicalDatastoreType.CONFIGURATION,
                 VpnUtil.getVpnIdToVpnInstanceIdentifier(vpnId),
-                vpnIdToVpnInstance, TransactionUtil.DEFAULT_CALLBACK);
+                vpnIdToVpnInstance);
         }
 
         try {
@@ -300,7 +297,7 @@ public class VpnInstanceListener extends AsyncDataTreeChangeListenerBase<VpnInst
         } else {
             TransactionUtil.syncWrite(dataBroker, LogicalDatastoreType.OPERATIONAL,
                 VpnUtil.getVpnInstanceOpDataIdentifier(primaryRd),
-                builder.build(), TransactionUtil.DEFAULT_CALLBACK);
+                builder.build());
         }
         LOG.info("{} addVpnInstance: VpnInstanceOpData populated successfully for vpn {} rd {}", LOGGING_PREFIX_ADD,
                 vpnInstanceName, primaryRd);
@@ -327,7 +324,7 @@ public class VpnInstanceListener extends AsyncDataTreeChangeListenerBase<VpnInst
              */
             VpnAfConfig config = vpnInstance.getIpv4Family();
             List<String> rd = config.getRouteDistinguisher();
-            if (rd == null || addBgpVrf(voids)) {
+            if (rd == null || addBgpVrf()) {
                 notifyTask();
                 vpnInterfaceManager.vpnInstanceIsReady(vpnName);
             }
@@ -368,7 +365,7 @@ public class VpnInstanceListener extends AsyncDataTreeChangeListenerBase<VpnInst
 
         // TODO Clean up the exception handling
         @SuppressWarnings("checkstyle:IllegalCatch")
-        private boolean addBgpVrf(List<Void> voids) {
+        private boolean addBgpVrf() {
             VpnAfConfig config = vpnInstance.getIpv4Family();
             String primaryRd = VpnUtil.getPrimaryRd(dataBroker, vpnName);
             List<VpnTarget> vpnTargetList = config.getVpnTargets().getVpnTarget();
index aaa3d58728e3de390a8f7ac24ccae031704d3935..7d7888e9340c8883f5f5ee525beb27f1e8954846 100755 (executable)
@@ -554,11 +554,11 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                     if (addOrRemove == NwConstants.ADD_FLOW) {
                         vpnManager.addArpResponderFlowsToExternalNetworkIps(routerName,
                                 VpnUtil.getIpsListFromExternalIps(router.getExternalIps()), router.getExtGwMacAddress(),
-                                dpId, vpnId, interfaceName, lportTag, writeInvTxn);
+                                dpId, interfaceName, lportTag, writeInvTxn);
                     } else {
                         vpnManager.removeArpResponderFlowsToExternalNetworkIps(routerName,
                                 VpnUtil.getIpsListFromExternalIps(router.getExternalIps()),
-                                dpId, interfaceName, lportTag, writeInvTxn);
+                                dpId, interfaceName, lportTag);
                     }
                 } else {
                     LOG.error("processExternalVpnInterface: No external-router found for router-id {}. Bailing out of"
@@ -768,8 +768,8 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                         // Use this for programming ARP_RESPONDER table here.  And save this
                         // info into vpnInterface operational, so it can used in VrfEntryProcessor
                         // to populate L3_GW_MAC_TABLE there.
-                        arpResponderHandler.addArpResponderFlow(dpnId, lportTag, vpnName, vpnId, interfaceName,
-                                subnetId, gatewayIp.get(), gwMac.get());
+                        arpResponderHandler.addArpResponderFlow(dpnId, lportTag, interfaceName,
+                                gatewayIp.get(), gwMac.get());
                         vpnInterfaceSubnetGwMacAddress = gwMac.get();
                     } else {
                         // A valid mac-address is not available for this subnet-gateway-ip
@@ -780,8 +780,8 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                         if (gwMac.isPresent()) {
                             VpnUtil.setupGwMacIfExternalVpn(dataBroker, mdsalManager, dpnId, interfaceName,
                                     vpnId, writeInvTxn, NwConstants.ADD_FLOW, gwMac.get());
-                            arpResponderHandler.addArpResponderFlow(dpnId, lportTag, vpnName, vpnId, interfaceName,
-                                    subnetId, gatewayIp.get(), gwMac.get());
+                            arpResponderHandler.addArpResponderFlow(dpnId, lportTag, interfaceName,
+                                    gatewayIp.get(), gwMac.get());
                         } else {
                             LOG.error("processVpnInterfaceAdjacencies: Gateway MAC for subnet ID {} could not be "
                                 + "obtained, cannot create ARP responder flow for interface name {}, vpnName {}, "
@@ -804,7 +804,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 synchronized (vpnPrefixKey.intern()) {
                     java.util.Optional<String> rdToAllocate = VpnUtil
                             .allocateRdForExtraRouteAndUpdateUsedRdsMap(dataBroker, vpnId, null,
-                            prefix, vpnName, nextHop.getNextHopIpList().get(0), dpnId, writeOperTxn);
+                            prefix, vpnName, nextHop.getNextHopIpList().get(0), dpnId);
                     if (rdToAllocate.isPresent()) {
                         rd = rdToAllocate.get();
                         LOG.info("processVpnInterfaceAdjacencies: The rd {} is allocated for the extraroute {}",
@@ -836,7 +836,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
             }
             if (nextHop.getAdjacencyType() != AdjacencyType.PrimaryAdjacency) {
                 vpnManager.addExtraRoute(vpnName, nextHop.getIpAddress(), nextHop.getNextHopIpList().get(0), rd,
-                        vpnName, operationalAdjacency.getLabel().intValue(), l3vni, origin,
+                        vpnName, l3vni, origin,
                         interfaceName, operationalAdjacency, encapType, writeConfigTxn);
             }
             value.add(operationalAdjacency);
@@ -847,7 +847,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 gwMac.isPresent() ? gwMac.get() : null, writeOperTxn);
 
         L3vpnInput input = new L3vpnInput().setNextHopIp(nextHopIp).setL3vni(l3vni).setPrimaryRd(primaryRd)
-                .setGatewayMac(gwMac.isPresent() ? gwMac.get() : null).setInterfaceName(interfaceName)
+                .setGatewayMac(gwMac.orNull()).setInterfaceName(interfaceName)
                 .setVpnName(vpnName).setDpnId(dpnId).setEncapType(encapType);
 
         for (Adjacency nextHop : aug.getAdjacency()) {
@@ -856,7 +856,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 RouteOrigin origin = nextHop.getAdjacencyType() == AdjacencyType.PrimaryAdjacency ? RouteOrigin.LOCAL
                         : RouteOrigin.STATIC;
                 input.setNextHop(nextHop).setRd(nextHop.getVrfId()).setRouteOrigin(origin);
-                registeredPopulator.populateFib(input, writeConfigTxn, writeOperTxn);
+                registeredPopulator.populateFib(input, writeConfigTxn);
             }
         }
     }
@@ -935,7 +935,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                         + " vpn {} vpnid {} rd {} interface {}", label, srcDpnId , prefix, nhList,
                         vpnInterface.getVpnInstanceName(), vpnId, rd, vpnInterface.getName());
                 // Update the VRF entry with nextHop
-                fibManager.updateRoutePathForFibEntry(dataBroker, primaryRd, prefix, srcTepIp,
+                fibManager.updateRoutePathForFibEntry(primaryRd, prefix, srcTepIp,
                         label, true, writeConfigTxn);
 
                 //Get the list of VPN's importing this route(prefix) .
@@ -945,7 +945,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 for (VpnInstanceOpDataEntry vpn : vpnsToImportRoute) {
                     String vpnRd = vpn.getVrfId();
                     if (vpnRd != null) {
-                        fibManager.updateRoutePathForFibEntry(dataBroker, vpnRd, prefix,
+                        fibManager.updateRoutePathForFibEntry(vpnRd, prefix,
                                 srcTepIp, label, true, writeConfigTxn);
                         LOG.info("updateVpnInterfaceOnTepAdd: Exported route with rd {} prefix {} nhList {} label {}"
                                 + " interface {} dpn {} from vpn {} to VPN {} vpnRd {}", rd, prefix, nhList, label,
@@ -1041,7 +1041,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                             prefix, nhList, vpnName,
                             vpnId, rd, vpnInterface.getName());
                     // Update the VRF entry with removed nextHop
-                    fibManager.updateRoutePathForFibEntry(dataBroker, primaryRd, prefix, srcTepIp,
+                    fibManager.updateRoutePathForFibEntry(primaryRd, prefix, srcTepIp,
                             label, false, writeConfigTxn);
 
                     //Get the list of VPN's importing this route(prefix) .
@@ -1051,7 +1051,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                     for (VpnInstanceOpDataEntry vpn : vpnsToImportRoute) {
                         String vpnRd = vpn.getVrfId();
                         if (vpnRd != null) {
-                            fibManager.updateRoutePathForFibEntry(dataBroker, vpnRd, prefix,
+                            fibManager.updateRoutePathForFibEntry(vpnRd, prefix,
                                     srcTepIp, label, false, writeConfigTxn);
                             LOG.info("updateVpnInterfaceOnTepDelete: Exported route with rd {} prefix {} nhList {}"
                                     + " label {} interface {} dpn {} from vpn {} to VPN {} vpnRd {}", rd, prefix,
@@ -1151,7 +1151,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                                 LOG.info("handleVpnsExportingRoutesImporting: Importing fib entry rd {} prefix {}"
                                         + " nexthop {} label {} to vpn {} vpnRd {}", vpn.getVrfId(), prefix, nh, label,
                                         vpnName, vpnRd);
-                                fibManager.addOrUpdateFibEntry(dataBroker, vpnRd, null /*macAddress*/, prefix,
+                                fibManager.addOrUpdateFibEntry(vpnRd, null /*macAddress*/, prefix,
                                         Collections.singletonList(nh), VrfEntry.EncapType.Mplsgre, label,
                                         0 /*l3vni*/, gwMac,  null /*parentVpnRd*/, RouteOrigin.SELF_IMPORTED,
                                         writeConfigTxn);
@@ -1335,7 +1335,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 if (nextHop.isPhysNetworkFunc()) {
                     LOG.info("removeAdjacenciesFromVpn: Removing PNF FIB entry rd {} prefix {}",
                             nextHop.getSubnetId().getValue(), nextHop.getIpAddress());
-                    fibManager.removeFibEntry(dataBroker, nextHop.getSubnetId().getValue(), nextHop.getIpAddress(),
+                    fibManager.removeFibEntry(nextHop.getSubnetId().getValue(), nextHop.getIpAddress(),
                             null/*writeCfgTxn*/);
                 } else {
                     String rd = nextHop.getVrfId();
@@ -1346,7 +1346,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                         // This is a primary adjacency
                         nhList = nextHop.getNextHopIpList() != null ? nextHop.getNextHopIpList()
                                 : Collections.emptyList();
-                        removeGwMacAndArpResponderFlows(nextHop, vpnId, vpnName, dpnId, lportTag, gwMac,
+                        removeGwMacAndArpResponderFlows(nextHop, vpnId, dpnId, lportTag, gwMac,
                                 interfaceName, writeInvTxn);
                     }
                     if (!nhList.isEmpty()) {
@@ -1364,7 +1364,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                                         + " interface {}", nextHop.getIpAddress(), rd,
                                 nextHop.getAdjacencyType().toString(), interfaceName);
                         bgpManager.withdrawPrefix(rd, nextHop.getIpAddress());
-                        fibManager.removeFibEntry(dataBroker, primaryRd, nextHop.getIpAddress(), writeConfigTxn);
+                        fibManager.removeFibEntry(primaryRd, nextHop.getIpAddress(), writeConfigTxn);
                     }
                 }
                 String ip = nextHop.getIpAddress().split("/")[0];
@@ -1388,7 +1388,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                                                                    String interfaceName, BigInteger dpnId,
                                                                    WriteTransaction writeConfigTxn) {
         return (nh) -> {
-            fibManager.removeOrUpdateFibEntry(dataBroker, vpnName, nextHop.getIpAddress(), nh,
+            fibManager.removeOrUpdateFibEntry(vpnName, nextHop.getIpAddress(), nh,
                     writeConfigTxn);
             LOG.info("removeAdjacenciesFromVpn: removed/updated FIB with rd {} prefix {}"
                             + " nexthop {} for interface {} on dpn {} for internal vpn {}",
@@ -1408,7 +1408,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
             for (VpnInstanceOpDataEntry vpn : vpnsToImportRoute) {
                 String vpnRd = vpn.getVrfId();
                 if (vpnRd != null) {
-                    fibManager.removeOrUpdateFibEntry(dataBroker, vpnRd,
+                    fibManager.removeOrUpdateFibEntry(vpnRd,
                             nextHop.getIpAddress(), nh, writeConfigTxn);
                     LOG.info("removeAdjacenciesFromVpn: Removed Exported route with rd {}"
                                     + " prefix {} nextHop {} from VPN {} parentVpn {}"
@@ -1419,7 +1419,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
         });
     }
 
-    private void removeGwMacAndArpResponderFlows(Adjacency nextHop, long vpnId, String vpnName, BigInteger dpnId,
+    private void removeGwMacAndArpResponderFlows(Adjacency nextHop, long vpnId, BigInteger dpnId,
                                                  int lportTag, String gwMac, String interfaceName,
                                                  WriteTransaction writeInvTxn) {
         final Uuid subnetId = nextHop.getSubnetId();
@@ -1430,8 +1430,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
             VpnUtil.setupGwMacIfExternalVpn(dataBroker, mdsalManager, dpnId, interfaceName,
                     vpnId, writeInvTxn, NwConstants.DEL_FLOW, gwMac);
         }
-        arpResponderHandler.removeArpResponderFlow(dpnId, lportTag, interfaceName, vpnName, vpnId,
-                subnetId);
+        arpResponderHandler.removeArpResponderFlow(dpnId, lportTag, interfaceName, subnetId);
     }
 
     private List<String> getNextHopForNonPrimaryAdjacency(Adjacency nextHop, String vpnName, BigInteger dpnId,
@@ -1816,13 +1815,13 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 String vpnPrefixKey = VpnUtil.getVpnNamePrefixKey(vpnName, prefix);
                 synchronized (vpnPrefixKey.intern()) {
                     java.util.Optional<String> rdToAllocate = VpnUtil.allocateRdForExtraRouteAndUpdateUsedRdsMap(
-                            dataBroker, vpnId, null, prefix, vpnName, nh, dpnId, writeOperTxn);
+                            dataBroker, vpnId, null, prefix, vpnName, nh, dpnId);
                     if (rdToAllocate.isPresent()) {
                         input.setRd(rdToAllocate.get());
                         operationalAdjacency = populator.createOperationalAdjacency(input);
                         int label = operationalAdjacency.getLabel().intValue();
                         vpnManager.addExtraRoute(vpnName, adj.getIpAddress(), nh, rdToAllocate.get(),
-                                currVpnIntf.getVpnInstanceName(), label, l3vni, origin,
+                                currVpnIntf.getVpnInstanceName(), l3vni, origin,
                                 currVpnIntf.getName(), operationalAdjacency, encapType, writeConfigTxn);
                         LOG.info("addNewAdjToVpnInterface: Added extra route ip {} nh {} rd {} vpnname {} label {}"
                                 + " Interface {} on dpn {}", adj.getIpAddress(), nh, rdToAllocate.get(),
@@ -1842,14 +1841,12 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                             if (vpn.getVrfId() != null) {
                                 VpnUtil.allocateRdForExtraRouteAndUpdateUsedRdsMap(
                                         dataBroker, vpn.getVpnId(), vpnId, prefix,
-                                        VpnUtil.getVpnName(dataBroker, vpn.getVpnId()), nh, dpnId,
-                                        writeOperTxn)
+                                        VpnUtil.getVpnName(dataBroker, vpn.getVpnId()), nh, dpnId)
                                         .ifPresent(
                                             rds -> vpnManager.addExtraRoute(
                                                     VpnUtil.getVpnName(dataBroker, vpn.getVpnId()),
                                                     adj.getIpAddress(), nh, rds,
                                                     currVpnIntf.getVpnInstanceName(),
-                                                    opAdjacency.getLabel().intValue(),
                                                     l3vni, RouteOrigin.SELF_IMPORTED,
                                                     currVpnIntf.getName(), opAdjacency, encapType, writeConfigTxn));
                             }
@@ -1869,7 +1866,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                         VpnUtil.getPrefixToInterface(BigInteger.ZERO, currVpnIntf.getName(), prefix,
                                 adj.getSubnetId(), Prefixes.PrefixCue.PhysNetFunc), true);
 
-                fibManager.addOrUpdateFibEntry(dataBroker, adj.getSubnetId().getValue(), adj.getMacAddress(),
+                fibManager.addOrUpdateFibEntry(adj.getSubnetId().getValue(), adj.getMacAddress(),
                         adj.getIpAddress(), Collections.emptyList(), null /* EncapType */, 0 /* label */, 0 /*l3vni*/,
                       null /* gw-mac */, parentVpnRd, RouteOrigin.LOCAL, writeConfigTxn);
 
@@ -1932,7 +1929,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                             } else if (adj.isPhysNetworkFunc()) {
                                 LOG.info("delAdjFromVpnInterface: deleting PNF adjacency prefix {} subnet [}",
                                         adj.getIpAddress(), adj.getSubnetId());
-                                fibManager.removeFibEntry(dataBroker, adj.getSubnetId().getValue(), adj.getIpAddress(),
+                                fibManager.removeFibEntry(adj.getSubnetId().getValue(), adj.getIpAddress(),
                                         writeConfigTxn);
                             }
                             break;
@@ -2093,7 +2090,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
 
                 RouterInterface routerInt = new RouterInterfaceBuilder().setUuid(vpnName)
                         .setIpAddress(primaryInterfaceIp).setMacAddress(macAddress).build();
-                fibManager.addFibEntryForRouterInterface(dataBroker, primaryRd, prefix,
+                fibManager.addFibEntryForRouterInterface(primaryRd, prefix,
                         routerInt, label, writeConfigTxn);
                 LOG.info("createFibEntryForRouterInterface: Router interface {} for vpn {} rd {} prefix {} label {}"
                         + " macAddress {} processed successfully;", interfaceName, vpnName, primaryRd, prefix, label,
@@ -2116,7 +2113,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                 if (adj.getAdjacencyType() == AdjacencyType.PrimaryAdjacency) {
                     String primaryInterfaceIp = adj.getIpAddress();
                     String prefix = VpnUtil.getIpPrefix(primaryInterfaceIp);
-                    fibManager.removeFibEntry(dataBroker, rd, prefix, writeConfigTxn);
+                    fibManager.removeFibEntry(rd, prefix, writeConfigTxn);
                     LOG.info("deleteFibEntryForRouterInterface: FIB for router interface {} deleted for vpn {} rd {}"
                             + " prefix {}", vpnInterface.getName(), vpnName, rd, prefix);
                     return;
@@ -2305,8 +2302,7 @@ public class VpnInterfaceManager extends AsyncDataTreeChangeListenerBase<VpnInte
                             () -> {
                                 WriteTransaction writeConfigTxn = dataBroker.newWriteOnlyTransaction();
                                 WriteTransaction writeOperTxn = dataBroker.newWriteOnlyTransaction();
-                                if (VpnUtil.isAdjacencyEligibleToVpn(
-                                     dataBroker, adjacency, vpnName, vpnInterface.getInterfaceName())) {
+                                if (VpnUtil.isAdjacencyEligibleToVpn(dataBroker, adjacency, vpnName)) {
                                     addNewAdjToVpnInterface(existingVpnInterfaceId, primaryRd, adjacency,
                                             vpnInterfaceOptional.get().getDpnId(), writeConfigTxn, writeOperTxn);
                                     ListenableFuture<Void> operFuture = writeOperTxn.submit();
index 53c17956dab515460b27b02b7dc3de151a9ef218..6ddc4c5b48d64a2d6546be42c86695447a01a87f 100644 (file)
@@ -30,7 +30,6 @@ import org.opendaylight.genius.mdsalutil.FlowEntity;
 import org.opendaylight.genius.mdsalutil.MDSALUtil;
 import org.opendaylight.genius.mdsalutil.MatchInfoBase;
 import org.opendaylight.genius.mdsalutil.MetaDataUtil;
-import org.opendaylight.genius.mdsalutil.NwConstants;
 import org.opendaylight.genius.mdsalutil.UpgradeState;
 import org.opendaylight.genius.mdsalutil.instructions.InstructionWriteMetadata;
 import org.opendaylight.genius.mdsalutil.interfaces.IMdsalApiManager;
@@ -97,7 +96,6 @@ public class VpnManagerImpl implements IVpnManager {
     public VpnManagerImpl(final DataBroker dataBroker,
                           final IdManagerService idManagerService,
                           final IMdsalApiManager mdsalManager,
-                          final VpnFootprintService vpnFootprintService,
                           final IElanService elanService,
                           final IInterfaceManager interfaceManager,
                           final VpnSubnetRouteHandler vpnSubnetRouteHandler,
@@ -176,13 +174,13 @@ public class VpnManagerImpl implements IVpnManager {
         VpnInstanceOpDataEntry vpnOpEntry = VpnUtil.getVpnInstanceOpData(dataBroker, rd);
         Boolean isVxlan = VpnUtil.isL3VpnOverVxLan(vpnOpEntry.getL3vni());
         VrfEntry.EncapType encapType = VpnUtil.getEncapType(isVxlan);
-        addExtraRoute(vpnName, destination, nextHop, rd, routerID, label, vpnOpEntry.getL3vni(),
+        addExtraRoute(vpnName, destination, nextHop, rd, routerID, vpnOpEntry.getL3vni(),
                 origin,/*intfName*/ null, null /*Adjacency*/, encapType, null);
     }
 
     @Override
     public void addExtraRoute(String vpnName, String destination, String nextHop, String rd, String routerID,
-            int label, Long l3vni, RouteOrigin origin, String intfName, Adjacency operationalAdj,
+            Long l3vni, RouteOrigin origin, String intfName, Adjacency operationalAdj,
             VrfEntry.EncapType encapType, WriteTransaction writeConfigTxn) {
 
         Boolean writeConfigTxnPresent = true;
@@ -234,8 +232,7 @@ public class VpnManagerImpl implements IVpnManager {
                         + " route for destination {}", destination);
                 return;
             }
-            ivpnLinkService.leakRoute(interVpnLink, srcVpnUuid, dstVpnUuid, destination, newLabel, RouteOrigin.STATIC,
-                    NwConstants.ADD_FLOW);
+            ivpnLinkService.leakRoute(interVpnLink, srcVpnUuid, dstVpnUuid, destination, newLabel, RouteOrigin.STATIC);
         } else {
             Optional<Routes> optVpnExtraRoutes = VpnExtraRouteHelper
                     .getVpnExtraroutes(dataBroker, vpnName, rd != null ? rd : routerID, destination);
@@ -248,7 +245,7 @@ public class VpnManagerImpl implements IVpnManager {
                     L3vpnInput input = new L3vpnInput().setNextHop(operationalAdj).setNextHopIp(nextHop).setL3vni(l3vni)
                             .setPrimaryRd(primaryRd).setVpnName(vpnName).setDpnId(dpnId)
                             .setEncapType(encapType).setRd(rd).setRouteOrigin(origin);
-                    L3vpnRegistry.getRegisteredPopulator(encapType).populateFib(input, writeConfigTxn, null);
+                    L3vpnRegistry.getRegisteredPopulator(encapType).populateFib(input, writeConfigTxn);
                 }
             }
         }
@@ -290,7 +287,7 @@ public class VpnManagerImpl implements IVpnManager {
             LOG.info("delExtraRoute: Removed extra route {} from interface {} for rd {}", destination, intfName, rd);
         } else {
             // add FIB route directly
-            fibManager.removeOrUpdateFibEntry(dataBroker, routerID, destination, tunnelIp, writeConfigTxn);
+            fibManager.removeOrUpdateFibEntry(routerID, destination, tunnelIp, writeConfigTxn);
             LOG.info("delExtraRoute: Removed extra route {} from interface {} for rd {}", destination, intfName,
                     routerID);
         }
@@ -338,7 +335,7 @@ public class VpnManagerImpl implements IVpnManager {
                         return;
                     }
                 }
-                fibManager.removeOrUpdateFibEntry(dataBroker, primaryRd, prefix, tunnelIp, writeConfigTxn);
+                fibManager.removeOrUpdateFibEntry(primaryRd, prefix, tunnelIp, writeConfigTxn);
                 if (VpnUtil.isEligibleForBgp(rd, vpnName, dpnId, null /*networkName*/)) {
                     // TODO: Might be needed to include nextHop here
                     bgpManager.withdrawPrefix(rd, prefix);
@@ -546,7 +543,7 @@ public class VpnManagerImpl implements IVpnManager {
 
     @Override
     public void addArpResponderFlowsToExternalNetworkIps(String id, Collection<String> fixedIps, String macAddress,
-                     BigInteger dpnId, long vpnId, String extInterfaceName, int lportTag, WriteTransaction writeTx) {
+            BigInteger dpnId, String extInterfaceName, int lportTag, WriteTransaction writeTx) {
         if (fixedIps == null || fixedIps.isEmpty()) {
             LOG.debug("No external IPs defined for {}", id);
             return;
@@ -559,14 +556,14 @@ public class VpnManagerImpl implements IVpnManager {
                 tx -> {
                     for (String fixedIp : fixedIps) {
                         installArpResponderFlowsToExternalNetworkIp(macAddress, dpnId, extInterfaceName, lportTag,
-                                vpnId,fixedIp, tx);
+                                fixedIp);
                     }
                 });
             ListenableFutures.addErrorLogging(future, LOG, "Commit transaction");
         } else {
             for (String fixedIp : fixedIps) {
-                installArpResponderFlowsToExternalNetworkIp(macAddress, dpnId, extInterfaceName, lportTag, vpnId,
-                        fixedIp, writeTx);
+                installArpResponderFlowsToExternalNetworkIp(macAddress, dpnId, extInterfaceName, lportTag,
+                        fixedIp);
             }
         }
     }
@@ -593,14 +590,13 @@ public class VpnManagerImpl implements IVpnManager {
             return;
         }
 
-        long vpnId = getVpnIdFromExtNetworkId(extNetworkId);
-        addArpResponderFlowsToExternalNetworkIps(id, fixedIps, macAddress, dpnId, vpnId, extInterfaceName, lportTag,
+        addArpResponderFlowsToExternalNetworkIps(id, fixedIps, macAddress, dpnId, extInterfaceName, lportTag,
                 writeTx);
     }
 
     @Override
     public void removeArpResponderFlowsToExternalNetworkIps(String id, Collection<String> fixedIps, String macAddress,
-            BigInteger dpnId, Uuid extNetworkId, WriteTransaction writeTx) {
+            BigInteger dpnId, Uuid extNetworkId) {
 
         if (dpnId == null || BigInteger.ZERO.equals(dpnId)) {
             LOG.warn("Failed to remove arp responder flows for router {}. DPN id is missing.", id);
@@ -634,12 +630,12 @@ public class VpnManagerImpl implements IVpnManager {
         }
 
         removeArpResponderFlowsToExternalNetworkIps(id, fixedIps, dpnId,
-                extInterfaceName, lportTag, writeTx);
+                extInterfaceName, lportTag);
     }
 
     @Override
     public void removeArpResponderFlowsToExternalNetworkIps(String id, Collection<String> fixedIps,
-            BigInteger dpnId, String extInterfaceName, int lportTag, WriteTransaction writeTx) {
+            BigInteger dpnId, String extInterfaceName, int lportTag) {
         if (fixedIps == null || fixedIps.isEmpty()) {
             LOG.debug("No external IPs defined for {}", id);
             return;
@@ -670,7 +666,7 @@ public class VpnManagerImpl implements IVpnManager {
     }
 
     private void installArpResponderFlowsToExternalNetworkIp(String macAddress, BigInteger dpnId,
-            String extInterfaceName, int lportTag, long vpnId, String fixedIp, WriteTransaction writeTx) {
+            String extInterfaceName, int lportTag, String fixedIp) {
         // reset the split-horizon bit to allow traffic to be sent back to the
         // provider port
         List<Instruction> instructions = new ArrayList<>();
@@ -691,16 +687,6 @@ public class VpnManagerImpl implements IVpnManager {
         elanService.removeArpResponderFlow(arpInput);
     }
 
-    private long getVpnIdFromExtNetworkId(Uuid extNetworkId) {
-        Uuid vpnInstanceId = VpnUtil.getExternalNetworkVpnId(dataBroker, extNetworkId);
-        if (vpnInstanceId == null) {
-            LOG.debug("Network {} is not associated with VPN", extNetworkId.getValue());
-            return VpnConstants.INVALID_ID;
-        }
-
-        return VpnUtil.getVpnId(dataBroker, vpnInstanceId.getValue());
-    }
-
     @Override
     public void onSubnetAddedToVpn(Subnetmap subnetmap, boolean isBgpVpn, Long elanTag) {
         vpnSubnetRouteHandler.onSubnetAddedToVpn(subnetmap, isBgpVpn, elanTag);
index 1163a54cf13fdfcc1d103036e8297936bab64289..155bbf6f0c80731cddbb8a405e8c985665612818 100644 (file)
@@ -101,7 +101,7 @@ public class VpnOpStatusListener extends AsyncDataTreeChangeListenerBase<VpnInst
                 VpnUtil.removeVpnInstanceToVpnId(dataBroker, vpnName, writeTxn);
                 LOG.trace("Removed vpnIdentifier for  rd{} vpnname {}", primaryRd, vpnName);
                 // Clean up FIB Entries Config DS
-                fibManager.removeVrfTable(dataBroker, primaryRd, null);
+                fibManager.removeVrfTable(primaryRd, null);
                 // Clean up VPNExtraRoutes Operational DS
                 if (VpnUtil.isBgpVpn(vpnName, primaryRd)) {
                     if (update.getType() == VpnInstanceOpDataEntry.Type.L2) {
index e6a87a23a48f265dfc703ede788507b8db2bed7f..4dc432c31c5eba4d635699f1d82c57fdcffb0947 100644 (file)
@@ -178,7 +178,7 @@ public class VpnRpcServiceImpl implements VpnRpcService {
             }
         } else {
             vpnManager.addExtraRoute(vpnInstanceName, destination, nexthop, vpnRd, null /* routerId */,
-                    label.intValue(), vpnOpEntry.getL3vni(), RouteOrigin.STATIC, null /* intfName */,
+                    vpnOpEntry.getL3vni(), RouteOrigin.STATIC, null /* intfName */,
                             null /*Adjacency*/, encapType, null);
         }
 
@@ -232,7 +232,7 @@ public class VpnRpcServiceImpl implements VpnRpcService {
 
         Optional<InterVpnLinkDataComposite> optVpnLink = interVpnLinkCache.getInterVpnLinkByEndpoint(nexthop);
         if (optVpnLink.isPresent()) {
-            fibManager.removeOrUpdateFibEntry(dataBroker,  vpnRd, destination, nexthop, /*writeTx*/ null);
+            fibManager.removeOrUpdateFibEntry(vpnRd, destination, nexthop, /*writeTx*/ null);
             bgpManager.withdrawPrefix(vpnRd, destination);
         } else {
             vpnManager.delExtraRoute(vpnInstanceName, destination,
index e4c710d7b7a5cde6954fe18c58d4c39b5ee4d41b..14c14b8a401f510c9c388223096e6a07261cb6e1 100644 (file)
@@ -832,7 +832,7 @@ public class VpnSubnetRouteHandler {
                 .setSubnetIp(subnetIp).setNextHopIp(nextHopIp).setL3vni(l3vni).setLabel(label).setElanTag(elanTag)
                 .setDpnId(nhDpnId).setEncapType(encapType).setNetworkName(networkName).setPrimaryRd(rd);
         if (!isBgpVpn) {
-            vpnPopulator.populateFib(input, null /*writeCfgTxn*/, null /*writeOperTxn*/);
+            vpnPopulator.populateFib(input, null /*writeCfgTxn*/);
             return true;
         }
         Preconditions.checkNotNull(nextHopIp, LOGGING_PREFIX + "NextHopIp cannot be null or empty!");
@@ -840,7 +840,7 @@ public class VpnSubnetRouteHandler {
                 .getPrefixToInterfaceIdentifier(VpnUtil.getVpnId(dataBroker, vpnName), subnetIp), VpnUtil
                 .getPrefixToInterface(nhDpnId, subnetId.getValue(), subnetIp, subnetId,
                         Prefixes.PrefixCue.SubnetRoute));
-        vpnPopulator.populateFib(input, null /*writeCfgTxn*/, null /*writeOperTxn*/);
+        vpnPopulator.populateFib(input, null /*writeCfgTxn*/);
         try {
             // BGP manager will handle withdraw and advertise internally if prefix
             // already exist
@@ -883,11 +883,11 @@ public class VpnSubnetRouteHandler {
     }
 
     public void deleteSubnetRouteFibEntryFromDS(String rd, String prefix, String vpnName) {
-        fibManager.removeFibEntry(dataBroker, rd, prefix, null);
+        fibManager.removeFibEntry(rd, prefix, null);
         List<VpnInstanceOpDataEntry> vpnsToImportRoute = VpnUtil.getVpnsImportingMyRoute(dataBroker, vpnName);
         for (VpnInstanceOpDataEntry vpnInstance : vpnsToImportRoute) {
             String importingRd = vpnInstance.getVrfId();
-            fibManager.removeFibEntry(dataBroker, importingRd, prefix, null);
+            fibManager.removeFibEntry(importingRd, prefix, null);
             LOG.info("SUBNETROUTE: deleteSubnetRouteFibEntryFromDS: Deleted imported subnet route rd {} prefix {}"
                     + " from vpn {} importingRd {}", rd, prefix, vpnInstance.getVpnInstanceName(), importingRd);
         }
index 3795a9bc5bc435bd54ea42830f4d3db986fdad87..e210f4a6b97b75212077187c85ca219be322a84d 100755 (executable)
@@ -1328,7 +1328,7 @@ public final class VpnUtil {
     }
 
     static Optional<IpAddress> getGatewayIpAddressFromInterface(String srcInterface,
-            INeutronVpnManager neutronVpnService, DataBroker dataBroker) {
+            INeutronVpnManager neutronVpnService) {
         Optional<IpAddress> gatewayIp = Optional.absent();
         if (neutronVpnService != null) {
             //TODO(Gobinath): Need to fix this as assuming port will belong to only one Subnet would be incorrect"
@@ -1346,7 +1346,7 @@ public final class VpnUtil {
     }
 
     static Optional<String> getGWMacAddressFromInterface(MacEntry macEntry, IpAddress gatewayIp,
-        DataBroker dataBroker, OdlInterfaceRpcService interfaceRpc) {
+            DataBroker dataBroker) {
         Optional<String> gatewayMac = Optional.absent();
         long vpnId = getVpnId(dataBroker, macEntry.getVpnName());
         InstanceIdentifier<org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.vpn.id.to.vpn
@@ -1396,7 +1396,6 @@ public final class VpnUtil {
     }
 
     public static Optional<VpnPortipToPort> getRouterInterfaceForVpnInterface(DataBroker dataBroker,
-                                                                              String interfaceName,
                                                                               String vpnName,
                                                                               Uuid subnetUuid) {
         Optional<VpnPortipToPort> gwPortOptional = Optional.absent();
@@ -1504,7 +1503,7 @@ public final class VpnUtil {
 
     static java.util.Optional<String> allocateRdForExtraRouteAndUpdateUsedRdsMap(
             DataBroker dataBroker, long vpnId, @Nullable Long parentVpnId, String prefix, String vpnName,
-            String nextHop, BigInteger dpnId, WriteTransaction writeOperTxn) {
+            String nextHop, BigInteger dpnId) {
         //Check if rd is already allocated for this extraroute behind the same VM. If yes, reuse it.
         //This is particularly useful during reboot scenarios.
         java.util.Optional<String> allocatedRd = VpnExtraRouteHelper
@@ -1855,7 +1854,7 @@ public final class VpnUtil {
     }
 
     public static boolean isAdjacencyEligibleToVpn(DataBroker dataBroker, Adjacency adjacency,
-                      String vpnName, String interfaceName) {
+            String vpnName) {
         // returns true if BGPVPN Internet and adjacency is IPv6, false otherwise
         boolean adjacencyEligible = true;
         // if BGPVPN internet, return false if subnetmap has not internetVpnId() filled in
index c3baa495b95798162c882149cddddabe98306718..a426016d194181b2c767e9c26afe97e103fe689f 100644 (file)
@@ -72,22 +72,16 @@ public class ArpResponderHandler {
      *            dpn Id on which ARP responder flow to be added
      * @param lportTag
      *            lport tag of the interface
-     * @param vpnName
-     *            vpnname of the interface
-     * @param vpnId
-     *            vpn id that interface belongs to
      * @param interfaceName
      *            interface to which ARP responder flow to be added
-     * @param subnetId
-     *            subnet Id of the interface
      * @param gatewayIp
      *            gateway ip of the interface
      * @param mac
      *            mac address
      */
 
-    public void addArpResponderFlow(BigInteger dpnId, int lportTag, String vpnName, long vpnId, String interfaceName,
-            Uuid subnetId, String gatewayIp, String mac) {
+    public void addArpResponderFlow(BigInteger dpnId, int lportTag, String interfaceName,
+            String gatewayIp, String mac) {
 
         LOG.trace("Creating the ARP Responder flow for VPN Interface {}", interfaceName);
         ArpReponderInputBuilder builder = new ArpReponderInputBuilder();
@@ -107,15 +101,10 @@ public class ArpResponderHandler {
      *            lport tag of the interface
      * @param ifName
      *            interface to which ARP responder flow to be removed
-     * @param vpnName
-     *            vpnname of the interface
-     * @param vpnId
-     *            vpn id that interface belongs to
-     *
      * @param subnetUuid
      *            subnet Id of the interface
      */
-    public void removeArpResponderFlow(BigInteger dpId, int lportTag, String ifName, String vpnName, long vpnId,
+    public void removeArpResponderFlow(BigInteger dpId, int lportTag, String ifName,
             Uuid subnetUuid) {
         Optional<String> gwIp = VpnUtil.getVpnSubnetGatewayIp(dataBroker, subnetUuid);
         if (gwIp.isPresent()) {
index 2bebfdcf4b74c338a643e6e83e2a830783457905..1f388bb6636efc6f9c242fb6ce1a57c7390b02de 100755 (executable)
@@ -120,7 +120,7 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
             String key = dstVpnRd + VpnConstants.SEPARATOR + prefix;
             long leakedLabel = VpnUtil.getUniqueId(idManager, VpnConstants.VPN_IDPOOL_NAME, key);
             String leakedNexthop = interVpnLink.getEndpointIpAddr(vpnName);
-            fibManager.addOrUpdateFibEntry(dataBroker, dstVpnRd, null /*macAddress*/, prefix,
+            fibManager.addOrUpdateFibEntry(dstVpnRd, null /*macAddress*/, prefix,
                                            Collections.singletonList(leakedNexthop), VrfEntry.EncapType.Mplsgre,
                                            (int) leakedLabel, 0 /*l3vni*/, null /*gatewayMacAddress*/,
                                            null /*parentVpnRd*/, RouteOrigin.INTERVPN, null /*writeConfigTxn*/);
@@ -138,7 +138,7 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
             }
         } else {
             LOG.debug("Removing leaked route to {} from VPN {}", prefix, dstVpnName);
-            fibManager.removeFibEntry(dataBroker, dstVpnRd, prefix, null /*writeConfigTxn*/);
+            fibManager.removeFibEntry(dstVpnRd, prefix, null /*writeConfigTxn*/);
             bgpManager.withdrawPrefix(dstVpnRd, prefix);
         }
     }
@@ -147,7 +147,7 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
     @SuppressWarnings("checkstyle:IllegalCatch")
     @Override
     public void leakRoute(InterVpnLinkDataComposite interVpnLink, String srcVpnUuid, String dstVpnUuid,
-                          String prefix, Long label, RouteOrigin forcedOrigin, int addOrRemove) {
+                          String prefix, Long label, RouteOrigin forcedOrigin) {
         String ivpnLinkName = interVpnLink.getInterVpnLinkName();
         // The source VPN must participate in the InterVpnLink
         Preconditions.checkArgument(interVpnLink.isVpnLinked(srcVpnUuid),
@@ -265,11 +265,11 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
 
         if (! originsToConsider.isEmpty()) {
             // 1st Endpoint ==> 2nd endpoint
-            leakRoutes(ivpnLink, vpn1Uuid, vpn2Uuid, originsToConsider, NwConstants.ADD_FLOW);
+            leakRoutes(ivpnLink, vpn1Uuid, vpn2Uuid, originsToConsider);
 
 
             // 2nd Endpoint ==> 1st endpoint
-            leakRoutes(ivpnLink, vpn2Uuid, vpn1Uuid, originsToConsider, NwConstants.ADD_FLOW);
+            leakRoutes(ivpnLink, vpn2Uuid, vpn1Uuid, originsToConsider);
         }
 
         // Static routes in Vpn1 pointing to Vpn2's endpoint
@@ -306,7 +306,7 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
                                       vpnLink.getInterVpnLinkName(), vpn1Rd, vrfEntry.getDestPrefix());
                         } else {
                             leakRoute(vpnLink, vpn2Uuid, vpn1Uuid, vrfEntry.getDestPrefix(), label,
-                                      RouteOrigin.value(vrfEntry.getOrigin()), NwConstants.ADD_FLOW);
+                                      RouteOrigin.value(vrfEntry.getOrigin()));
                         }
                     });
 
@@ -314,7 +314,7 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
     }
 
     private void leakRoutes(InterVpnLinkDataComposite vpnLink, String srcVpnUuid, String dstVpnUuid,
-                            List<RouteOrigin> originsToConsider, int addOrRemove) {
+                            List<RouteOrigin> originsToConsider) {
         String srcVpnRd = VpnUtil.getVpnRd(dataBroker, srcVpnUuid);
         String dstVpnRd = VpnUtil.getVpnRd(dataBroker, dstVpnUuid);
         List<VrfEntry> srcVpnRemoteVrfEntries = VpnUtil.getVrfEntriesByOrigin(dataBroker, srcVpnRd, originsToConsider);
@@ -327,8 +327,7 @@ public class IVpnLinkServiceImpl implements IVpnLinkService, AutoCloseable {
                         vpnLink.getInterVpnLinkName(), dstVpnRd, vrfEntry.getDestPrefix());
                 continue;
             }
-            leakRoute(vpnLink, srcVpnUuid, dstVpnUuid, vrfEntry.getDestPrefix(), label, null /*NotForcedOrigin*/,
-                      addOrRemove);
+            leakRoute(vpnLink, srcVpnUuid, dstVpnUuid, vrfEntry.getDestPrefix(), label, null /*NotForcedOrigin*/);
         }
     }
 
index aa33a331f9a15bfef88ff032d6637110cf4f88bb..07bed23d5e4e2eef644153cabf6affb75904a6ee 100755 (executable)
@@ -70,8 +70,7 @@ public class InterVpnLinkNodeAddTask implements Callable<List<ListenableFuture<V
                         && shouldConfigureLinkIntoDpn(i.getInterVpnLinkState(), numberOfDpns))
             .forEach(i -> {
                 installLPortDispatcherTable(i.getInterVpnLinkState(), firstDpnList, secondDpnList);
-                result.add(updateInterVpnLinkState(i.getInterVpnLinkState(), firstDpnList, secondDpnList,
-                                                   numberOfDpns));
+                result.add(updateInterVpnLinkState(i.getInterVpnLinkState(), firstDpnList, secondDpnList));
             });
 
         return result;
@@ -95,7 +94,7 @@ public class InterVpnLinkNodeAddTask implements Callable<List<ListenableFuture<V
 
     private ListenableFuture<Void>
         updateInterVpnLinkState(InterVpnLinkState interVpnLinkState, List<BigInteger> firstDpnList,
-                                List<BigInteger> secondDpnList, int numberOfDpns) {
+                                List<BigInteger> secondDpnList) {
 
         FirstEndpointState firstEndPointState =
             new FirstEndpointStateBuilder(interVpnLinkState.getFirstEndpointState()).setDpId(firstDpnList).build();
index 39fc0ed3a9cc951ac448a7365449f0e6bab27f5a..80b11e6755cc84510b30fe3617bcb6823d0e55aa 100755 (executable)
@@ -374,29 +374,10 @@ public final class InterVpnLinkUtil {
      * @param prefix Prefix of the route
      * @param label Label of the route in the original VPN
      */
-    public static void leakRoute(DataBroker broker, IBgpManager bgpManager, InterVpnLink interVpnLink,
-        String srcVpnUuid, String dstVpnUuid, String prefix, Long label) {
-        leakRoute(broker, bgpManager, interVpnLink, srcVpnUuid, dstVpnUuid, prefix, label, RouteOrigin.INTERVPN);
-    }
-
-    /**
-     * Leaks a route from one VPN to another.
-     *
-     * @param broker dataBroker service reference
-     * @param bgpManager Used to advertise routes to the BGP Router
-     * @param interVpnLink Reference to the object that holds the info about the link between the 2 VPNs
-     * @param srcVpnUuid UUID of the VPN that has the route that is going to be leaked to the other VPN
-     * @param dstVpnUuid UUID of the VPN that is going to receive the route
-     * @param prefix Prefix of the route
-     * @param label Label of the route in the original VPN
-     * @param forcedOrigin By default, origin for leaked routes should be INTERVPN, however it is possible to provide
-     *     a different origin if desired.
-     */
     // TODO Clean up the exception handling
     @SuppressWarnings("checkstyle:IllegalCatch")
     public static void leakRoute(DataBroker broker, IBgpManager bgpManager, InterVpnLink interVpnLink,
-        String srcVpnUuid, String dstVpnUuid, String prefix, Long label,
-        RouteOrigin forcedOrigin) {
+        String srcVpnUuid, String dstVpnUuid, String prefix, Long label) {
         Preconditions.checkNotNull(interVpnLink);
 
         // The source VPN must participate in the InterVpnLink
@@ -468,7 +449,7 @@ public final class InterVpnLinkUtil {
         }
         LOG.debug("Writing FibEntry to DS:  vpnRd={}, prefix={}, label={}, nexthop={} (interVpnLink)",
             vpnRd, destination, label, nexthop);
-        fibManager.addOrUpdateFibEntry(dataBroker, vpnRd, null /*macAddress*/, destination,
+        fibManager.addOrUpdateFibEntry(vpnRd, null /*macAddress*/, destination,
                 Collections.singletonList(nexthop), VrfEntry.EncapType.Mplsgre, label,
                 0 /*l3vni*/, null /*gatewayMacAddress*/, null /*parentVpnRd*/, RouteOrigin.STATIC, null /*writeTxn*/);
 
index fa5d24a86f4302f6831fecd3d8c4da0c553d72c8..e3572d374edd98ce4607b8a6339c6a828c14708a 100644 (file)
@@ -57,7 +57,7 @@ public class L3vpnOverMplsGrePopulator extends L3vpnPopulator {
     }
 
     @Override
-    public void populateFib(L3vpnInput input, WriteTransaction writeConfigTxn, WriteTransaction writeOperTxn) {
+    public void populateFib(L3vpnInput input, WriteTransaction writeConfigTxn) {
         if (input.getRouteOrigin() == RouteOrigin.CONNECTED) {
             LOG.info("populateFib : Found SubnetRoute for subnet {} rd {}", input.getSubnetIp(), input.getPrimaryRd());
             addSubnetRouteFibEntry(input);
@@ -89,7 +89,7 @@ public class L3vpnOverMplsGrePopulator extends L3vpnPopulator {
             for (VpnInstanceOpDataEntry vpn : vpnsToImportRoute) {
                 String vpnRd = vpn.getVrfId();
                 if (vpnRd != null) {
-                    fibManager.addOrUpdateFibEntry(broker, vpnRd, null /*macAddress*/,
+                    fibManager.addOrUpdateFibEntry(vpnRd, null /*macAddress*/,
                             nextHopIpAddress, Arrays.asList(nextHopIp), encapType, (int) label,
                             0 /*l3vni*/, input.getGatewayMac(), null /*parentVpnRd*/, RouteOrigin.SELF_IMPORTED,
                             writeConfigTxn);
@@ -100,7 +100,7 @@ public class L3vpnOverMplsGrePopulator extends L3vpnPopulator {
             }
         } else {
             // ### add FIB route directly
-            fibManager.addOrUpdateFibEntry(broker, vpnName, null /*macAddress*/,
+            fibManager.addOrUpdateFibEntry(vpnName, null /*macAddress*/,
                     nextHopIpAddress, Arrays.asList(nextHopIp), encapType, (int) label,
                     0 /*l3vni*/, input.getGatewayMac(), null /*parentVpnRd*/, input.getRouteOrigin(), writeConfigTxn);
             LOG.info("populateFib: Added internal FIB entry for prefix {} nexthop {} label {}"
index 490508c1ae927dc201f5e8596bca04eaa2a3e83c..ea627fcb68db4873d33c503400ea9b1dcec16b2e 100644 (file)
@@ -50,8 +50,7 @@ public class L3vpnOverVxlanPopulator extends L3vpnPopulator {
     }
 
     @Override
-    public void populateFib(L3vpnInput input, WriteTransaction writeConfigTxn,
-                            WriteTransaction writeOperTxn) {
+    public void populateFib(L3vpnInput input, WriteTransaction writeConfigTxn) {
         if (input.getRouteOrigin() == RouteOrigin.CONNECTED) {
             LOG.info("populateFib : Found SubnetRoute for subnet {} rd {}", input.getSubnetIp(), input.getPrimaryRd());
             addSubnetRouteFibEntry(input);
index c7557275aec94f17c3eda9e0d05e0c623db34500..88b5e613e997dde251f1bb433ae72d9c3d66b9b6 100644 (file)
@@ -63,8 +63,7 @@ public abstract class L3vpnPopulator implements VpnPopulator {
     }
 
     @Override
-    public void populateFib(L3vpnInput input, WriteTransaction writeCfgTxn,
-                            WriteTransaction writeOperTxn) {}
+    public void populateFib(L3vpnInput input, WriteTransaction writeCfgTxn) {}
 
     public void addSubnetRouteFibEntry(L3vpnInput input) {
         String rd = input.getRd();
@@ -199,7 +198,7 @@ public abstract class L3vpnPopulator implements VpnPopulator {
             List<String> nextHopList = Collections.singletonList(nextHopIp);
             LOG.info("ADD: addPrefixToBGP: Adding Fib entry rd {} prefix {} nextHop {} label {} gwMac {}", rd, prefix,
                     nextHopList, label, gatewayMac);
-            fibManager.addOrUpdateFibEntry(broker, primaryRd, macAddress, prefix, nextHopList,
+            fibManager.addOrUpdateFibEntry(primaryRd, macAddress, prefix, nextHopList,
                     encapType, (int)label, l3vni, gatewayMac, null /*parentVpnRd*/, origin, writeConfigTxn);
             LOG.info("ADD: addPrefixToBGP: Added Fib entry rd {} prefix {} nextHop {} label {} gwMac {}", rd, prefix,
                     nextHopList, label, gatewayMac);
index 5a29dbe8d104441be9c35266c8750e43bdfe3d83..8b0c80e0c3a6d6ac98e9c0f162675e79dbec6e7f 100644 (file)
@@ -12,7 +12,7 @@ import org.opendaylight.netvirt.vpnmanager.populator.input.L3vpnInput;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.netvirt.l3vpn.rev130911.adjacency.list.Adjacency;
 
 public interface VpnPopulator {
-    void populateFib(L3vpnInput input, WriteTransaction writeCfgTxn, WriteTransaction writeOperTxn);
+    void populateFib(L3vpnInput input, WriteTransaction writeCfgTxn);
 
     Adjacency createOperationalAdjacency(L3vpnInput input);
 }