Skip to content

[Celestica] Ladakh800bcls: Config: Support DVT#1213

Open
chee-zhang wants to merge 2 commits into
facebook:mainfrom
chee-zhang:ladakh800bcls_support_dvt
Open

[Celestica] Ladakh800bcls: Config: Support DVT#1213
chee-zhang wants to merge 2 commits into
facebook:mainfrom
chee-zhang:ladakh800bcls_support_dvt

Conversation

@chee-zhang
Copy link
Copy Markdown
Contributor

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.........................................(no files to check)Skipped
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...............................................................Passed
check for merge conflicts................................................Passed
ruff check...........................................(no files to check)Skipped
ruff format..........................................(no files to check)Skipped
Prevent sai_impl in fboss manifest.......................................Passed

Summary

To support DVT unit, need to update platform_manager.json and sensor_service.json.

  1. platform_manager.json: Fix error reading sensor of PMBUS_3 by setting page 0.
  2. sensor_service.json: Update dvt1 related sensor thresholds.

Test Plan

The platform_manager and sensor_service have been tested on following units.

logs_evt1a.zip

logs_evt1b.zip

logs_dvt1.zip

@chee-zhang chee-zhang requested a review from a team as a code owner May 20, 2026 05:31
@meta-cla meta-cla Bot added the CLA Signed label May 20, 2026
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