From f78306f0847dc78f0542b2a55a1a918ee31b5b8c Mon Sep 17 00:00:00 2001 From: Daniel Park Date: Wed, 6 Apr 2016 17:57:48 +0900 Subject: [PATCH] Fixing arp event log in OpenstackRoutingArpHandler Change-Id: I8d7c1737fcceccb201703a009243039db9e40e41 --- .../routing/OpenstackRoutingArpHandler.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/apps/openstacknetworking/openstackrouting/src/main/java/org/onosproject/openstacknetworking/routing/OpenstackRoutingArpHandler.java b/apps/openstacknetworking/openstackrouting/src/main/java/org/onosproject/openstacknetworking/routing/OpenstackRoutingArpHandler.java index 8314847950..ff0a4efe6b 100644 --- a/apps/openstacknetworking/openstackrouting/src/main/java/org/onosproject/openstacknetworking/routing/OpenstackRoutingArpHandler.java +++ b/apps/openstacknetworking/openstackrouting/src/main/java/org/onosproject/openstacknetworking/routing/OpenstackRoutingArpHandler.java @@ -18,7 +18,6 @@ package org.onosproject.openstacknetworking.routing; import org.onlab.packet.ARP; import org.onlab.packet.EthType; import org.onlab.packet.Ethernet; -import org.onlab.packet.IPv4; import org.onlab.packet.Ip4Address; import org.onlab.packet.IpAddress; import org.onlab.packet.MacAddress; @@ -94,11 +93,13 @@ public class OpenstackRoutingArpHandler { checkNotNull(context, "context can not be null"); checkNotNull(ethernet, "ethernet can not be null"); - log.info("arpEvent called from {} to {}", - Ip4Address.valueOf(((IPv4) ethernet.getPayload()).getSourceAddress()).toString(), - Ip4Address.valueOf(((IPv4) ethernet.getPayload()).getDestinationAddress()).toString()); + ARP arp = (ARP) ethernet.getPayload(); + log.debug("arpEvent called from {} to {}", + Ip4Address.valueOf(arp.getSenderProtocolAddress()).toString(), + Ip4Address.valueOf(arp.getTargetProtocolAddress()).toString()); + if (arp.getOpCode() != ARP.OP_REQUEST) { return; }