Skip to content

Comments

RDKEMW-13749: Add Info log level logs for subtle fails#47

Merged
gomathishankar37 merged 3 commits intodevelopfrom
topic/RDKEMW-13749
Feb 17, 2026
Merged

RDKEMW-13749: Add Info log level logs for subtle fails#47
gomathishankar37 merged 3 commits intodevelopfrom
topic/RDKEMW-13749

Conversation

@gomathishankar37
Copy link
Contributor

No description provided.

@gomathishankar37 gomathishankar37 requested a review from a team as a code owner February 12, 2026 15:34
Copilot AI review requested due to automatic review settings February 12, 2026 15:34
Copy link

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 PR updates log levels from ERROR to INFO for expected failure scenarios in the crash upload service. The changes address cases where the absence of crash dumps or inaccessible directories are normal operational conditions rather than errors requiring investigation.

Changes:

  • Downgraded log level from ERROR to INFO for directory access failures in directory_has_pattern()
  • Downgraded log level from ERROR to INFO for prerequisites check failures in main()
  • Minor grammar improvement: changed "So exit" to "Exiting"

Reviewed changes

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

File Description
c_sourcecode/src/utils/prerequisites.c Downgraded opendir failure and no dumps found messages from ERROR to INFO level; improved grammar
c_sourcecode/src/main.c Downgraded prerequisites check failure message from ERROR to INFO level

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

Copy link
Contributor

@satya200 satya200 left a comment

Choose a reason for hiding this comment

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

Looks good

Copilot AI review requested due to automatic review settings February 17, 2026 12:21
Copy link

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 2 out of 2 changed files in this pull request and generated 3 comments.


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

@gomathishankar37 gomathishankar37 merged commit 43ce212 into develop Feb 17, 2026
19 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Feb 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants