search

sonic-net / sonic-platform-daemons

Platform module daemons for SONiC
Other
25 stars 159 forks source link

Refactor ledd daemon and fix high CPU usage due to unexpected socket … #548

Open prgeor opened 1 month ago

prgeor commented 1 month ago

…close

Description

[ledd] Refactor ledd daemon and fix high CPU usage due to unexpected redis DB socket close

Motivation and Context

For unkown reason if redis closes the socket, then ledd daemon is continuously busy waiting for socket event and failing all the time. This is resulting in high CPI

How Has This Been Tested?

Close the socket abruptly and see ledd daemon is respawned so that new socket connection is established to redis DB without causing high CPU

Additional Information (Optional)

prgeor commented 1 month ago

@Staphylo could you please help review?

prgeor commented 1 month ago

@assrinivasan @vvolam please review

vvolam commented 1 month ago

@prgeor nit: spelling issue in PR description.

image

Also, could you fix the build failure?