commit 730d546f0555f80de893b3acd68bf10864482575
author Charles Chan <charles@opennetworking.org> Sat Jan 09 08:05:57 2021 +0000
committer Charles Chan <charles@opennetworking.org> Tue Jan 19 16:47:21 2021 -0800
tree ff2822a5e38d9f0ecd6bdc8f36fa2d64442c2d1c
parent 48103846ec1ddda1b0ab17088325eab97e11dbf1

Add debug log in P4RuntimePipelineConfigClient

Change-Id: Ie71af2f80b12b8ad99c8f597af1c3f026a532c4b

protocols/p4runtime/ctl/src/main/java/org/onosproject/p4runtime/ctl/client/PipelineConfigClientImpl.java

diff --git a/protocols/p4runtime/ctl/src/main/java/org/onosproject/p4runtime/ctl/client/PipelineConfigClientImpl.java b/protocols/p4runtime/ctl/src/main/java/org/onosproject/p4runtime/ctl/client/PipelineConfigClientImpl.java
index 3b628b4..c7e21cb 100644
--- a/protocols/p4runtime/ctl/src/main/java/org/onosproject/p4runtime/ctl/client/PipelineConfigClientImpl.java
+++ b/protocols/p4runtime/ctl/src/main/java/org/onosproject/p4runtime/ctl/client/PipelineConfigClientImpl.java
@@ -166,16 +166,20 @@
     private boolean comparePipelineConfig(
             PiPipeconf pipeconf, ForwardingPipelineConfig cfgFromDevice) {
         if (cfgFromDevice == null) {
+            log.debug("Failed to comparePipelineConfig. cfgFromDevice is null");
             return false;
         }
 
         final ForwardingPipelineConfig expectedCfg = buildForwardingPipelineConfigMsg(
                 pipeconf, null);
         if (expectedCfg == null) {
+            // Problem logged by buildForwardingPipelineConfigMsg
             return false;
         }
 
         if (cfgFromDevice.hasCookie()) {
+            log.debug("Cookie from device = {}", cfgFromDevice.getCookie().getCookie());
+            log.debug("Pipeconf fingerprint = {}", pipeconf.fingerprint());
             return cfgFromDevice.getCookie().getCookie() == pipeconf.fingerprint();
         }