Skip to content

[202511][PR:23107] Enable link training support in sonic-mgmt#23331

Open
sdszhang wants to merge 1 commit intosonic-net:202511from
sdszhang:cherry/202511/23107
Open

[202511][PR:23107] Enable link training support in sonic-mgmt#23331
sdszhang wants to merge 1 commit intosonic-net:202511from
sdszhang:cherry/202511/23107

Conversation

@sdszhang
Copy link
Contributor

@sdszhang sdszhang commented Mar 26, 2026

Cherry-pick #23107 to 202511.

Description of PR

Summary:
Enable link-training support in sonic-mgmt

Currently, enabling them for all server facing ports in F2 T0 topo. Once we have port-channel support, we can enable them only for port-channel in F2 topo.

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?

Enable link-training support in sonic-mgmt

How did you do it?

Enabling link training in golden_config_db for F2 topology.

How did you verify/test it?

local testbed.

Any platform specific information?

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

Documentation

Signed-off-by: Dashuai Zhang <dashuaizhang@microsoft.com>

(cherry picked from commit 7512a71)
@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@sdszhang sdszhang changed the title Enable link training support in sonic-mgmt (#23107) [202511][PR:23107] Enable link training support in sonic-mgmt Mar 26, 2026
Copy link
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.

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