From 730d546f0555f80de893b3acd68bf10864482575 Mon Sep 17 00:00:00 2001
From: Charles Chan <charles@opennetworking.org>
Date: Sat, 9 Jan 2021 08:05:57 +0000
Subject: [PATCH] Add debug log in P4RuntimePipelineConfigClient

Change-Id: Ie71af2f80b12b8ad99c8f597af1c3f026a532c4b
---
 .../p4runtime/ctl/client/PipelineConfigClientImpl.java        | 4 ++++
 1 file changed, 4 insertions(+)

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 3b628b45d1..c7e21cbdd0 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 @@ final class PipelineConfigClientImpl implements P4RuntimePipelineConfigClient {
     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();
         }