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: Added RUNBMC_THERMAL_SENSOR in sensor config #343

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

karthik-gengan
Copy link

Description:-

The RUNBMC_THERMAL_SENSOR entry, which was previously absent, has been added to the sensor configuration

Motivation:-

The RUNBMC_THERMAL_SENSOR entry in the sensor service configuration was missing the sensor_config file specification.
This has been corrected, ensuring the sensor service now monitors the BMC card's thermal value

Testcase:-

1.Run the jq to format json file.
sensor_service.txt
2.Run the latest platform_manager
3.Validated the Sensor_service
sensor_service_ut.txt

2946 SensorServiceImpl.cpp:95] Processing MINIPACK3_BMC PMUnit: 1 sensors 2946 SensorServiceImpl.cpp:186] RUNBMC_THERMAL_SENSOR (/run/devmap/sensors/RUNBMC_THERMAL_SENSOR/temp1_input) : 18.5

Description:-
-----------
The RUNBMC_THERMAL_SENSOR entry, which was previously absent, has been
added to the sensor configuration

Motivation:-
-----------
The RUNBMC_THERMAL_SENSOR entry in the sensor service configuration was
missing the sensor_config file specification. This has been corrected,
ensuring the sensor service now monitors the BMC card's thermal value

Testcase:-
---------
1.Run the jq to format json file.
2.Run the latest platform_manager
3.Validated the Sensor_service
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