Skip to content

feat: Add support for multihost configuration - #13

Merged
mchromin merged 1 commit into
intel:mainfrom
dhiraj3880:psm_mh
Nov 4, 2026
Merged

mchromin merged 1 commit into
intel:mainfrom
dhiraj3880:psm_mh

Conversation

@dhiraj3880

Copy link
Copy Markdown
Contributor

PSM commands were all running on host 0 by default. Add support so we can run the commands on xhc1, xhc2 etc.

Copilot AI balanced review requested due to automatic review settings November 3, 2026 17:27

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.

🟢 Approval recommended

No active review issues are reported for the PSM host-selection changes, which retain host 0 as the default.

No open issues 🎉

What changed in this PR

Adds host selection to PSM configuration methods so callers can target hosts beyond host 0, while preserving host 0 as the default.

Changes:

  • Adds and forwards host_id for VM creation, VF-to-VM mapping, and VM rate limits.
  • Includes host information in logs and errors, and updates API documentation.
  • Extends unit tests to verify non-default host selection and default-host forwarding.
File Description
tests/​unit/​test_mfd_cli_client/​test_base.py Verifies host forwarding and generated commands for non-default hosts.
README.md Documents host parameters and corrects the rate-limit method description.
mfd_cli_client/​base.py Adds host selection to PSM methods, commands, logs, and errors.

Review effort: Balanced

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.

🟢 Approval recommended

There are no active review issues for the host-selection changes.

No open issues 🎉

Review effort: Balanced

@mchromin
mchromin merged commit b3e907f into intel:main Nov 4, 2026
23 checks passed
PSM commands were all running on host 0 by default.
Add support so we can run the commands on xhc1, xhc2 etc.

Signed-off-by: Dhiraj Lulla <dhiraj.lulla@intel.com>
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