-
Notifications
You must be signed in to change notification settings - Fork 376
BLACKWOLF800BANW: platform_manager support #851
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
0da4acc to
8d86d3a
Compare
8d86d3a to
9cfb775
Compare
|
@RomanChoporov-Meta has imported this pull request. If you are a Meta employee, you can view this in D91694570. |
|
Hey @dzarista, |
@arajeev-ARISTA will make a PR for this shortly. Thanks. |
This is part of a series of PRs to add support for BLACKWOLF800BANW. This PR adds: - Platform type definition (fboss_common.thrift, PlatformMode.h, PlatformProductInfo.cpp) - Platform mapping (Blackwolf800banwPlatformMapping) - SAI platform integration (SaiBcmBlackwolf800banwPlatform) Other PRs in this series: facebook#876 - Q4D ASIC definitions (dependency) facebook#851 - platform_manager support facebook#852 - sensor_service configs facebook#853 - fan_service configs Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
This is part of a series of PRs to add support for BLACKWOLF800BANW. This PR adds: - Platform type definition (fboss_common.thrift, PlatformMode.h, PlatformProductInfo.cpp) - Platform mapping (Blackwolf800banwPlatformMapping) - SAI platform integration (SaiBcmBlackwolf800banwPlatform) Other PRs in this series: facebook#876 - Q4D ASIC definitions (dependency) facebook#851 - platform_manager support facebook#852 - sensor_service configs facebook#853 - fan_service configs
Pre-submission checklist
pip install -r requirements-dev.txt && pre-commit installpre-commit runSummary
This is the seventh PR in a series of PRs to add initial support for BLACKWOLF800BANW.
This PR adds platform_manager support for BLACKWOLF800BANW. This includes:
Additionally, the config assumes PR #774 is landed, where CHASSIS/SMB idprom share the same physical idprom but a different offset.
PRs in this series, ordered by landing priority:
Landing Priority: This PR should be landed seventh.
Test Plan
Build passes when stacked on top of PRs 1-6 and #774.
Testing was also performed in an environment with all the PRs in this series combined, with the following results:
All sw_tests pass
platform_manager_hw_test pass:
sensor_service_hw_test pass:
fan_service_hw_test pass:
data_corral_service_hw_test pass:
weutil_hw_test pass: