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

Skip thermalctld for DellEMC S5248F platform #7022

Merged

Conversation

vpsubramaniam
Copy link
Contributor

Why I did it

To skip thermalctld in DellEMC S5248F platform since it requires platform API.

How I did it

Add 'skip_thermalctld:true' in pmon_daemon_control.json for DellEMC S5248F platform.

How to verify it

Verified thermalctld not running in PMON.

admin@S5248F-10860:~$ docker exec pmon ps -eaf
UID        PID  PPID  C STIME TTY          TIME CMD
root         1     0  0 Mar07 pts/0    00:01:48 /usr/bin/python3 /usr/local/bin/supervisord
root        23     1  0 Mar07 pts/0    00:00:21 python3 /usr/bin/supervisor-proc-exit-listener --container-name pmon
root        26     1  0 Mar07 pts/0    00:00:26 /usr/sbin/rsyslogd -n -iNONE
root        30     1  0 Mar07 pts/0    00:15:26 python2 /usr/local/bin/xcvrd
root        31     1  0 Mar07 pts/0    00:07:12 python2 /usr/local/bin/psud
root        32     1  0 Mar07 pts/0    00:03:04 python2 /usr/local/bin/syseepromd
root        40    30  0 Mar07 pts/0    00:32:14 python2 /usr/local/bin/xcvrd
root      7311    31  0 19:10 pts/0    00:00:00 sh -c { ipmitool sdr list; } 2>&1
root      7312  7311  0 19:10 pts/0    00:00:00 ipmitool sdr list
root      7313     0  0 19:10 ?        00:00:00 ps -eaf
admin@S5248F-10860:~$

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

  • 201811
  • 201911
  • 202006
  • 202012

Description for the changelog

Skip thermalctld in DellEMC S5248F platform

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

@jleveque jleveque merged commit 4aa3c6f into sonic-net:master Mar 12, 2021
raphaelt-nvidia pushed a commit to raphaelt-nvidia/sonic-buildimage that referenced this pull request May 23, 2021
Skip thermalctld in DellEMC S5248F platform since it requires platform API support

Co-authored-by: V P Subramaniam <Subramaniam_Vellalap@dell.com>
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this pull request Aug 7, 2021
Skip thermalctld in DellEMC S5248F platform since it requires platform API support

Co-authored-by: V P Subramaniam <Subramaniam_Vellalap@dell.com>
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