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.

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

[Mellanox] Incorrect permission of /tmp in syncd container #16034

Closed
bingwang-ms opened this issue Aug 4, 2023 · 2 comments · Fixed by #17777
Closed

[Mellanox] Incorrect permission of /tmp in syncd container #16034

bingwang-ms opened this issue Aug 4, 2023 · 2 comments · Fixed by #17777
Assignees
Labels
NVIDIA Triaged this issue has been triaged

Comments

@bingwang-ms
Copy link
Contributor

bingwang-ms commented Aug 4, 2023

Description

The permission of /tmp path in syncd container is incorrect. The expected permission should be 777, but it's 755.

admin@str-msn2700-22:~$ docker exec -it syncd bash
root@str-msn2700-22:/# ls -al /
total 1944
...
drwxr-xr-x   2 root root 4096 Aug  4 04:07 tmp

It's because the /tmp path is mounted from /tmp/nv-syncd-shared from host, and the /tmp/nv-syncd-shared is created by nv-syncd-shared.service sonic-buildimage/platform/mellanox/nv-syncd-shared/nv-syncd-shared.service at master · sonic-net/sonic-buildimage (github.com)

The fix is to create the path and set the permission to 777 in syncd-shared.service.

Steps to reproduce the issue:

Describe the results you received:

Describe the results you expected:

Output of show version:

(paste your output here)

Output of show techsupport:

(paste your output here or download and attach the file here )

Additional information you deem important (e.g. issue happens only occasionally):

@bingwang-ms
Copy link
Contributor Author

bingwang-ms commented Aug 14, 2023

The syncd-shared.service was introduced in PR #14547. So the issue should be in 202305 branch and above.

@bingwang-ms
Copy link
Contributor Author

@volodymyrsamotiy Can you please help assign this issue?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
NVIDIA Triaged this issue has been triaged
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants