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.

Sign up for GitHub

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

Jump to bottom

[platform-monitor]: Un-daemonize lm-sensors #842

Closed
jleveque opened this issue Jul 25, 2017 · 2 comments
Closed

[platform-monitor]: Un-daemonize lm-sensors #842

jleveque opened this issue Jul 25, 2017 · 2 comments
Assignees
@jleveque
Labels
Enhancement ➕

Comments

@jleveque
Copy link
Contributor

lm-sensors is daemonized and run as a service. When Supervisor starts lm-sensors, it thinks it exits immediately, so it retries a few times before giving up. This is harmless, as the service starts the first time.

However, there are erroneous log messages in syslog as follows:

INFO spawned: 'lm-sensors' with pid 20
INFO exited: lm-sensors (exit status 0; not expected)
INFO spawned: 'lm-sensors' with pid 29
INFO exited: lm-sensors (exit status 0; not expected)
INFO spawned: 'lm-sensors' with pid 44
INFO exited: lm-sensors (exit status 0; not expected)
INFO spawned: 'lm-sensors' with pid 58
INFO exited: lm-sensors (exit status 0; not expected)
INFO gave up: lm-sensors entered FATAL state, too many start retries too quickly

Also. supervisor cannot watch it properly, and thus cannot react if lm-sensors ever terminates.

Need to find a way to run lm-sensors in the foreground.
@jleveque jleveque self-assigned this Jul 25, 2017
@jleveque
Copy link
Contributor Author

jleveque commented Jul 3, 2019

Once we transition to the new SONiC platform API, we will deprecate the use of lm-sensors. All sensor data will be retrieved using the new platform API.

@jleveque
Copy link
Contributor Author

Closing, as the new platform API will eventually remove the need for lm-sensors in SONiC.

yxieca added a commit to yxieca/sonic-buildimage that referenced this issue Apr 28, 2020
@yxieca
[201811][utilities] advance utilities submodule head
d5dbfe8 
Submodule src/sonic-utilities d7e8f84cf..8c21fc151:
  > [utility] Filter FDB entries (sonic-net#890)
  > Fix the warm-reboot script to support FRR based warm-reboot (sonic-net#842)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
@yxieca yxieca mentioned this issue Apr 28, 2020
Merged
yxieca added a commit that referenced this issue Apr 29, 2020
@yxieca
[201811][utilities] advance utilities submodule head (#4490)
e2ae4ff 
Submodule src/sonic-utilities d7e8f84cf..8c21fc151:
  > [utility] Filter FDB entries (#890)
  > Fix the warm-reboot script to support FRR based warm-reboot (#842)

Signed-off-by: Ying Xie <ying.xie@microsoft.com>
stepanblyschak added a commit to stepanblyschak/sonic-buildimage that referenced this issue Jun 30, 2021
@stepanblyschak
[sonic-sairedis] udpate submodule
bdcac11 
Changes:

```
3c485e5 [recorder] Fix incorrect attribute enum value capability query (sonic-net#843)
677ebca [sairedis] Client/Server support zmq configuration file (sonic-net#845)
7c70e34 [sairedis] Add support for bulk api in client/server (sonic-net#844)
76d28a6 [pyext] Use SAI autogenerated saiswig.i (sonic-net#837)
9949c48 [vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (sonic-net#842)
e385212 [MPLS] Minor tweaks to VS for MPLS support for CRM polling of MPLS In-segments and NHs.
d819f97 [meta] Add support for ignored attributes names (sonic-net#836)
c163238 Add cisco-8000 checks to syncd_init_common (sonic-net#839)
9aed2ff [sairedis] Add support for client server architecture (sonic-net#838)
```

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
liat-grozovik pushed a commit that referenced this issue Jul 7, 2021
@stepanblyschak
[sonic-sairedis] udpate submodule (#8043)
82b3a39 
Changes:

3c485e5 [recorder] Fix incorrect attribute enum value capability query (#843)
677ebca [sairedis] Client/Server support zmq configuration file (#845)
7c70e34 [sairedis] Add support for bulk api in client/server (#844)
76d28a6 [pyext] Use SAI autogenerated saiswig.i (#837)
9949c48 [vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (#842)
e385212 [MPLS] Minor tweaks to VS for MPLS support for CRM polling of MPLS In-segments and NHs.
d819f97 [meta] Add support for ignored attributes names (#836)
c163238 Add cisco-8000 checks to syncd_init_common (#839)
9aed2ff [sairedis] Add support for client server architecture (#838)

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
xumia pushed a commit that referenced this issue Jul 8, 2021
@stepanblyschak @xumia
[sonic-sairedis] udpate submodule (#8043)
f5c72fa 
Changes:

3c485e5 [recorder] Fix incorrect attribute enum value capability query (#843)
677ebca [sairedis] Client/Server support zmq configuration file (#845)
7c70e34 [sairedis] Add support for bulk api in client/server (#844)
76d28a6 [pyext] Use SAI autogenerated saiswig.i (#837)
9949c48 [vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (#842)
e385212 [MPLS] Minor tweaks to VS for MPLS support for CRM polling of MPLS In-segments and NHs.
d819f97 [meta] Add support for ignored attributes names (#836)
c163238 Add cisco-8000 checks to syncd_init_common (#839)
9aed2ff [sairedis] Add support for client server architecture (#838)

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
prsunny pushed a commit that referenced this issue Jul 30, 2021
@nazariig
[submodule]: Advance sonic-swss & sonic-sairedis. (#8296)
ada56ab 
sairedis:
*[recorder] Fix incorrect attribute enum value capability query (#843) d86b051
*[syncd] Fix fdb flood queue size limit check (#863) 3a2af76
*[vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (#842) 575dcb4 

swss:
*[portsorch] fix errors when moving port from one lag to anoth… a67d8af
*[debugcounterorch] check if counter type is supported before querying… ( 04105a4
*Td2: Reclaim buffer from unused ports (#1830) ac7f5cf
*[Dynamic Buffer Calc][202012]Bug fix: Don't create lossless buffer pr… f54b7d0 

Signed-off-by: Nazarii Hnydyn <nazariig@nvidia.com>
judyjoseph added a commit that referenced this issue Aug 7, 2021
@judyjoseph
9949c48 [vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (
546dab2 
#842)

e385212 [MPLS] Minor tweaks to VS for MPLS support for CRM polling of MPLS In-segments and NHs.
c163238 Add cisco-8000 checks to syncd_init_common (#839)
carl-nokia pushed a commit to carl-nokia/sonic-buildimage that referenced this issue Aug 7, 2021
@stepanblyschak
[sonic-sairedis] udpate submodule (sonic-net#8043)
e811439 
Changes:

3c485e5 [recorder] Fix incorrect attribute enum value capability query (sonic-net#843)
677ebca [sairedis] Client/Server support zmq configuration file (sonic-net#845)
7c70e34 [sairedis] Add support for bulk api in client/server (sonic-net#844)
76d28a6 [pyext] Use SAI autogenerated saiswig.i (sonic-net#837)
9949c48 [vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (sonic-net#842)
e385212 [MPLS] Minor tweaks to VS for MPLS support for CRM polling of MPLS In-segments and NHs.
d819f97 [meta] Add support for ignored attributes names (sonic-net#836)
c163238 Add cisco-8000 checks to syncd_init_common (sonic-net#839)
9aed2ff [sairedis] Add support for client server architecture (sonic-net#838)

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
AidanCopeland pushed a commit to Metaswitch/sonic-buildimage that referenced this issue Apr 14, 2022
@stepanblyschak
[vslib] implement query for SAI_DEBUG_COUNTER_TYPE enum values (sonic…
9949c48 
…-net#842)

Signed-off-by: Stepan Blyschak <stepanb@nvidia.com>
mssonicbld added a commit that referenced this issue Jan 10, 2024
@mssonicbld
[submodule] Update submodule sonic-swss-common to the latest HEAD aut…
2e27b11 
…omatically (#17410)

#### Why I did it
src/sonic-swss-common
```
* 16bc247 - (HEAD -> master, origin/master, origin/HEAD) [tests] fix binary_data_get unit test (#841) (72 minutes ago) [Yakiv Huryk]
* b2480ad - Add SonicDBConfig::reset method (#843) (4 weeks ago) [ganglv]
* ab3ce86 - [Azp]: Fix azp dash dependency (#842) (5 weeks ago) [Ze Gan]
* 5d1fe2d - add support for binary data read for Table::get() (#836) (5 weeks ago) [Yakiv Huryk]
```
#### How I did it
#### How to verify it
#### Description for the changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jleveque jleveque

Labels
Enhancement ➕
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@jleveque