From 434e17e6cb1ec70b200e84b1ab29c98e9e3857f7 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 3 Apr 2026 09:51:06 +0000 Subject: [PATCH 1/2] Initial plan From f5e7063d48ceb0b57daa5e77d47fee481332f3ca Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Fri, 3 Apr 2026 09:59:52 +0000 Subject: [PATCH 2/2] Fix telemetry markers to only emit on critical errors, not normal events Agent-Logs-Url: https://github.com/rdkcentral/middleware-player-interface/sessions/fd2e29b3-4a81-42cd-bd05-3eda1680630b Co-authored-by: dp0000 <53818367+dp0000@users.noreply.github.com> --- playerLogManager/PlayerLogManager.cpp | 21 ++++++++++++++++----- 1 file changed, 16 insertions(+), 5 deletions(-) diff --git a/playerLogManager/PlayerLogManager.cpp b/playerLogManager/PlayerLogManager.cpp index 213ec1e1..adf2161b 100644 --- a/playerLogManager/PlayerLogManager.cpp +++ b/playerLogManager/PlayerLogManager.cpp @@ -138,7 +138,9 @@ void logprintf(MW_LogLevel logLevelIndex, const char* func, int line, const char { // remap MW log levels to Ethan log levels int ethanLogLevel; // Important: in production builds, Ethan logger filters out everything - // except ETHAN_LOG_MILESTONE and ETHAN_LOG_FATAL + // except ETHAN_LOG_MILESTONE and ETHAN_LOG_FATAL. + // Only errors and critical failures should emit telemetry markers; + // normal/success events should not generate telemetry. switch (logLevelIndex) { case mLOGLEVEL_TRACE: @@ -146,15 +148,24 @@ void logprintf(MW_LogLevel logLevelIndex, const char* func, int line, const char ethanLogLevel = ETHAN_LOG_DEBUG; break; - case mLOGLEVEL_ERROR: - ethanLogLevel = ETHAN_LOG_FATAL; + case mLOGLEVEL_INFO: + ethanLogLevel = ETHAN_LOG_INFO; break; - case mLOGLEVEL_INFO: // note: we rely on eLOGLEVEL_INFO at tune time for triage case mLOGLEVEL_WARN: + ethanLogLevel = ETHAN_LOG_WARNING; + break; + case mLOGLEVEL_MIL: + ethanLogLevel = ETHAN_LOG_WARNING; + break; + + case mLOGLEVEL_ERROR: + ethanLogLevel = ETHAN_LOG_FATAL; + break; + default: - ethanLogLevel = ETHAN_LOG_MILESTONE; + ethanLogLevel = ETHAN_LOG_WARNING; break; } vethanlog(ethanLogLevel,NULL,NULL,-1,format_ptr, args);