Fixed logging level

Change-Id: I96b5d6549166aa5db8293098b41e9c9d8a8443d2
Signed-off-by: Martin Sunal <msunal@cisco.com>

diff --git a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/InterfaceManager.java b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/InterfaceManager.java
index 18504fa821a4bf18b65e7b303319b7a9578bb92c..5340f929865337cc066ffb7c5e022fb1a4b2718a 100644 (file)
--- a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/InterfaceManager.java
+++ b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/iface/InterfaceManager.java
@@ -386,11 +386,11 @@ public class InterfaceManager implements AutoCloseable {
 
     private static boolean hasNodeAndInterface(VppEndpoint vppEp) {
         if (vppEp.getVppNodePath() == null) {
-            LOG.trace("vpp-node is missing. {}", vppEp);
+            LOG.debug("vpp-node is missing. {}", vppEp);
             return false;
         }
         if (Strings.isNullOrEmpty(vppEp.getVppInterfaceName())) {
-            LOG.trace("vpp-interface-name is missing. {}", vppEp);
+            LOG.debug("vpp-interface-name is missing. {}", vppEp);
             return false;
         }
         return true;

diff --git a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/BridgeDomainManagerImpl.java b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/BridgeDomainManagerImpl.java
index e57aed4ad87fbc6956f264a3d1082e850e4fdb9c..54f2b8d0da3ab96708ad832407fa3bcb3214009b 100644 (file)
--- a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/BridgeDomainManagerImpl.java
+++ b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/BridgeDomainManagerImpl.java
@@ -97,7 +97,7 @@ public class BridgeDomainManagerImpl implements BridgeDomainManager {
             this.modificationForFutureSet = Preconditions.checkNotNull(modificationForFutureSet);
             this.iid = Preconditions.checkNotNull(iid);
             registeredListener = dataProvider.registerDataTreeChangeListener(iid, this);
-            LOG.trace("Registered listener for path {}", iid.getRootIdentifier());
+            LOG.debug("Registered listener for path {}", iid.getRootIdentifier());
         }
 
         @Override
@@ -114,7 +114,7 @@ public class BridgeDomainManagerImpl implements BridgeDomainManager {
 
         private void unregisterOnTrue(boolean _true) {
             if (_true) {
-                LOG.trace("Unregistering listener for path {}", iid.getRootIdentifier());
+                LOG.debug("Unregistering listener for path {}", iid.getRootIdentifier());
                 if (registeredListener != null) {
                     registeredListener.close();
                 }

diff --git a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/ForwardingManager.java b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/ForwardingManager.java
index dd04d7735813a9d755589b1e407cff22bab6e8b9..6e7ed5a1717898c33c32bcf13679597d77b72b1e 100644 (file)
--- a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/ForwardingManager.java
+++ b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/ForwardingManager.java
@@ -105,7 +105,7 @@ public final class ForwardingManager {
     private void createVxlanBridgeDomains(String bd, VxlanVni vni, Set<NodeId> vppNodes) {
         for (NodeId vppNode : vppNodes) {
             try {
-                LOG.trace("Creating VXLAN bridge-domain {} on node {} with VNI {}", bd, vppNode.getValue(),
+                LOG.debug("Creating VXLAN bridge-domain {} on node {} with VNI {}", bd, vppNode.getValue(),
                         vni);
                 // TODO think about propagating ListenableFuture - timeout set as workaround
                 bdManager.createVxlanBridgeDomainOnVppNode(bd, vni, vppNode).get(WAIT_FOR_BD_CREATION,
@@ -122,7 +122,7 @@ public final class ForwardingManager {
     private void createVlanBridgeDomains(String bd, VlanId vlanId, Set<NodeId> vppNodes) {
         for (NodeId vppNode : vppNodes) {
             try {
-                LOG.trace("Creating VLAN bridge-domain {} on node {} with VLAN ID {}", bd, vppNode.getValue(),
+                LOG.debug("Creating VLAN bridge-domain {} on node {} with VLAN ID {}", bd, vppNode.getValue(),
                         vlanId.getValue());
                 // TODO think about propagating ListenableFuture - timeout set as workaround
                 bdManager.createVlanBridgeDomainOnVppNode(bd, vlanId, vppNode).get(WAIT_FOR_BD_CREATION,

diff --git a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/VppRendererPolicyManager.java b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/VppRendererPolicyManager.java
index 85d4205e3a73f6de7e29da086ea42dc82bada8d9..4ab79e399c87391df4b37fad819eff229d3acf67 100644 (file)
--- a/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/VppRendererPolicyManager.java
+++ b/renderers/vpp/src/main/java/org/opendaylight/groupbasedpolicy/renderer/vpp/policy/VppRendererPolicyManager.java
@@ -65,12 +65,12 @@ public class VppRendererPolicyManager {
         RendererPolicyBuilder responseBuilder = new RendererPolicyBuilder();
         switch (event.getDtoModificationType()) {
             case CREATED:
-                LOG.trace("CREATED : {}", event.getIid());
+                LOG.debug("CREATED : {}", event.getIid());
                 responseBuilder.setVersion(event.getAfter().get().getVersion());
                 rendererPolicyCreated(event.getAfter().get());
                 break;
             case UPDATED:
-                LOG.trace("UPDATED: {}", event.getIid());
+                LOG.debug("UPDATED: {}", event.getIid());
                 RendererPolicy rPolicyBefore = event.getBefore().get();
                 RendererPolicy rPolicyAfter = event.getAfter().get();
                 responseBuilder.setVersion(rPolicyAfter.getVersion());
@@ -83,7 +83,7 @@ public class VppRendererPolicyManager {
                 }
                 break;
             case DELETED:
-                LOG.trace("DELETED: {}", event.getIid());
+                LOG.debug("DELETED: {}", event.getIid());
                 responseBuilder.setVersion(event.getBefore().get().getVersion());
                 rendererPolicyDeleted(event.getBefore().get());
                 break;
@@ -151,20 +151,20 @@ public class VppRendererPolicyManager {
         ImmutableSet<RendererEndpointKey> rendEpsAfter = policyCtxAfter.getPolicyTable().rowKeySet();
 
         SetView<RendererEndpointKey> removedRendEps = Sets.difference(rendEpsBefore, rendEpsAfter);
-        LOG.trace("Removed renderer endpoints {}", removedRendEps);
+        LOG.debug("Removed renderer endpoints {}", removedRendEps);
         removedRendEps.forEach(rEpKey -> fwManager.removeForwardingForEndpoint(rEpKey, policyCtxBefore));
 
-        LOG.trace("Removed bridge domains on nodes {}", removedVppNodesByL2Fd);
-        LOG.trace("Created bridge domains on nodes {}", createdVppNodesByL2Fd);
+        LOG.debug("Removed bridge domains on nodes {}", removedVppNodesByL2Fd);
+        LOG.debug("Created bridge domains on nodes {}", createdVppNodesByL2Fd);
         fwManager.removeBridgeDomainOnNodes(removedVppNodesByL2Fd);
         fwManager.createBridgeDomainOnNodes(createdVppNodesByL2Fd);
 
         SetView<RendererEndpointKey> createdRendEps = Sets.difference(rendEpsAfter, rendEpsBefore);
-        LOG.trace("Created renderer endpoints {}", createdRendEps);
+        LOG.debug("Created renderer endpoints {}", createdRendEps);
         createdRendEps.forEach(rEpKey -> fwManager.createForwardingForEndpoint(rEpKey, policyCtxAfter));
 
         SetView<RendererEndpointKey> updatedRendEps = Sets.intersection(rendEpsBefore, rendEpsAfter);
-        LOG.trace("Updated renderer endpoints {}", updatedRendEps);
+        LOG.debug("Updated renderer endpoints {}", updatedRendEps);
         // update forwarding for endpoint
         updatedRendEps.forEach(rEpKey -> {
             AddressEndpointWithLocation addrEpWithLocBefore =
@@ -173,7 +173,7 @@ public class VppRendererPolicyManager {
                     policyCtxAfter.getAddrEpByKey().get(KeyFactory.addressEndpointKey(rEpKey));
             if (isLocationChanged(addrEpWithLocBefore, addrEpWithLocAfter)) {
                 LOG.debug("Location is changed in endpoint {}", rEpKey);
-                LOG.trace("\nLocation before: {}\nLocation after: {}", addrEpWithLocBefore.getAbsoluteLocation(),
+                LOG.debug("\nLocation before: {}\nLocation after: {}", addrEpWithLocBefore.getAbsoluteLocation(),
                         addrEpWithLocAfter.getAbsoluteLocation());
                 fwManager.removeForwardingForEndpoint(rEpKey, policyCtxBefore);
                 fwManager.createForwardingForEndpoint(rEpKey, policyCtxAfter);