-
Notifications
You must be signed in to change notification settings - Fork 247
chore: add kernel log validation to abe2es #7835
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
Conversation
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 new E2E validation step to scan kernel logs for critical errors (panic/lockup/OOM/I/O), intended to catch severe node issues during validation runs.
Changes:
- Added
ValidateKernelLogsvalidator that grepsdmesgfor critical kernel error patterns and fails the scenario if matches are found. - Wired the new validator into
ValidateCommonLinuxso it runs as part of default Linux validations.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| e2e/validators.go | Introduces ValidateKernelLogs with category-based regex matching and failure reporting/logging. |
| e2e/validation.go | Adds ValidateKernelLogs to the common Linux validation sequence. |
…add-kernel-warning-e2e-validator
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.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 2 out of 2 changed files in this pull request and generated 3 comments.
…b.com/Azure/AgentBaker into lily/add-kernel-warning-e2e-validator
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 2 out of 2 changed files in this pull request and generated 2 comments.
What this PR does / why we need it:
Adds validation against critical kernel errors and aims to catch issues like these: https://supportability.visualstudio.com/AzureContainers/_workitems/edit/163429
Which issue(s) this PR fixes:
Fixes #