commit 53c361eb5f10392e2fc303dec7d74bfd4d9dc20b
author Charles Chan <charles@opennetworking.org> Sat Jan 09 08:05:57 2021 +0000
committer Charles Chan <charles@opennetworking.org> Tue Jan 19 16:46:50 2021 -0800
tree ca5822c3da720e34a687bfc4338bce969dfb67df
parent e2aecd0d491843bb202f44c740932e3b2d6a60d1

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();
         }