Skip to content

ICECUBE800BANW: bsp_tests update#904

Open
aalamsi22 wants to merge 1 commit intofacebook:mainfrom
aalamsi22:icecube800banw_bsp_tests
Open

ICECUBE800BANW: bsp_tests update#904
aalamsi22 wants to merge 1 commit intofacebook:mainfrom
aalamsi22:icecube800banw_bsp_tests

Conversation

@aalamsi22
Copy link
Contributor

@aalamsi22 aalamsi22 commented Feb 5, 2026

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

Summary

Update bsp_tests.json and platform_manager to point to the latest bsp.

PRs in this series, ordered by landing priority:

  1. ICECUBE800BANW: New platform definition & Platform Mapping #873
  2. ICECUBE800BANW: bsp mapping #871
  3. ICECUBE800BANW: qsfp_service support #870
  4. ICECUBE800BANW: led service support #875
  5. ICECUBE800BANW: vendor mapping and asic config #874
  6. ICECUBE800BANW: initial test_configs #877
  7. ICECUBE800BANW: Platform_manager support #845
  8. ICECUBE800BANW: sensor_service config #846
  9. ICECUBE800BANW: handle smb eeprom on cpu bus #892
  10. ICECUBE800BANW: bsp_tests update #904 (THIS PR)

Landing Priority: This PR should be landed tenth.

Test Plan

bsp_tests pass

#bsp_tests --config_file /opt/fboss/share/platform_configs/bsp_tests.json --platform_manager_config_file /opt/fboss/share/platform_configs/platform_manager.json -noenable_pkg_mgmnt
...
[----------] Global test environment tear-down
[==========] 31 tests from 10 test suites ran. (411089 ms total)
[  PASSED  ] 25 tests.
[  SKIPPED ] 6 tests, listed below:
[  SKIPPED ] KmodStressTest.KmodLoadUnloadStress
[  SKIPPED ] WatchdogTest.WatchdogStart
[  SKIPPED ] WatchdogTest.WatchdogPing
[  SKIPPED ] WatchdogTest.WatchdogSetTimeout
[  SKIPPED ] WatchdogTest.WatchdogMagicClose
[  SKIPPED ] WatchdogTest.WatchdogDriverUnload

platform_hw_test

# platform_hw_test -config-file /opt/fboss/share/platform_configs/platform_manager.json
[==========] Running 2 tests from 1 test suite.
[----------] Global test environment set-up.
[----------] 2 tests from PlatformHwTest
[ RUN      ] PlatformHwTest.CorrectMacX86
I0130 17:48:21.349339  3529 PlatformNameLib.cpp:56] Getting platform name from bios using dmidecode ...
I0130 17:48:21.351416  3529 PlatformNameLib.cpp:66] Platform name inferred from bios: ICECUBE800BANW
I0130 17:48:21.351424  3529 PlatformNameLib.cpp:68] Platform name mapped: ICECUBE800BANW
I0130 17:48:21.351717  3529 ConfigLib.cpp:50] Using config file: /opt/fboss/share/platform_configs/platform_manager.json
I0130 17:48:21.351956  3529 ConfigValidator.cpp:666] Validating platform_manager config
I0130 17:48:21.351993  3529 ConfigValidator.cpp:707] Validating SlotTypeConfig for Slot SMB_SLOT...
I0130 17:48:21.352001  3529 ConfigValidator.cpp:724] Validating PmUnitConfig for PmUnit SMB in Slot SMB_SLOT...
I0130 17:48:21.375403  3529 ConfigValidator.cpp:771] Validating Symbolic links...
I0130 17:48:25.561958  3529 ConfigValidator.cpp:778] Validating Transceiver symbolic links...
I0130 17:48:25.562065  3529 ConfigLib.cpp:50] Using config file: /opt/fboss/share/platform_configs/platform_manager.json
I0130 17:48:25.562348  3529 MacAddressCheck.cpp:99] eepromName: SMB x86CpuMac: b8:a1:b8:a5:4c:ee
[       OK ] PlatformHwTest.CorrectMacX86 (15050 ms)
[ RUN      ] PlatformHwTest.PCIDevicesPresent
I0130 17:48:36.399600  3529 PlatformNameLib.cpp:56] Getting platform name from bios using dmidecode ...
I0130 17:48:36.401355  3529 PlatformNameLib.cpp:66] Platform name inferred from bios: ICECUBE800BANW
I0130 17:48:36.401361  3529 PlatformNameLib.cpp:68] Platform name mapped: ICECUBE800BANW
I0130 17:48:36.401553  3529 ConfigLib.cpp:50] Using config file: /opt/fboss/share/platform_configs/platform_manager.json
I0130 17:48:36.401744  3529 ConfigValidator.cpp:666] Validating platform_manager config
I0130 17:48:36.401767  3529 ConfigValidator.cpp:707] Validating SlotTypeConfig for Slot SMB_SLOT...
I0130 17:48:36.401772  3529 ConfigValidator.cpp:724] Validating PmUnitConfig for PmUnit SMB in Slot SMB_SLOT...
I0130 17:48:36.424476  3529 ConfigValidator.cpp:771] Validating Symbolic links...
I0130 17:48:40.593261  3529 ConfigValidator.cpp:778] Validating Transceiver symbolic links...
I0130 17:48:40.593371  3529 ConfigLib.cpp:50] Using config file: /opt/fboss/share/platform_configs/platform_manager.json
[       OK ] PlatformHwTest.PCIDevicesPresent (4196 ms)
[----------] 2 tests from PlatformHwTest (19246 ms total)

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

platform_manager_hw_test

# platform_manager_hw_test --config-file /opt/fboss/share/platform_configs/platform_manager.json -noenable_pkg_mgmnt
...
[       OK ] PlatformManagerHwTest.XcvrLedFiles (28708 ms)
[----------] 8 tests from PlatformManagerHwTest (360987 ms total)

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

@meta-cla meta-cla bot added the CLA Signed label Feb 5, 2026
@aalamsi22 aalamsi22 marked this pull request as ready for review February 5, 2026 19:21
@meta-codesync
Copy link

meta-codesync bot commented Feb 10, 2026

@joancaneus has imported this pull request. If you are a Meta employee, you can view this in D92805149.

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