RDK-60409: Component independent Tool Development for Noisy Log detection#430
RDK-60409: Component independent Tool Development for Noisy Log detection#430dshett549 wants to merge 13 commits intordkcentral:developfrom
Conversation
…tion Reason for change: Component independent tool for Noisy Log detection Test Procedure: Tested and verified Risks: Medium Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
This PR introduces a component-independent tool for detecting noisy logs, sensitive information, and severity violations in log files. The tool is designed to analyze log quality and generate HTML reports highlighting potential issues.
Changes:
- Changed default log level from
RT_LOG_WARNtoRT_LOG_INFOin rtLog.c to capture more detailed logs - Added Python-based log analyzer script (
noisylogdetector.py) with pattern matching capabilities - Added configurable YAML rules file for defining detection patterns
- Added GitHub Actions workflow to automatically run log analysis on test outputs
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 12 comments.
| File | Description |
|---|---|
| src/rtmessage/rtLog.c | Changes default log level to INFO to enable more verbose logging for analysis |
| noisylogdetector.py | New Python script that analyzes logs for noisy patterns, sensitive data, and severity violations |
| rules.yml | Configuration file defining patterns for sensitive data detection, noisy log levels, and failure keywords |
| .github/workflows/log-quality-rules.yml | GitHub Actions workflow that builds, tests, and analyzes logs from various test executions |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 4 out of 4 changed files in this pull request and generated 19 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 11 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 8 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
Reason for change: Component independent tool for Noisy Log detection
Test Procedure: Tested and verified
Risks: Medium
Signed-off-by: dshett549 DEEPTHICHANDRASHEKAR_SHETTY@comcast.com