Skip to content

RDKEMW-11362: memcr with reduced logs for PROD#479

Open
adrianM27 wants to merge 5 commits intodevelopfrom
feature/RDKEMW-11362-memcr_logs_prod
Open

RDKEMW-11362: memcr with reduced logs for PROD#479
adrianM27 wants to merge 5 commits intodevelopfrom
feature/RDKEMW-11362-memcr_logs_prod

Conversation

@adrianM27
Copy link
Contributor

No description provided.

@adrianM27 adrianM27 requested review from a team as code owners January 21, 2026 13:14
Copilot AI review requested due to automatic review settings January 21, 2026 13:14
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This pull request implements reduced logging for production builds of the memcr memory hibernate/restore tool. The changes switch the source repository to a personal fork and add conditional log level compilation based on the BUILD_VARIANT.

Changes:

  • Modified build recipe to use LOG_LEVEL variable (1 for prod, 3 for non-prod) during compilation
  • Changed source repository from LibertyGlobal official repo to adrianM27 personal fork
  • Updated SRCREV to point to a new commit with the logging changes
  • Updated patch metadata with new commit hashes and dates

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 2 comments.

File Description
recipes-extended/memcr/memcr_git.bb Added LOG_LEVEL build variable, changed repository URL to personal fork, updated SRCREV, removed branch specification
recipes-extended/memcr/files/0001-RDK-54059-retry-ptrace-seize-on-EPERM.patch Updated patch metadata (commit hash, author email, date, line numbers)
recipes-extended/memcr/files/0001-RDK-47153-Option-to-set-dump-dir-and-compression-per.patch Updated patch metadata and added conditional compilation for debug message based on LOG_LEVEL

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings February 3, 2026 08:47
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants