Skip to content

Support generating golden_config for topo ft2-64#18360

Merged
bingwang-ms merged 1 commit intosonic-net:masterfrom
bingwang-ms:golden_config_fec
May 13, 2025
Merged

Support generating golden_config for topo ft2-64#18360
bingwang-ms merged 1 commit intosonic-net:masterfrom
bingwang-ms:golden_config_fec

Conversation

@bingwang-ms
Copy link
Collaborator

Description of PR

Summary:
This PR is to update ansible/library/generate_golden_config_db.py to support generating golden_config for topo ft2-64 to enable FEC.
It has to be done because minigraph.py doesn't support generating AN config for Non-100Gb ports.
https://github.com/sonic-net/sonic-buildimage/blob/8fa62b5c03414940d1c74ba99e20472c4a4a243b/src/sonic-config-engine/minigraph.py#L2432

Type of change

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

Back port request

  • 202012
  • 202205
  • 202305
  • 202311
  • 202405
  • 202411

Approach

What is the motivation for this PR?

This PR is to update ansible/library/generate_golden_config_db.py to support generating golden_config for topo ft2-64 to enable FEC.

How did you do it?

Update generate_golden_config_db.py script.

How did you verify/test it?

The change is verified on a physical testbed.

Any platform specific information?

No.

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

Not a new test.

Documentation

@mssonicbld
Copy link
Collaborator

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@bingwang-ms
Copy link
Collaborator Author

@yxieca @auspham @vdahiya12 Could you help review?

@bingwang-ms bingwang-ms merged commit da270a3 into sonic-net:master May 13, 2025
19 checks passed
bingwang-ms added a commit to Azure/sonic-mgmt.msft that referenced this pull request May 13, 2025
opcoder0 pushed a commit to opcoder0/sonic-mgmt that referenced this pull request Dec 8, 2025
Signed-off-by: opcoder0 <110003254+opcoder0@users.noreply.github.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Dec 16, 2025
Signed-off-by: Aharon Malkin <amalkin@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Dec 21, 2025
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
gshemesh2 pushed a commit to gshemesh2/sonic-mgmt that referenced this pull request Jan 26, 2026
Signed-off-by: Guy Shemesh <gshemesh@nvidia.com>
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.

3 participants