-
Notifications
You must be signed in to change notification settings - Fork 7
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
Module need Chownroot permission to init #1
Comments
The idea is to simply load the module and get metrics in return without worrying to much about preparation. |
Thanks for such reactivity ! |
I would not remove the chown, but put it in a |
Good new : |
sure, please prepare a PR and i am happy to merge it. |
As module intialization depends on :
[module.go :: startMetricServer] err = os.Chown(socketPath, userID, groupID)
It cause following init error:
Cannot chown metricssocket: chown /tmp/mtr.0juus5: operation not permitted
When such module is run within a "secured" container it cannot pass the init state.
As long as we can configure the DEFAULTTMPFOLDER with proper permission upfront: => it could worth avoid to do this Chown.
(Thanks a lot in advance for any support you may provide, we are definitly lacking such module)
The text was updated successfully, but these errors were encountered: