Bug-590: Packet loss on first time ping test

Bump severity of log in exception code path.
Undo copy and paste mistake, which caused unwanted line left behind.

Change-Id: If73e516635172e31e794cd1524611c1cdb15f89f
Signed-off-by: Flavio Fernandes <ffernand@redhat.com>

diff --git a/opendaylight/samples/simpleforwarding/src/main/java/org/opendaylight/controller/samples/simpleforwarding/internal/SimpleForwardingImpl.java b/opendaylight/samples/simpleforwarding/src/main/java/org/opendaylight/controller/samples/simpleforwarding/internal/SimpleForwardingImpl.java
index d6957385bd8278d566ba31fc0cf795539d1161c8..a48d3317738fff283cf37d6eecbbb4cf96781ab4 100644 (file)
--- a/opendaylight/samples/simpleforwarding/src/main/java/org/opendaylight/controller/samples/simpleforwarding/internal/SimpleForwardingImpl.java
+++ b/opendaylight/samples/simpleforwarding/src/main/java/org/opendaylight/controller/samples/simpleforwarding/internal/SimpleForwardingImpl.java
@@ -152,7 +152,7 @@ public class SimpleForwardingImpl implements IfNewHostNotify,
                     }
                 }
             } catch (IllegalStateException e) {
-                log.debug("IllegalStateException Received by PendingPacketsAgerTimerHandler from: {}",
+                log.warn("IllegalStateException Received by PendingPacketsAgerTimerHandler from: {}",
                         e.getMessage());
             }
         }
@@ -186,7 +186,6 @@ public class SimpleForwardingImpl implements IfNewHostNotify,
      * hope that the destination is known by hostTracker.
      */
     private void sendPendingPacket(InetAddress dIP) {
-        pendingPacketDestinations.get(dIP);
         PendingPacketData pendingPacketData = pendingPacketDestinations.get(dIP);
         if (pendingPacketData != null) {
             handlePuntedIPPacket(pendingPacketData.pkt, pendingPacketData.incomingNodeConnector, false);