Skip to content

[action] [PR:26198] Add bmc-shared-mgmt topology#1320

Merged
mssonicbld merged 1 commit into
Azure:202608from
mssonicbld:cherry/msft-202608/26198
Jul 17, 2026
Merged

[action] [PR:26198] Add bmc-shared-mgmt topology#1320
mssonicbld merged 1 commit into
Azure:202608from
mssonicbld:cherry/msft-202608/26198

Conversation

@mssonicbld

Copy link
Copy Markdown
Collaborator

Description of PR

Adds a bmc-shared-mgmt topology var file for BMC devices that only has one mgmt interface.

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • New Test case
    • Skipped for non-supported platforms
  • Test case improvement

Back port request

  • 202205
  • 202305
  • 202311
  • 202405
  • 202411
  • 202505
  • 202511

Approach

What is the motivation for this PR?

We need a topology definition for BMC devices that use a shared management network.

How did you do it?

Added ansible/vars/topo_bmc-shared-mgmt.yml, following the existing bmc-dual-mgmt topology.

How did you verify/test it?

No testbed yet. Adding topo definition first.

Any platform specific information?

N/A

Supported testbed topology if it's a new test case?

N/A

Documentation

No doc change needed.

Signed-off-by: Sonic Build Admin sonicbld@microsoft.com

### Description of PR

Adds a `bmc-shared-mgmt` topology var file for BMC devices that only has one mgmt interface.

### Type of change

- [ ] Bug fix
- [x] Testbed and Framework(new/improvement)
- [ ] New Test case
    - [ ] Skipped for non-supported platforms
- [ ] Test case improvement

### Back port request
- [ ] 202205
- [ ] 202305
- [ ] 202311
- [ ] 202405
- [ ] 202411
- [ ] 202505
- [ ] 202511

### Approach
#### What is the motivation for this PR?
We need a topology definition for BMC devices that use a shared management network.

#### How did you do it?
Added `ansible/vars/topo_bmc-shared-mgmt.yml`, following the existing `bmc-dual-mgmt` topology.

#### How did you verify/test it?
No testbed yet. Adding topo definition first.

#### Any platform specific information?
N/A

#### Supported testbed topology if it's a new test case?
N/A

### Documentation
No doc change needed.

Signed-off-by: Sonic Build Admin <sonicbld@microsoft.com>
@mssonicbld

Copy link
Copy Markdown
Collaborator Author

Original PR: sonic-net/sonic-mgmt#26198

@mssonicbld

Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.

@mssonicbld
mssonicbld merged commit 2ea5746 into Azure:202608 Jul 17, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant