-
Notifications
You must be signed in to change notification settings - Fork 45
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
Allow configuration of container capabilities #58
Comments
After inspecting the code and the service API, it looks like this will not be possible (without work on the docker engine as well...and I don't want to hassle with that). A workaround would be to set the MAC address within the container, but we'd need to allow for certain capabilities to be set. The API does not expose this either in the most recent releases (which also prevents us from being able to build docker containers using the swarm plugin). A change is pending release that will allow us to do this (moby/moby#39173). Perhaps, we should wait for the next Docker release and then add capabilities to a label instead so that we kill two birds with one stone. |
Just saw the "dind" code after my last post. I suppose I can use that for now as a workaround, but will leave this open to have a tighter integration with the Docker API in the future. |
Spoke too soon. Dind will only allow me to run privileged with the docker in docker image. Rethinking the approach for the workaround now as well. |
I needed this feature to be able to mount an NFS drive as a deployment step in one of my builds. However, my docker cloud agent template is already set up to bind-mount the swarm host's |
In the same spirit of #13, I have the same requirement to set a custom MAC address for my container at runtime. I know that there's a separate issue open for more customized parameters, but I'd like to submit a PR for an approach that is similar to the one that was taken for DNS. Let me know if there any objections, otherwise, I will take this up in the next couple of days. Thanks!
The text was updated successfully, but these errors were encountered: