Optimize DHCP relay processing for VPP
[groupbasedpolicy.git] / renderers / vpp / src / main / java / org / opendaylight / groupbasedpolicy / renderer / vpp / policy / VppRendererPolicyManager.java
index 3d21b6224d31ca2cde306c6c4360faae970c801a..b380358c567d634563fb05025c6aaa2ae85eda5e 100644 (file)
@@ -8,8 +8,10 @@
 
 package org.opendaylight.groupbasedpolicy.renderer.vpp.policy;
 
+import java.util.ArrayList;
 import java.util.Collection;
 import java.util.HashSet;
+import java.util.List;
 import java.util.Map;
 import java.util.Map.Entry;
 import java.util.Set;
@@ -21,21 +23,26 @@ import org.opendaylight.controller.config.yang.config.vpp_provider.impl.VppRende
 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.groupbasedpolicy.renderer.vpp.commands.DhcpRelayCommand;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.config.ConfigUtil;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.event.NodeOperEvent;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.event.RendererPolicyConfEvent;
+import org.opendaylight.groupbasedpolicy.renderer.vpp.policy.acl.AclManager;
 import org.opendaylight.groupbasedpolicy.renderer.vpp.util.KeyFactory;
 import org.opendaylight.groupbasedpolicy.util.IidFactory;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.base_endpoint.rev160427.has.absolute.location.absolute.location.location.type.ExternalLocationCase;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.rev151103.renderers.renderer.RendererPolicy;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.rev151103.renderers.renderer.RendererPolicyBuilder;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.rev151103.renderers.renderer.renderer.policy.configuration.RendererForwarding;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.rev151103.renderers.renderer.renderer.policy.configuration.endpoints.AddressEndpointWithLocation;
 import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.rev151103.renderers.renderer.renderer.policy.configuration.renderer.endpoints.RendererEndpointKey;
+import org.opendaylight.yang.gen.v1.urn.opendaylight.groupbasedpolicy.renderer.rev151103.renderers.renderer.renderer.policy.configuration.rule.groups.RuleGroupKey;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.NodeId;
 import org.opendaylight.yang.gen.v1.urn.tbd.params.xml.ns.yang.network.topology.rev131021.network.topology.topology.Node;
 import org.opendaylight.yangtools.yang.binding.InstanceIdentifier;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
-import com.google.common.base.Optional;
 import com.google.common.base.Preconditions;
 import com.google.common.collect.HashMultimap;
 import com.google.common.collect.ImmutableSet;
@@ -46,18 +53,19 @@ import com.google.common.collect.SetMultimap;
 import com.google.common.collect.Sets;
 import com.google.common.collect.Sets.SetView;
 import com.google.common.eventbus.Subscribe;
-import com.google.common.util.concurrent.FutureCallback;
-import com.google.common.util.concurrent.Futures;
 
 public class VppRendererPolicyManager {
 
     private static final Logger LOG = LoggerFactory.getLogger(VppRendererPolicyManager.class);
     private final DataBroker dataProvider;
     private ForwardingManager fwManager;
+    private final AclManager aclManager;
 
-    public VppRendererPolicyManager(@Nonnull ForwardingManager fwManager, @Nonnull DataBroker dataProvider) {
+    public VppRendererPolicyManager(@Nonnull ForwardingManager fwManager, @Nonnull AclManager aclManager,
+                                    @Nonnull DataBroker dataProvider) {
         this.fwManager = Preconditions.checkNotNull(fwManager);
         this.dataProvider = Preconditions.checkNotNull(dataProvider);
+        this.aclManager = Preconditions.checkNotNull(aclManager);
     }
 
     @Subscribe
@@ -65,16 +73,16 @@ public class VppRendererPolicyManager {
         RendererPolicyBuilder responseBuilder = new RendererPolicyBuilder();
         switch (event.getDtoModificationType()) {
             case CREATED:
-                LOG.trace("CREATED : {}", event);
+                LOG.debug("CREATED : {}", event.getIid());
                 responseBuilder.setVersion(event.getAfter().get().getVersion());
                 rendererPolicyCreated(event.getAfter().get());
                 break;
             case UPDATED:
-                LOG.trace("UPDATED: {}", event);
+                LOG.debug("UPDATED: {}", event.getIid());
                 RendererPolicy rPolicyBefore = event.getBefore().get();
                 RendererPolicy rPolicyAfter = event.getAfter().get();
                 responseBuilder.setVersion(rPolicyAfter.getVersion());
-                if (!isConfigurationChanged(rPolicyBefore, rPolicyAfter)) {
+                if (rPolicyBefore.getConfiguration() == null && rPolicyAfter.getConfiguration() == null) {
                     LOG.debug("Configuration is not changed only updating config version from {} to {}",
                             rPolicyBefore.getVersion(), rPolicyAfter.getVersion());
                 } else {
@@ -83,7 +91,7 @@ public class VppRendererPolicyManager {
                 }
                 break;
             case DELETED:
-                LOG.trace("DELETED: {}", event);
+                LOG.debug("DELETED: {}", event.getIid());
                 responseBuilder.setVersion(event.getBefore().get().getVersion());
                 rendererPolicyDeleted(event.getBefore().get());
                 break;
@@ -92,37 +100,24 @@ public class VppRendererPolicyManager {
         RendererPolicy response = responseBuilder.build();
         wTx.put(LogicalDatastoreType.OPERATIONAL, IidFactory.rendererIid(VppRenderer.NAME).child(RendererPolicy.class),
                 response, true);
-        Futures.addCallback(wTx.submit(), new FutureCallback<Void>() {
-
-            @Override
-            public void onSuccess(Void result) {
-                LOG.debug("Renderer updated renderer policy {}", response);
-            }
-
-            @Override
-            public void onFailure(Throwable t) {
-                LOG.warn("Renderer DIDN'T update renderer-policy {}", response);
-            }
-        });
-    }
-
-    private boolean isConfigurationChanged(RendererPolicy before, RendererPolicy after) {
-        if (before.getConfiguration() == null && after.getConfiguration() == null) {
-            return false;
+        try {
+            wTx.submit().get();
+        } catch (Exception ex) {
+            LOG.trace("Got Exception in renderer policy update. Exception: {}", ex);
         }
-        return true;
     }
 
     private void rendererPolicyUpdated(RendererPolicy rPolicyBefore, RendererPolicy rPolicyAfter) {
+        LOG.trace("VPP renderer policy updated");
         PolicyContext policyCtxBefore = new PolicyContext(rPolicyBefore);
         PolicyContext policyCtxAfter = new PolicyContext(rPolicyAfter);
-
+        aclManager.cacheEndpointsByInterfaces(policyCtxAfter);
         MapDifference<String, Collection<NodeId>> vppNodesByL2FlDiff =
                 createDiffForVppNodesByL2Fd(policyCtxBefore, policyCtxAfter);
         SetMultimap<String, NodeId> removedVppNodesByL2Fd = HashMultimap.create();
         SetMultimap<String, NodeId> createdVppNodesByL2Fd = HashMultimap.create();
         for (Entry<String, ValueDifference<Collection<NodeId>>> entry : vppNodesByL2FlDiff.entriesDiffering()
-            .entrySet()) {
+                .entrySet()) {
             String bridgeDomain = entry.getKey();
             Collection<NodeId> beforeNodes = entry.getValue().leftValue();
             Collection<NodeId> afterNodes = entry.getValue().rightValue();
@@ -160,64 +155,150 @@ public class VppRendererPolicyManager {
         SetView<RendererEndpointKey> removedRendEps = Sets.difference(rendEpsBefore, rendEpsAfter);
         removedRendEps.forEach(rEpKey -> fwManager.removeForwardingForEndpoint(rEpKey, policyCtxBefore));
 
-        fwManager.removeVxlanBridgeDomainsOnNodes(removedVppNodesByL2Fd);
-        fwManager.createVxlanBridgeDomainsOnNodes(createdVppNodesByL2Fd);
+        if (!ConfigUtil.getInstance().isL3FlatEnabled()) {
+            LOG.debug("Removing bridge domains on nodes {}", removedVppNodesByL2Fd);
+            fwManager.removeBridgeDomainOnNodes(removedVppNodesByL2Fd);
+            LOG.debug("Creating bridge domains on nodes {}", createdVppNodesByL2Fd);
+            fwManager.createBridgeDomainOnNodes(createdVppNodesByL2Fd);
+        } else {
+            List<DhcpRelayCommand> deletedDhcpRelays = new ArrayList<>();
+            List<DhcpRelayCommand> createdDhcpRelays = new ArrayList<>();
+            if (rPolicyBefore.getConfiguration() != null) {
+                RendererForwarding rendererForwardingBefore = rPolicyBefore.getConfiguration().getRendererForwarding();
+
+                SetMultimap<String, NodeId> vppNodesByL2FdBefore =
+                        resolveVppNodesByL2Fd(policyCtxBefore.getPolicyTable().rowKeySet(), policyCtxBefore);
+                if (!vppNodesByL2FdBefore.isEmpty()) {
+                    deletedDhcpRelays = fwManager.deleteDhcpRelay(rendererForwardingBefore, vppNodesByL2FdBefore);
+                }
+            }
+
+            if (rPolicyAfter.getConfiguration() != null) {
+                RendererForwarding rendererForwardingAfter = rPolicyAfter.getConfiguration().getRendererForwarding();
+                SetMultimap<String, NodeId> vppNodesByL2FdAfter =
+                        resolveVppNodesByL2Fd(policyCtxAfter.getPolicyTable().rowKeySet(), policyCtxAfter);
+                if (!vppNodesByL2FdAfter.isEmpty()) {
+                    createdDhcpRelays = fwManager.createDhcpRelay(rendererForwardingAfter, vppNodesByL2FdAfter);
+                }
+            }
+
+            fwManager.syncDhcpRelay(createdDhcpRelays, deletedDhcpRelays);
+        }
+
+        fwManager.syncNatEntries(policyCtxAfter);
+
+        fwManager.deleteRouting(policyCtxBefore);
+        fwManager.syncRouting(policyCtxAfter);
 
         SetView<RendererEndpointKey> createdRendEps = Sets.difference(rendEpsAfter, rendEpsBefore);
         createdRendEps.forEach(rEpKey -> fwManager.createForwardingForEndpoint(rEpKey, policyCtxAfter));
 
         SetView<RendererEndpointKey> updatedRendEps = Sets.intersection(rendEpsBefore, rendEpsAfter);
-        // TODO think about all cases, but keep it simple for now
-        updatedRendEps.forEach(rEpKey -> fwManager.removeForwardingForEndpoint(rEpKey, policyCtxBefore));
-        updatedRendEps.forEach(rEpKey -> fwManager.createForwardingForEndpoint(rEpKey, policyCtxAfter));
+        // update forwarding for endpoint
+        updatedRendEps.forEach(rEpKey -> {
+            AddressEndpointWithLocation addrEpWithLocBefore =
+                    policyCtxBefore.getAddrEpByKey().get(KeyFactory.addressEndpointKey(rEpKey));
+            AddressEndpointWithLocation addrEpWithLocAfter =
+                    policyCtxAfter.getAddrEpByKey().get(KeyFactory.addressEndpointKey(rEpKey));
+            if (isLocationChanged(addrEpWithLocBefore, addrEpWithLocAfter)) {
+                LOG.debug("Location is changed in endpoint {}", rEpKey);
+                LOG.debug("\nLocation before: {}\nLocation after: {}", addrEpWithLocBefore.getAbsoluteLocation(),
+                        addrEpWithLocAfter.getAbsoluteLocation());
+                fwManager.removeForwardingForEndpoint(rEpKey, policyCtxBefore);
+                fwManager.createForwardingForEndpoint(rEpKey, policyCtxAfter);
+            }
+        });
+        ImmutableSet<RuleGroupKey> rulesBefore = policyCtxAfter.getRuleGroupByKey().keySet();
+        ImmutableSet<RuleGroupKey> rulesAfter = policyCtxBefore.getRuleGroupByKey().keySet();
+        SetView<RuleGroupKey> removedRules = Sets.difference(rulesAfter, rulesBefore);
+        SetView<RuleGroupKey> createdRules = Sets.difference(rulesBefore, rulesAfter);
+        LOG.debug("Updated rules: {}", Sets.intersection(rulesBefore, rulesAfter));
+        LOG.debug("Removed rules {}", removedRules);
+        LOG.debug("Created rules {}", createdRules);
+        LOG.debug("Updated renderer endpoints {}", updatedRendEps);
+        LOG.debug("Created renderer endpoints {}", createdRendEps);
+        LOG.debug("Updated renderer endpoints {}", updatedRendEps);
+        aclManager.resolveRulesToConfigure(policyCtxBefore, removedRendEps, removedRules, false);
+        aclManager.resolveRulesToConfigure(policyCtxAfter, createdRendEps, createdRules, true);
+    }
+
+    private static boolean isLocationChanged(AddressEndpointWithLocation before, AddressEndpointWithLocation after) {
+        ExternalLocationCase locationBefore = ForwardingManager.resolveAndValidateLocation(before);
+        ExternalLocationCase locationAfter = ForwardingManager.resolveAndValidateLocation(after);
+        if (locationBefore == null && locationAfter == null) {
+            return false;
+        }
+        if (locationBefore == null || locationAfter == null) {
+            return true;
+        }
+        return !locationBefore.equals(locationAfter);
     }
 
     private static MapDifference<String, Collection<NodeId>> createDiffForVppNodesByL2Fd(PolicyContext policyCtxBefore,
-            PolicyContext policyCtxAfter) {
+                                                                                         PolicyContext policyCtxAfter) {
         ImmutableSet<RendererEndpointKey> rendEpsBefore = policyCtxBefore.getPolicyTable().rowKeySet();
         ImmutableSet<RendererEndpointKey> rendEpsAfter = policyCtxAfter.getPolicyTable().rowKeySet();
         SetMultimap<String, NodeId> vppNodesByL2FdBefore = resolveVppNodesByL2Fd(rendEpsBefore, policyCtxBefore);
-        SetMultimap<String, NodeId> vppNodesByL2FdAfter = resolveVppNodesByL2Fd(rendEpsAfter, policyCtxBefore);
+        SetMultimap<String, NodeId> vppNodesByL2FdAfter = resolveVppNodesByL2Fd(rendEpsAfter, policyCtxAfter);
         return Maps.difference(vppNodesByL2FdBefore.asMap(), vppNodesByL2FdAfter.asMap());
     }
 
     private void rendererPolicyCreated(RendererPolicy rPolicy) {
+        LOG.trace("VPP renderer policy version {} created", rPolicy.getVersion());
         PolicyContext policyCtx = new PolicyContext(rPolicy);
+        aclManager.cacheEndpointsByInterfaces(policyCtx);
         ImmutableSet<RendererEndpointKey> rEpKeys = policyCtx.getPolicyTable().rowKeySet();
-
         SetMultimap<String, NodeId> vppNodesByL2Fd = resolveVppNodesByL2Fd(rEpKeys, policyCtx);
-        fwManager.createVxlanBridgeDomainsOnNodes(vppNodesByL2Fd);
+        if (!ConfigUtil.getInstance().isL3FlatEnabled()) {
+            fwManager.createBridgeDomainOnNodes(vppNodesByL2Fd);
+        } else {
+            RendererForwarding rendererForwarding = rPolicy.getConfiguration().getRendererForwarding();
+            List<DhcpRelayCommand> createdDhcpRelays = fwManager.createDhcpRelay(rendererForwarding, vppNodesByL2Fd);
+            fwManager.syncDhcpRelay(createdDhcpRelays, new ArrayList<>());
+        }
 
         rEpKeys.forEach(rEpKey -> fwManager.createForwardingForEndpoint(rEpKey, policyCtx));
+        fwManager.syncNatEntries(policyCtx);
+        fwManager.syncRouting(policyCtx);
     }
 
     private void rendererPolicyDeleted(RendererPolicy rendererPolicy) {
+        LOG.trace("VPP renderer policy version {} deleted", rendererPolicy.getVersion());
         PolicyContext policyCtx = new PolicyContext(rendererPolicy);
+        aclManager.cacheEndpointsByInterfaces(policyCtx);
         ImmutableSet<RendererEndpointKey> rEpKeys = policyCtx.getPolicyTable().rowKeySet();
 
         rEpKeys.forEach(rEpKey -> fwManager.removeForwardingForEndpoint(rEpKey, policyCtx));
-
         SetMultimap<String, NodeId> vppNodesByL2Fd = resolveVppNodesByL2Fd(rEpKeys, policyCtx);
-        fwManager.removeVxlanBridgeDomainsOnNodes(vppNodesByL2Fd);
+        if (!ConfigUtil.getInstance().isL3FlatEnabled()) {
+            fwManager.removeBridgeDomainOnNodes(vppNodesByL2Fd);
+        } else {
+            RendererForwarding rendererForwarding = rendererPolicy.getConfiguration().getRendererForwarding();
+            List<DhcpRelayCommand> deletedDhcpRelays = fwManager.deleteDhcpRelay(rendererForwarding, vppNodesByL2Fd);
+            fwManager.syncDhcpRelay(new ArrayList<>(), deletedDhcpRelays);
+        }
+        fwManager.deleteNatEntries(policyCtx);
+        fwManager.deleteRouting(policyCtx);
     }
 
     private static SetMultimap<String, NodeId> resolveVppNodesByL2Fd(Set<RendererEndpointKey> rEpKeys,
-            PolicyContext policyCtx) {
+                                                                     PolicyContext policyCtx) {
         SetMultimap<String, NodeId> vppNodesByL2Fd = HashMultimap.create();
         rEpKeys.stream()
-            .map(rEpKey -> KeyFactory.addressEndpointKey(rEpKey))
-            .map(addrEpKey -> policyCtx.getAddrEpByKey().get(addrEpKey))
-            .collect(Collectors.toSet())
-            .forEach(addrEpWithLoc -> {
-                Optional<String> optL2Fd =
-                        ForwardingManager.resolveL2FloodDomain(addrEpWithLoc.getNetworkContainment());
-                if (optL2Fd.isPresent()) {
-                    ExternalLocationCase rEpLoc = ForwardingManager.resolveAndValidateLocation(addrEpWithLoc);
-                    InstanceIdentifier<?> externalNodeMountPoint = rEpLoc.getExternalNodeMountPoint();
-                    NodeId vppNode = externalNodeMountPoint.firstKeyOf(Node.class).getNodeId();
-                    vppNodesByL2Fd.put(optL2Fd.get(), vppNode);
-                }
-            });
+                .map(rEpKey -> KeyFactory.addressEndpointKey(rEpKey))
+                .map(addrEpKey -> policyCtx.getAddrEpByKey().get(addrEpKey))
+                .collect(Collectors.toSet())
+                .forEach(addrEpWithLoc -> {
+                    java.util.Optional<String> optL2Fd = ForwardingManager.resolveL2FloodDomain(addrEpWithLoc, policyCtx);
+                    if (optL2Fd.isPresent()) {
+                        ExternalLocationCase rEpLoc = ForwardingManager.resolveAndValidateLocation(addrEpWithLoc);
+                        if (rEpLoc != null) {
+                            InstanceIdentifier<?> externalNodeMountPoint = rEpLoc.getExternalNodeMountPoint();
+                            NodeId vppNode = externalNodeMountPoint.firstKeyOf(Node.class).getNodeId();
+                            vppNodesByL2Fd.put(optL2Fd.get(), vppNode);
+                        }
+                    }
+                });
         return vppNodesByL2Fd;
     }