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

do we actually need cap_add, if yes, we could document on why and if not, remove it. #135

Closed
may11544 opened this issue Dec 2, 2022 · 3 comments
Labels
type/question Further information is requested

Comments

@may11544
Copy link

may11544 commented Dec 2, 2022

The first question: I see that the docker permission (cap_add) of graphd, storaged and metad has enabled SYS_PTRACE. Is this necessary, and can I ignore it.

The second question: Taking the storage configuration as an example, I see that for the http protocol, both the external port and the internal port are open, and they are 19779 and 19780 respectively. But for the rpc protocol, only the external port (9779) is open, but the internal port is not open. According to the official document, they are 9777, 9778 and 9780 respectively. May I ask if I should configure it where the port mapping is.

@wey-gu
Copy link
Contributor

wey-gu commented Dec 2, 2022

Thanks @may11544 for taking time feedback this :)

for the first one, we will revisit to see if we could get rid of it.

for the second one, it's reported here: vesoft-inc/nebula-docs-cn#2371

@wey-gu wey-gu changed the title Some questions about docker-compose configuration do we actually need cap_add, if yes, we could document on why and if not, remove it. Dec 2, 2022
@yixinglu
Copy link
Contributor

@may11544 If I remember correctly, the reason for the first question should be to allow developers to gdb the process inside the container. If you need not to debug something, I think you could disable the option.

@QingZ11
Copy link
Contributor

QingZ11 commented Jan 30, 2023

@may11544 hi, I have noticed that the issue you created hasn’t been updated for nearly a month, is this issue been resolved? If not resolved, can you provide some more information? If solved, can you close this issue?

Thanks a lot for your contribution anyway 😊

@wey-gu wey-gu added the type/question Further information is requested label Feb 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type/question Further information is requested
Projects
None yet
Development

No branches or pull requests

4 participants