Skip to content

redis: use local sockets for Alpine clients - #56

Closed
ashutosh-agrawal wants to merge 1 commit into
sonic-net:masterfrom
ashutosh-agrawal:redis-local-clients-uds
Closed

ashutosh-agrawal wants to merge 1 commit into
sonic-net:masterfrom
ashutosh-agrawal:redis-local-clients-uds

Conversation

@ashutosh-agrawal

Copy link
Copy Markdown
Member

Why

The Alpine VS xcvrd and Python process-exit listener read local Redis databases through TCP even though the local Unix socket is available.

What changed

  • Use Unix sockets for xcvrd APPL_DB, APPL_STATE_DB, and STATE_DB.
  • Pass the existing Unix-socket option to the syncd Python process-exit listener.

Validation

  • git diff --check
  • This repository has no focused unit tests for these launch paths.
  • No tests were run.

Scope

This changes only local Alpine VS Redis clients. It does not change raw redis-cli, chassis, remote DPU, BMC, or Redfish behavior.

Signed-off-by: Ashutosh Agrawal <ashu@cisco.com>
@mssonicbld

Copy link
Copy Markdown

/azp run

@azure-pipelines

Copy link
Copy Markdown
No pipelines are associated with this pull request.

@ashutosh-agrawal

Copy link
Copy Markdown
Member Author

Closing this vendor or platform draft to keep the Redis migration effort within the 25-open-PR limit. The branch is retained and queued for a later platform batch.

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.

2 participants