Skip to content
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

montblanc: config: Update sensor config based on the latest sensors list #272

Closed

Conversation

clslucas
Copy link
Contributor

Description
Updated sensors config files based on the latest sensors list.

Motivation

  1. Sync up sensor config from fboss repo.
  2. Updated sensor config file based on the latest sensors list.

Test Plan

  • Run the jq to format json file.
  • build the latest fboss bsp kmod.
  • build the latest fboss platform manager binary.
  • Run the platform manager with the changed config files on DVT and 2nd source DVT device normally: LD_LIBRARY_PATH=./lib/ ./bin/platform_manager --config_file ./platform_manager.json --run_once=false --noenable_pkg_mgmnt
  • Run sensor_service with the new config file: LD_LIBRARY_PATH=./lib/ ./bin/sensor_service --config_file ./sensor_service.json --logging=DBG1

Current tested the new config file on two tpye board normal.
montblanc_sensor_config_IFX_test_log_20241015.txt
montblanc_sensor_config_MP_test_log_20241015.txt

Limitation:
Due to the firmware problem of the PSU, the standby voltage cannot be found, so it is necessary to upgrade to the latest firmware.
image

@facebook-github-bot
Copy link
Contributor

@kimdo8736 has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@facebook-github-bot
Copy link
Contributor

@kimdo8736 merged this pull request in d94980d.

@clslucas clslucas deleted the montblanc_update_sensors_config branch October 31, 2024 06:02
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.

3 participants