-
-
Notifications
You must be signed in to change notification settings - Fork 466
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
[FEATURE] Support custom options for docker network #165
Comments
Hi there, thanks for this issue 👍 |
Hi, Yes, thanks, works for my use case. |
Great, closing in favor of #220 |
Hi @iwilltry42 , it doesn't really work for my usecase... is there no way to tell k3d docker to create a different |
Hey @AntonOfTheWoods unfortunately it would be really cumbersome to expose all possible options for docker network creation via k3d. We're currently only exposing the choice of a subnet to use. |
@iwilltry42 genuine brainfart, sorry. I have some generic scripts and didn't want it to be specific to docker (vs podman) but that was just being dumb. It is actually fine. I did notice that if you create using the name it would have created anyway, it will use it without you explicitly specifying it (with a warning). That tripped me up because for some reason when I was deleting the cluster it, wasn't deleting the automatically created network. As such my I guess it is pretty obvious what is happening after a couple of minutes though, so probably not worth adding to the docs. Thanks again, |
Great, thanks for the feedback! :) |
Scope of your request
Add the ability to provide custom options for docker network create.
As k3d creates its own docker network, the network uses default MTU (1500).
In some environments, like OpenStack, configured with VxLAN, the host is already configured with a lower MTU (1450) so we need a way to provide a custom MTU for the docker network.
Describe the solution you'd like
An extra option for k3d create, like:
k3d create --docker-network-opt "com.docker.network.driver.mtu=1450" --docker-network-opt another-value
The text was updated successfully, but these errors were encountered: