RDKEMW-12849 : Update to print logs to console#502
RDKEMW-12849 : Update to print logs to console#502karuna2git wants to merge 2 commits intodevelopfrom
Conversation
RDKEMW-12849: Update to print logs to console Reason for change: RDKLogger sends the output directly to syslog to gain the performance. But the platform components like `rdkfwupdater`, `rfc`, `rdm`, `logupload` & `crashupload` are spun by scripts and console output are redirected to file manually. So, for these components, we have to configure logger to print to console instead syslog. Once these components become standalone and launched by systemd service independently, this change can be removed to gain performance. Test Procedure: Ensure logs from above mentioned components are printed. Risks: None. In the past also output was redirected; so whatever gained in last 2 weeks is not present anymore. Signed-off-by: Karunakaran A <[[email protected]](mailto:[email protected])>
There was a problem hiding this comment.
Pull request overview
This PR updates the commonutilities component to print logs to console instead of syslog for specific platform components (rdkfwupdater, rfc, rdm, logupload, crashupload). This is a temporary workaround needed because these components are launched by scripts that redirect console output to files, rather than being launched as standalone systemd services.
Changes:
- Updated SRCREV in commonutilities_git.bb to point to commit 9fd106a65225482af2321ce342b74cb37fbef05f which implements console logging
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| # To have a possibility to override SRC_URI later, we are introducing the following workaround: | ||
| SRCREV = "5c7dfc405cd34da3ab1a1121caf8733ebe35d879" | ||
| SRCREV = "9fd106a65225482af2321ce342b74cb37fbef05f" |
There was a problem hiding this comment.
The SRCREV is being updated in the recipe file, but there's a potential override in conf/distro/include/rdkb.inc at line 199 that sets SRCREV:pn-commonutilities to "071361f284ba9049bf7d8cb9a75b583b9b1e353b". This override will take precedence over the SRCREV in the recipe for RDKB builds, which means the new commit "9fd106a65225482af2321ce342b74cb37fbef05f" may not be used in those configurations. If this change is intended to apply to RDKB builds, the SRCREV in conf/distro/include/rdkb.inc should also be updated to match this new value.
| SRCREV = "9fd106a65225482af2321ce342b74cb37fbef05f" | |
| SRCREV = "071361f284ba9049bf7d8cb9a75b583b9b1e353b" |
1 participant
RDKEMW-12849: Update to print logs to console
Reason for change: RDKLogger sends the output directly to syslog to gain the performance. But the platform components like
rdkfwupdater,rfc,rdm,logupload&crashuploadare spun by scripts and console output are redirected to file manually. So, for these components, we have to configure logger to print to console instead syslog.Once these components become standalone and launched by systemd service independently, this change can be removed to gain performance.
Test Procedure: Ensure logs from above mentioned components are printed.
Risks: None. In the past also output was redirected; so whatever gained in last 2 weeks is not present anymore.
Signed-off-by: Karunakaran A <[email protected]>