Skip to content

Change sensorshow conn to use TCP socket#290

Closed
cyw233 wants to merge 1 commit into
Azure:202405from
cyw233:cherry/202405/4343
Closed

Change sensorshow conn to use TCP socket#290
cyw233 wants to merge 1 commit into
Azure:202405from
cyw233:cherry/202405/4343

Conversation

@cyw233

@cyw233 cyw233 commented Mar 10, 2026

Copy link
Copy Markdown

What I did

Change the DB connection in scripts/sensorshow to use TCP socket.

How I did it

How to verify it

Run the commands that invoke sensorshow (e.g. show platform voltage) without write permission to see if it's returning the correct output instead of the RuntimeError: Unable to connect to redis (unix-socket) - Permission denied(1): Cannot assign requested address error

Previous command output (if the output of a command-line utility has changed)

RuntimeError: Unable to connect to redis (unix-socket) - Permission denied(1): Cannot assign requested address

New command output (if the output of a command-line utility has changed)

user@SONiC-DUT-sup00:~$ show platform voltage  
         Sensor    Voltage    High TH    Low TH    Crit High TH    Crit Low TH    Warning          Timestamp
---------------  ---------  ---------  --------  --------------  -------------  ---------  -----------------
FC0_HOTSWAP_VIN   55275 mV      58860     49140           59400          48600      False  20260310 03:04:31
FC2_HOTSWAP_VIN   55350 mV      58860     49140           59400          48600      False  20260310 03:04:31
FC4_HOTSWAP_VIN   55300 mV      58860     49140           59400          48600      False  20260310 03:04:31
FC5_HOTSWAP_VIN   55275 mV      58860     49140           59400          48600      False  20260310 03:04:31
FC6_HOTSWAP_VIN   55250 mV      58860     49140           59400          48600      False  20260310 03:04:31
     MB_VP54P0V   55350 mV      59343     48806           59913          48298      False  20260310 03:04:31

Signed-off-by: Chenyang Wang <chenyangw233@gmail.com>
@cyw233 cyw233 changed the title Change sensorshow conn to use TCP socket [manual] Change sensorshow conn to use TCP socket Mar 10, 2026
@cyw233 cyw233 changed the title [manual] Change sensorshow conn to use TCP socket Change sensorshow conn to use TCP socket Mar 10, 2026
@cyw233

cyw233 commented Mar 10, 2026

Copy link
Copy Markdown
Author

Original PR: sonic-net/sonic-utilities#4343

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.

1 participant