[docker-platform-monitor]: Add fancontrol #735

jleveque · 2017-06-21T22:08:28Z

Resolves #705

lguohan · 2017-06-22T01:55:35Z

@NStetskovych, can you help to test this on your platform?

stcheng · 2017-06-22T02:11:33Z

dockers/docker-platform-monitor/supervisord.conf

@@ -27,9 +27,17 @@ autorestart=false
 stdout_logfile=syslog
 stderr_logfile=syslog

+[program:fancontrol]
+command=/usr/sbin/fancontrol


That's simply where it gets installed via the .deb package.

NStetskovych-zz · 2017-06-23T06:34:11Z

looks good, attached logs
fan_log.txt

jleveque · 2017-06-23T17:16:24Z

Thanks, @NStetskovych!

jleveque · 2017-06-23T17:16:27Z

Retest this please.

* [build_debian]: Add dbus package to update timezone (#702) Signed-off-by: Hiayang Zheng haiyang.z@alibaba.inc * [Accton]: Add a new supported device and platform, AS7712-32X (#662) * platform/broadcom: Add a new supported device and platform, AS7712-32X * Switch Vendor: Edge-core * Switch SKU: AS7712-32X * ASIC Vendor: Broadcom * Swich ASIC: Tomahawk * Port Configuration: 32x100G * SONiC Image: SONiC-ONIE-Broadcom Signed-off-by: polly_hsu@accton.com * [image]: load platform specific settings in sonic-to-sonic upgrade (#710) * [submodule]: Update sairedis submodule (#712) Include fix for saithrift port split parsing * [docker-ptf]: Install exabgp in docker ptf (#709) * [platform]: fix file mode for acction platform to create clean build (#714) * [image]: expand dockerfs.tar.gz directly on to disk without intermediate file (#715) * [submodule]: update swss/sairedis/swss-common submodule (#716) fix bug for logrotate sairedis log * [submodule]: update sonic-swss (#719) * [image]: add debian security update in the apt source list (#724) * [submodule]: update sonic-platform-daemons (#722) * [platform]: turn on/off four leds for breakout-able front panel ports (#723) turn on/off four leds for breakout-able front panel ports on arista 7050 qx32 platform * [sonic-quagga]:update submodule (#718) Signed-off-by: Sihui Han <sihan@microsoft.com> * Add Broadcom LED microprocessor initialization for Dell S6000 (#726) * Add Broadcom LED microprocessor initialization for Dell S6000 * Increase bcmcmd timeout to 60 seconds * [platform]: Update sonic-platform-modules-arista (#727) * [Broadcom]: Update SAI package to support Accton-AS7712-C32 (#720) * Remove exsiting sonic-platform-modules-accton and apply submodule add… (#717) * Remove exsiting sonic-platform-modules-accton and apply submodule adding on this folder. * Remove redundant commnet * Remove folder platform/broadcom/sonic-platform-modules-accton. * Add this dir for submodule. * [submodule]: update sonic-platform-daemons submodule (#731) * [installer] Copy old config files rather than only minigraph (#730) * [BGPD]: add bgp dynamic neighbor configuration (#708) * add bgp dynamic neighbor configuration * [bgpd]: update as comments * update as comment * update to deployment_id_asn_map * minor change * [docker-syncd-brcm]: Sleep to allow syncd to create socket before calling bcmcmd (#733) * [docker-syncd-brcm]: Sleep to allow syncd to create socket before calling bcmcmd * Instead of fixed sleep interval, continually check for presence of socket * [sonic-cfggen]: Support multiple input yaml files with -y option (#729) * [sonic-cfggen]: Fix minigraph.py when port_alias_map absent (#738) * [bgp]: move allowas-in into ipv6 section to enable allowas-in for ipv6 (#741) * [swss]: Sleep 5 min regardless of arp_update return code (#743) - arp_update return code is not guaranteed to be true/false. When there is no VLAN, arp_update will return true. When there are VLANs, arp_update will return false because the command arping returns 1 due to the option '-w 0'. - This script should be run every 5 minutes regardless of the return code. * [Mellanox]: Update SAI version (#740) - Initialize ECMP default hashing with IPv6 packets * [sonic-cfggen]: Add default IP deny rule in translate_acl (#734) - Add IPv4 implicit rule: deny ip any any This implicit rule has lowest priority and ensures that the device denies all unmatched IP traffic. * [docker-platform-monitor]: Add fancontrol (#735) * Fix typo in README (#750) * Fix typo in README * Fix comment * Fix README * [DHCP Relay]: Add support for custom Option 82 circuit_id of the form '<hostname>:<portname>' (#747) * Add docker-dhcp-relay/Dockerfile to .gitignore * Add isc-dhcp-relay .deb package to image build process, along with my Option 82 patch * Install custom isc-dhcp-relay in dhcp_relay docker * Install isc-dhcp-relay build dependencies in sonic-slave Docker container * Copy the built .deb package to the destination directory * Add dependencies for isc-dhcp-relay * Change Option 82 string to '<hostname>:<portname>' * Install dependencies of .deb files implicitly in Dockerfile * Remove unused line * Remove unnecessary space * [Accton]: Add a new supported device and platform AS7716-32X (#732) Switch Vendor: Edge-core Switch SKU: AS7716-32X ASIC Vendor: Broadcom Swich ASIC: Tomahawk Port Configuration: 32x100G SONiC Image: SONiC-ONIE-Broadcom Signed-off-by: polly_hsu@accton.com * [.gitignore]: Update src, config engine (#754) Signed-off-by: marian-pritsak <marianp@mellanox.com> * [submodule]: update sonic-swss (#755) * [sonic-swss-common]: Submodule update (#759) * [mgmt slave] Install correct version of python-cffi (#760) * [DHCP Relay]: Fix Option 82 string - Remove quotes; add MAC address of receiving port as remote_id (#763) * [platform]: add front panel index in port_config.ini for s6100 (#752) * add front panel index in port_config.ini * [igb]: change download url to azure storage url (#770) * [baseimage]: Increase net.core.rmem_max to 2097152 (#767) * Increase net.core.rmem_max to 2097152 * Update Ingrasys platform submodule (#764) * Remove specific fancontrol service. Signed-off-by: Wade He <chihen.he@gmail.com> * [Broadcom]: Update SAI package to support Celestica Seastone DX010 and Accton AS7716 (#768) * [LLDP]: Port description (TLV 4) now contains '<neighbor_host_name>:<port_name>' (#772) * [sonic-slave]: install docker-ce 17.03.02 stable version (#774) * [device]: add minigraph and LED support for Arista 7050 QX32S (#773) * Update submodules: sairedis, swss

[meta] Enable strict cast-align warning (sonic-net#738) [vs] Use meta class instead info when using unittests (sonic-net#740) [vs] Support flush entry type all on virtual switch (sonic-net#735) [vslib]: Add MACsec state to state base (sonic-net#722) [README.md] Update installation steps (sonic-net#730) Switch Capability support (sonic-net#728) [vs] Fail switch create when warm boot requested and no warm boot state (sonic-net#739) Dynamic Port breakout fix the crash, port down event processing after<E2><80><A6> (sonic-net#727) Code clean (sonic-net#721) Signed-off-by: Sabareesh Kumar Anandan <sanandan@marvell.com>

[vs] Add workaround for clean up macsec ports (#752) [logfile]: Add handling of Sairedis rec filename (#747) Update README.md [meta] Fix stat_mode enums to sai_bulk_op_error_mode_t (#753) [syncd][tests] Add syncd deprecated attribute value test (#751) [vs] Skip MACsec clean up if /sbin/ip is not accessible (#750) Configure enable -Wcast-align=strict when supported by compiler (#749) [syncd] Translate depreacated attr enum values to new ones (#746) [sairedis]vs SAI support for voq neighbor (#725) [syncd] Translate removed RIDs in fdb notification (#734) [syncd] Move syncd classes to syncd namespace (#742) [vs] Use /sbin/ip absolute path for ip command in MACsecManager (#744) [saidiscovery] Update saidiscovery to use VendorSai object and metadata (#736) Remove Winline warning since it depends on external headers (#741) [meta] Enable strict cast-align warning (#738) [vs] Use meta class instead info when using unittests (#740) [vs] Support flush entry type all on virtual switch (#735) [vslib]: Add MACsec state to state base (#722) [README.md] Update installation steps (#730) Switch Capability support (#728) [vs] Fail switch create when warm boot requested and no warm boot state (#739) Dynamic Port breakout fix the crash, port down event processing after<80> (#727) Code clean (#721) Signed-off-by: Sabareesh Kumar Anandan <sanandan@marvell.com>

Signed-off-by: kcudnik <kcudnik@gmail.com>

Update sonic-swss-common submodule pointer to include the following: a4987b9 Change the dtor of ProducerStateTable to virtual method (#735) 7be565c [hash]: Add GH DB schema. (#733) Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>

[docker-platform-monitor]: Add fancontrol

9f0a1d0

jleveque added the Enhancement ➕ label Jun 21, 2017

jleveque self-assigned this Jun 21, 2017

jleveque requested a review from lguohan June 21, 2017 22:08

msftclas added the cla-not-required label Jun 21, 2017

jleveque mentioned this pull request Jun 21, 2017

Remove fancontrol installation / startup Ingrasys-sonic/sonic-platform-modules-ingrasys#1

Closed

stcheng reviewed Jun 22, 2017

View reviewed changes

stcheng approved these changes Jun 22, 2017

View reviewed changes

lguohan approved these changes Jun 22, 2017

View reviewed changes

stcheng merged commit 22819d9 into sonic-net:master Jun 23, 2017

jleveque deleted the add_fancontrol branch June 23, 2017 22:27

Sabareesh-Kumar-Anandan mentioned this pull request Dec 14, 2020

[sonic-sairedis] submodule update #6202

Merged

3 tasks

AidanCopeland pushed a commit to Metaswitch/sonic-buildimage that referenced this pull request Apr 14, 2022

[vs] Support flush entry type all on virtual switch (sonic-net#735)

a1c9b7a

Signed-off-by: kcudnik <kcudnik@gmail.com>

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[docker-platform-monitor]: Add fancontrol #735

[docker-platform-monitor]: Add fancontrol #735

jleveque commented Jun 21, 2017 •

edited

Loading

lguohan commented Jun 22, 2017

stcheng Jun 22, 2017

jleveque Jun 22, 2017

NStetskovych-zz commented Jun 23, 2017 •

edited

Loading

jleveque commented Jun 23, 2017

jleveque commented Jun 23, 2017

[docker-platform-monitor]: Add fancontrol #735

[docker-platform-monitor]: Add fancontrol #735

Conversation

jleveque commented Jun 21, 2017 • edited Loading

lguohan commented Jun 22, 2017

stcheng Jun 22, 2017

Choose a reason for hiding this comment

jleveque Jun 22, 2017

Choose a reason for hiding this comment

NStetskovych-zz commented Jun 23, 2017 • edited Loading

jleveque commented Jun 23, 2017

jleveque commented Jun 23, 2017

jleveque commented Jun 21, 2017 •

edited

Loading

NStetskovych-zz commented Jun 23, 2017 •

edited

Loading