Skip to content

[Accton][wedge800bact] Add PLATFORM_WEDGE800BACT to ProdConfigFactory#1024

Open
brandonchen-accton wants to merge 1 commit intofacebook:mainfrom
brandonchen-accton:fix-fsw-strict-priority-test
Open

[Accton][wedge800bact] Add PLATFORM_WEDGE800BACT to ProdConfigFactory#1024
brandonchen-accton wants to merge 1 commit intofacebook:mainfrom
brandonchen-accton:fix-fsw-strict-priority-test

Conversation

@brandonchen-accton
Copy link
Copy Markdown

@brandonchen-accton brandonchen-accton commented Mar 23, 2026

Summary

PLATFORM_WEDGE800BACT (platformType 41) was missing from the test utility functions in ProdConfigFactory.cpp. This caused HwProdInvariantsFswStrictPriorityTest.verifyInvariants to throw an exception in the sai_test-sai_impl with: "provided PlatformType: 41 has not been defined for uplinksCountFromSwitch"

PLATFORM_WEDGE800BACT has 16 uplinks when operating at 800G, so we added the corresponding definition to fix the test failure.

Pre-submission checklist

  • I've ran the linters locally and fixed lint errors related to the files I modified in this PR. You can install the linters by running pip install -r requirements-dev.txt && pre-commit install
  • pre-commit run
clang-format.............................................................Passed
shellcheck...........................................(no files to check)Skipped
shfmt................................................(no files to check)Skipped
trim trailing whitespace.................................................Passed
fix end of files.........................................................Passed
check yaml...........................................(no files to check)Skipped
check json...........................................(no files to check)Skipped
check for merge conflicts................................................Passed
ruff check...........................................(no files to check)Skipped
ruff format..........................................(no files to check)Skipped

Test Plan

[       OK ] HwProdInvariantsFswStrictPriorityTest.verifyInvariants (42435 ms)
[----------] 1 test from HwProdInvariantsFswStrictPriorityTest (42435 ms total)

[----------] Global test environment tear-down
[==========] 1 test from 1 test suite ran. (42435 ms total)
[  PASSED  ] 1 test.

Summary:
PLATFORM_WEDGE800BACT (platformType 41) was missing from the test utility functions in ProdConfigFactory.cpp.
This caused HwProdInvariantsFswStrictPriorityTest.verifyInvariants to throw an exception in the sai_test-sai_impl with:
"provided PlatformType: 41 has not been defined for uplinksCountFromSwitch"

PLATFORM_WEDGE800BACT has 16 uplinks when operating at 800G, so we added the corresponding definition to fix the test failure.
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