Skip to content

[BMC] Add bmc-dual-mgmt testbed entry to testbed.yaml#23261

Merged
yxieca merged 1 commit intosonic-net:masterfrom
Blueve:dev/jika/bmc4
Mar 27, 2026
Merged

[BMC] Add bmc-dual-mgmt testbed entry to testbed.yaml#23261
yxieca merged 1 commit intosonic-net:masterfrom
Blueve:dev/jika/bmc4

Conversation

@Blueve
Copy link
Copy Markdown
Collaborator

@Blueve Blueve commented Mar 24, 2026

Description of PR

Summary:
Add example bmc-dual-mgmt-01 testbed entry to testbed.yaml with the new bmc_host field for BMC-to-host CPU mapping.

This is part of the SONiC BMC test framework as described in the BMC High-Level Test Plan, Section 3.1.

Type of change

  • Testbed and Framework(new/improvement)

Back port request

Approach

What is the motivation for this PR?

BMC testbeds need a testbed.yaml entry that defines the BMC as DUT and maps it to the associated host (CPU) device via the bmc_host field. This field is consumed by get_bmc_host() to create a SonicHost instance for the host side during cross-device BMC tests (e.g., power cycle verification).

How did you do it?

Added a bmc-dual-mgmt-01 entry to ansible/testbed.yaml with:

  • topo: bmc-dual-mgmt — references the BMC dual-mgmt topology file
  • dut: [switch01-bmc] — the BMC device as DUT
  • bmc_host: switch01 — new field mapping BMC to its host CPU
  • vm_base: empty — no VMs needed for BMC topology
  • auto_recover: 'False' — BMC recovery requires special handling

How did you verify/test it?

  • Validated YAML parsing with yaml.safe_load() — all fields parse correctly
  • Verified bmc_host field is preserved in the parsed dict (it flows through to tbinfo automatically since _read_regular_testbed_topo_from_yaml preserves all fields)

Any platform specific information?

Applies to BMC testbeds using the bmc-dual-mgmt topology.

Supported testbed topology if it is a new test case?

N/A — this PR adds testbed configuration only.

Documentation

Add example bmc-dual-mgmt-01 testbed entry with the new bmc_host
field that maps the BMC DUT (switch01-bmc) to its associated host
CPU (switch01). This field is read by get_bmc_host() to create a
SonicHost instance for cross-device verification in BMC tests.

Co-authored-by: Copilot <[email protected]>
Signed-off-by: Jing Kan <[email protected]>
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@github-actions github-actions bot requested review from r12f, sdszhang and wangxin March 24, 2026 14:59
@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@Blueve Blueve marked this pull request as ready for review March 24, 2026 15:13
@Blueve
Copy link
Copy Markdown
Collaborator Author

Blueve commented Mar 25, 2026

/azpw run

@mssonicbld
Copy link
Copy Markdown
Collaborator

⚠️ Notice: /azpw run only runs failed jobs now. If you want to trigger a whole pipline run, please rebase your branch or close and reopen the PR.
💡 Tip: You can also use /azpw retry to retry failed jobs directly.

Retrying failed(or canceled) jobs...

@Blueve Blueve closed this Mar 25, 2026
@Blueve Blueve reopened this Mar 25, 2026
@mssonicbld
Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Copy Markdown
Collaborator

@yxieca yxieca left a comment

Choose a reason for hiding this comment

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

AI agent on behalf of Ying. Reviewed; no issues found.

@yxieca yxieca merged commit 9d2674b into sonic-net:master Mar 27, 2026
22 checks passed
selldinesh pushed a commit to selldinesh/sonic-mgmt that referenced this pull request Apr 1, 2026
Add example bmc-dual-mgmt-01 testbed entry with the new bmc_host
field that maps the BMC DUT (switch01-bmc) to its associated host
CPU (switch01). This field is read by get_bmc_host() to create a
SonicHost instance for cross-device verification in BMC tests.

Signed-off-by: Jing Kan <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: selldinesh <[email protected]>
albertovillarreal-keys pushed a commit to albertovillarreal-keys/sonic-mgmt that referenced this pull request Apr 3, 2026
Add example bmc-dual-mgmt-01 testbed entry with the new bmc_host
field that maps the BMC DUT (switch01-bmc) to its associated host
CPU (switch01). This field is read by get_bmc_host() to create a
SonicHost instance for cross-device verification in BMC tests.

Signed-off-by: Jing Kan <[email protected]>
Co-authored-by: Copilot <[email protected]>
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Apr 13, 2026
Add example bmc-dual-mgmt-01 testbed entry with the new bmc_host
field that maps the BMC DUT (switch01-bmc) to its associated host
CPU (switch01). This field is read by get_bmc_host() to create a
SonicHost instance for cross-device verification in BMC tests.

Signed-off-by: Jing Kan <[email protected]>
Co-authored-by: Copilot <[email protected]>
Signed-off-by: opcoder0 <[email protected]>
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