commit f12870f1abe039fce55fcb096e838e91bde7b9da
author Matteo Scandolo <teo@opennetworking.org> Thu Dec 05 13:29:20 2019 -0800
committer Matteo Scandolo <teo@opennetworking.org> Mon Dec 16 12:15:13 2019 -0800
tree 2f5d090f04785d376fb389110e86b6bded17910b
parent 8e412456c8d7a200b0a9c24f6b231d0088f3fd86

[VOL-2237] Adding device and port ID to meter error

Change-Id: I8af00c472c48c8d9291d2aaa07b5baa8e6a19f81

app/src/main/java/org/opencord/olt/impl/Olt.java

diff --git a/app/src/main/java/org/opencord/olt/impl/Olt.java b/app/src/main/java/org/opencord/olt/impl/Olt.java
index 9d5240f..e0ac2b3 100644
--- a/app/src/main/java/org/opencord/olt/impl/Olt.java
+++ b/app/src/main/java/org/opencord/olt/impl/Olt.java
@@ -735,8 +735,8 @@
                     }
                 }));
             } else {
-                log.warn("Meter installation error while sending upstream flows. " +
-                        "Result {} and MeterId {}", result, upstreamMeterId);
+                log.error("Meter installation error for {} on {} while sending upstream flows. " +
+                        "Result {} and MeterId {}", deviceId, subscriberPort, result, upstreamMeterId);
             }
         });
 
@@ -761,8 +761,8 @@
                     }
                 }));
             } else {
-                log.warn("Meter installation error while sending downstream flows. " +
-                        "Result {} and MeterId {}", result, downstreamMeterId);
+                log.error("Meter installation error for {} on {} while sending downstream flows. " +
+                        "Result {} and MeterId {}", deviceId, subscriberPort, result, downstreamMeterId);
             }
         });
 
@@ -981,8 +981,8 @@
                 }));
 
             } else {
-                log.warn("Meter installation error while sending upstream flows. " +
-                        "Result {} and MeterId {}", result, upstreamMeterId);
+                log.error("Meter installation error for {} on {} while sending upstream flows. " +
+                        "Result {} and MeterId {}", deviceId, subscriberPort, result, upstreamMeterId);
             }
         });
 
@@ -1006,8 +1006,8 @@
                     }
                 }));
             } else {
-                log.warn("Meter installation error while sending upstream flows. " +
-                        "Result {} and MeterId {}", result, downstreamMeterId);
+                log.error("Meter installation error for {} on {} while sending downstream flows. " +
+                        "Result {} and MeterId {}", deviceId, subscriberPort, result, downstreamMeterId);
             }
         });
 
@@ -1292,8 +1292,8 @@
 
                 flowObjectiveService.filter(devId, eapol);
             } else {
-                log.warn("Meter installation error while sending eapol trap flow. " +
-                        "Result {} and MeterId {}", result, mId);
+                log.error("Meter installation error for {} on {} while sending eapol trap flow. " +
+                        "Result {} and MeterId {}", devId, portNumber, result, mId);
             }
         });
     }