commit 3279cc588375a1e430b7e2723574d6eeea6e03e0
author Matt Spinler <spinler@us.ibm.com> Tue Feb 15 11:02:00 2022 -0600
committer Matt Spinler <spinler@us.ibm.com> Fri Feb 18 09:37:34 2022 -0600
tree 7e99d10735045dfef28ed19c5ecca5a8ce7932b4
parent 31d78f77034a895c07a14475f67e4303aea08938

PEL:  Change python plugin error traces to debug

There's been lots of complaints and questions about what these are, if
they're important, and why they show up so much so just make them debug
traces.  The plugin parser developers can still enable debug traces to
see the errors if they notice some unparsed sections.

Signed-off-by: Matt Spinler <spinler@us.ibm.com>
Change-Id: If68e0e080eff92225331e39c3c41faa007fe785e

extensions/openpower-pels/src.cpp

diff --git a/extensions/openpower-pels/src.cpp b/extensions/openpower-pels/src.cpp
index f4807f5..9da5992 100644
--- a/extensions/openpower-pels/src.cpp
+++ b/extensions/openpower-pels/src.cpp
@@ -254,10 +254,10 @@
     }
     if (!pErrStr.empty())
     {
-        log<level::ERR>("Python exception thrown by parser",
-                        entry("ERROR=%s", pErrStr.c_str()),
-                        entry("SRC=%s", hexwords.front().c_str()),
-                        entry("PARSER_MODULE=%s", module.c_str()));
+        log<level::DEBUG>("Python exception thrown by parser",
+                          entry("ERROR=%s", pErrStr.c_str()),
+                          entry("SRC=%s", hexwords.front().c_str()),
+                          entry("PARSER_MODULE=%s", module.c_str()));
     }
     return std::nullopt;
 }

extensions/openpower-pels/user_data_json.cpp

diff --git a/extensions/openpower-pels/user_data_json.cpp b/extensions/openpower-pels/user_data_json.cpp
index 107e026..1c1c966 100644
--- a/extensions/openpower-pels/user_data_json.cpp
+++ b/extensions/openpower-pels/user_data_json.cpp
@@ -382,12 +382,12 @@
     }
     if (!pErrStr.empty())
     {
-        log<level::ERR>("Python exception thrown by parser",
-                        entry("ERROR=%s", pErrStr.c_str()),
-                        entry("PARSER_MODULE=%s", module.c_str()),
-                        entry("SUBTYPE=0x%X", subType),
-                        entry("VERSION=%d", version),
-                        entry("DATA_LENGTH=%lu\n", data.size()));
+        log<level::DEBUG>("Python exception thrown by parser",
+                          entry("ERROR=%s", pErrStr.c_str()),
+                          entry("PARSER_MODULE=%s", module.c_str()),
+                          entry("SUBTYPE=0x%X", subType),
+                          entry("VERSION=%d", version),
+                          entry("DATA_LENGTH=%lu\n", data.size()));
     }
     return std::nullopt;
 }