-
Notifications
You must be signed in to change notification settings - Fork 32
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
docker: remove cni-plugins #8
Conversation
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
I didn't have a look yet, what issues did you have?
🤨 |
It's something along those lines, hashicorp/nomad#8371
Now that there is a convenient way to install Nomad (and Consul), I'm only using this image for the server. For the client, it works too, but not when it starts playing with network namespaces. Kubernetes probably knows how to work around this. |
|
I also forgot to remove |
Signed-off-by: Yoan Blanc <yoan@dosimple.ch>
Oh right, I missed the announcement of the HashiCorp APT repository somehow. |
🎉 Also, it needs the
|
@multani For what it's worth, I've just run into the same error.
The k8s logic that you mentioned is actually just the mount configuration, in particular the "bind-propagation" configuration (https://docs.docker.com/storage/bind-mounts/#configure-bind-propagation). I think changing the docker-compose to use this should fix the issue :D :
For reference: https://github.com/MatthewJohn/vault-nomad-consul-terraform/blob/setup-service-mesh/modules/nomad/client/container/main.tf#L81 |
Thanks for the heads-up @MatthewJohn! TBH, I never really played with any CNI plugins until now, I'm happy to make some changes but I would need to check how I could test that beforehand 👍 |
I've tried using it and failed miserably, only to fall back to installing Nomad via rpm/deb. It's probably not worth it making this image bigger with the CNI plugins if there is no possible use for them.