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

[PDDF] Rename temp for as7816/7326/7726 #10609

Merged
merged 2 commits into from
May 9, 2022

Conversation

jostar-yang
Copy link
Contributor

Signed-off-by: Jostar Yang jostar_yang@accton.com.tw

Why I did it

Renaming temp for cmd sensors and show platform temp. To let user is easy to know temp location from naming.

How I did it

  1. Modify name in sensors.conf
  2. Modify or add display_name in pddf-device.json

How to verify it

Test cmd, sensors, and show platform temp.
For as7816,
root@sonic:/home/admin# show platform temperature
Sensor Temperature High TH Low TH Crit High TH Crit Low TH Warning Timestamp


  CpuBoard_temp(0x4B)           21.5       80.0       N/A             N/A            N/A      False  20220321 02:51:19
      FB_1_temp(0x4D)           22.5       80.0       N/A             N/A            N/A      False  20220321 02:51:19
      FB_2_temp(0x4E)           22.5       80.0       N/A             N/A            N/A      False  20220321 02:51:19
    MB_MAC_temp(0x48)           25         80.0       N/A             N/A            N/A      False  20220321 02:51:19

MB_RearCenter_temp(0x49) 29.5 80.0 N/A N/A N/A False 20220321 02:51:19
MB_RightCenter_temp(0x4A) 20.5 80.0 N/A N/A N/A False 20220321 02:51:19
PSU1_TEMP1 15 N/A N/A N/A N/A False 20220321 02:51:19
PSU2_TEMP1 0 N/A N/A N/A

root@sonic:/home/admin# sensors
lm75-i2c-18-48
Adapter: i2c-1-mux (chan_id 1)
MB_MAC_temp: +25.0 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-17-4e
Adapter: i2c-1-mux (chan_id 0)
FB_2_temp: +22.0 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-18-4a
Adapter: i2c-1-mux (chan_id 1)
MB_RightCenter_temp: +20.5 C (high = +80.0 C, hyst = +75.0 C)

coretemp-isa-0000
Adapter: ISA adapter
Package id 0: +42.0 C (high = +82.0 C, crit = +104.0 C)
Core 0: +42.0 C (high = +82.0 C, crit = +104.0 C)
Core 1: +42.0 C (high = +82.0 C, crit = +104.0 C)
Core 2: +42.0 C (high = +82.0 C, crit = +104.0 C)
Core 3: +42.0 C (high = +82.0 C, crit = +104.0 C)

lm75-i2c-18-49
Adapter: i2c-1-mux (chan_id 1)
MB_RearCenter_temp: +29.5 C (high = +80.0 C, hyst = +75.0 C)

fan_ctrl-i2c-17-68
Adapter: i2c-1-mux (chan_id 0)
fan1: 9897 RPM
fan2: 7791 RPM
fan3: 9637 RPM
fan4: 7961 RPM
fan5: 9897 RPM
fan6: 7791 RPM
fan7: 9637 RPM
fan8: 7961 RPM

lm75-i2c-18-4b
Adapter: i2c-1-mux (chan_id 1)
CpuBoard_temp: +21.5 C (high = +80.0 C, hyst = +75.0 C)

lm75-i2c-17-4d
Adapter: i2c-1-mux (chan_id 0)
FB_1_temp: +22.0 C (high = +80.0 C, hyst = +75.0 C)

psu_pmbus-i2c-10-5b
Adapter: i2c-1-mux (chan_id 1)
in3: +0.00 V
fan1: 0 RPM
temp1: +0.0 C
power2: 0.00 W
curr2: +0.00 A

psu_pmbus-i2c-9-58
Adapter: i2c-1-mux (chan_id 0)
in3: +12.00 V
fan1: 11600 RPM
temp1: +15.0 C
power2: 226.00 W
curr2: +18.88 A

Which release branch to backport (provide reason below if selected)

  • 201811
  • 201911
  • 202006
  • 202012
  • 202106
  • 202111

Description for the changelog

Link to config_db schema for YANG module changes

A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: Jostar Yang <jostar_yang@accton.com.tw>
FuzailBrcm
FuzailBrcm previously approved these changes Apr 19, 2022
@prgeor prgeor merged commit b86499c into sonic-net:master May 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants