Skip to content

feat: Allow filtering interface using mac_address - #15

Merged
mchromin merged 1 commit into
intel:mainfrom
dhiraj3880:filter_mac_address
May 15, 2026
Merged

mchromin merged 1 commit into
intel:mainfrom
dhiraj3880:filter_mac_address

Conversation

@dhiraj3880

Copy link
Copy Markdown
Contributor

Specifying a mac_address in topology also requires the interface getting filtered in the call to mfd-network-adapter

Copilot AI review requested due to automatic review settings May 13, 2026 20:52

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Adds support for filtering topology-defined network interfaces by mac_address when resolving InterfaceInfo via mfd-network-adapter.

Changes:

  • Passes interface_model.mac_address into _filter_interfaces_info.
  • Preserves existing topology-based filters such as PCI address, device, family, speed, names, and random selection.

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

Comment thread mfd_host/base.py Resolved
Comment thread mfd_host/base.py Resolved
Comment thread mfd_host/base.py Resolved
@coveralls-official

coveralls-official Bot commented May 14, 2026 •

Copy link
Copy Markdown

Coverage Status

coverage: 94.021% (+0.005%) from 94.016% — dhiraj3880:filter_mac_address into intel:main

Copilot AI left a comment

Copy link
Copy Markdown

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 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread mfd_host/base.py Resolved
Specifying a mac_address in topology also requires the interface getting filtered in the call to mfd-network-adapter.

Signed-off-by: Dhiraj Lulla <dhiraj.lulla@intel.com>

Copilot AI left a comment

Copy link
Copy Markdown

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 no new comments.

@mchromin
mchromin merged commit 15772b9 into intel:main May 15, 2026
31 of 32 checks passed
@dhiraj3880
dhiraj3880 deleted the filter_mac_address branch May 15, 2026 20:55
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.

4 participants