-
Notifications
You must be signed in to change notification settings - Fork 46
RDK-60409: Component independent Tool Development for Noisy Log detection #432
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: develop
Are you sure you want to change the base?
Conversation
…ction Signed-off-by: dshett549 <DEEPTHICHANDRASHEKAR_SHETTY@comcast.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Adds a component-independent “noisy log detection” tool and CI integration to analyze generated logs for noisy levels, sensitive patterns, and severity misuse, producing HTML reports as build artifacts.
Changes:
- Introduces a YAML ruleset (
rules.yml) for sensitive pattern detection and severity/noise heuristics. - Adds a Python-based log analyzer (
noisylogdetector.py) that scans logs and generates an HTML report. - Adds a GitHub Actions workflow to run existing tests, analyze logs, and upload the resulting HTML reports as artifacts.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
rules.yml |
Defines sensitive regex patterns plus “noisy levels” and severity enforcement keywords/levels used by the analyzer. |
noisylogdetector.py |
Implements log parsing, rule-based detection, and HTML report generation. |
.github/workflows/log-quality-rules.yml |
Runs builds/tests, executes the analyzer on produced logs, and uploads HTML reports as artifacts. |
💡 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.
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 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>
There was a problem hiding this 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 5 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.
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 5 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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]