Skip to content

feat: Add parameter for verify_messages to use optional custom dmesg …#6

Open
lkleinax wants to merge 1 commit into
intel:mainfrom
lkleinax:feat__Add_parameter_for_verify_messages_to_use_optional_custom_dmesg_whitelist
Open

feat: Add parameter for verify_messages to use optional custom dmesg …#6
lkleinax wants to merge 1 commit into
intel:mainfrom
lkleinax:feat__Add_parameter_for_verify_messages_to_use_optional_custom_dmesg_whitelist

Conversation

@lkleinax
Copy link
Copy Markdown

@lkleinax lkleinax commented Jan 12, 2026

This pull request updates the verify_messages method in mfd_dmesg/base.py to make the list of benign messages (the whitelist) configurable, improving flexibility for users who want to customize which dmesg messages are ignored. The most important changes are:

Flexibility improvements

  • Modified the verify_messages method to accept an optional dmesg_whitelist parameter, defaulting to DMESG_WHITELIST, allowing callers to specify a custom list of benign messages to ignore.
  • Updated the internal logic of verify_messages to use the provided dmesg_whitelist parameter instead of the hardcoded DMESG_WHITELIST variable when checking for benign messages.

Copilot AI review requested due to automatic review settings January 12, 2026 13:35
Copy link
Copy Markdown

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 adds support for using a custom dmesg whitelist by introducing an optional parameter to the verify_messages method. This allows callers to override the default DMESG_WHITELIST with their own list of benign messages to ignore when checking for errors.

Changes:

  • Added optional custom_dmesg_whitelist parameter to verify_messages method
  • Updated method to use custom whitelist when provided, falling back to default DMESG_WHITELIST
  • Added parameter documentation

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

Comment thread mfd_dmesg/base.py Outdated
Comment thread mfd_dmesg/base.py Outdated
…whitelist

Fixes intel#5
Signed-off-by: Łukasz Kleina <lukaszx.kleina@intel.com>
@lkleinax lkleinax force-pushed the feat__Add_parameter_for_verify_messages_to_use_optional_custom_dmesg_whitelist branch from 0618bd7 to 140b3be Compare February 25, 2026 15:26
@lkleinax lkleinax requested a review from mchromin February 25, 2026 15:26
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.

3 participants