Skip to content

Conversation

@joywu-coder
Copy link
Contributor

@joywu-coder joywu-coder commented Dec 24, 2025

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
image

Summary

  1. One issue found when run the lastest fboss platform_manager on Icetea, icecube and tahansb800bc (non-IDPROM CHASSIS EEPROMs): Failed to report hardware versions. No versions available
image
  1. The issue was introduced by Commit [9ebd96b.]
  2. Improved the code to add field for hardware versions to data store from non-IDPROM CHASSIS EEPROMs.

Test Plan

  1. Platform services build passed.
  2. Run platform_manager, platform_manager_platform_explorer_test on the testbeds of icetea, icecube, janga800bic, tahan800bc, montblanc and check the results.
image image

icetea_platform_manager_log.txt
icetea_platform_manager_platform_explorer_test_log.txt
icecube_platform_manager_log.txt
janga_platform_manager_log.txt
montblanc_platform_manager_log.txt
tahan_platform_manager_log.txt

@meta-cla meta-cla bot added the CLA Signed label Dec 24, 2025
@joywu-coder joywu-coder marked this pull request as ready for review December 25, 2025 01:03
@meta-codesync
Copy link

meta-codesync bot commented Jan 5, 2026

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

@meta-codesync
Copy link

meta-codesync bot commented Jan 5, 2026

@binhuang00 merged this pull request in 1436276.

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.

2 participants