Add unix domain socket for grpc communication#47
Open
spraveenio wants to merge 1 commit intoROCm:mainfrom
Open
Add unix domain socket for grpc communication#47spraveenio wants to merge 1 commit intoROCm:mainfrom
spraveenio wants to merge 1 commit intoROCm:mainfrom
Conversation
- to enable socket based grpc new argument with the socket path to use is introduced `-s, --grpc-unix-socket <path>`
- gpuctl updated to check for default socket file present to automatically detect socket for gracefully switching to socket based access or revert to tcp based, arg `node-svc-socket` added to set non default custom socket for grpc connection
```bash
root@33e2dab2b163:/# ps aux
USER PID %CPU %MEM VSZ RSS TTY STAT START TIME COMMAND
root 1 0.0 0.0 4364 3188 pts/0 Ss 20:37 0:00 /bin/bash -e /home/amd/tools/entrypoint.sh
root 8 0.0 0.0 4628 3812 pts/0 S+ 20:37 0:00 bash
root 48 0.4 0.0 942488 31092 ? Sl 20:46 0:00 /home/amd/bin/gpuagent -s /var/run/gpuagent.sock
root 72 0.0 0.0 4628 3932 pts/2 Ss 20:46 0:00 bash
root 91 0.0 0.0 7064 1548 pts/2 R+ 20:47 0:00 ps aux
root@33e2dab2b163:/# gpuctl show gpu all | grep gpus
No. of gpus : 1
root@33e2dab2b163:/# gpuctl --node-svc-socket /var/run/gpuagent.sock show gpu all | grep -i vendor
Card vendor : AMD
Memory vendor : samsung
VRAM vendor : samsung
root@33e2dab2b163:/
```
4c229aa to
ed736b6
Compare
sarat-k
reviewed
Feb 11, 2026
| "[-s <socket-path> | --grpc-unix-socket <socket-path>]\n\n", | ||
| argv[0]); | ||
| fprintf(stdout, "Options:\n"); | ||
| fprintf(stdout, " -p, --grpc-server-port <port> gRPC server port (default: %d)\n", |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivation
Add support to run grpc over unix domain socket for secure access
Technical Details
-s, --grpc-unix-socket <path>node-svc-socketadded to set non default custom socket for grpc connectionTest Plan