-
-
Notifications
You must be signed in to change notification settings - Fork 455
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
[FIX] do not ignore --no-hostip and don't try to inject if network = host #471
Conversation
Not sure if I'm missing something else, but with this fix I am able to create my cluster with |
I also noticed this behavior. I may be wrong but I think this is unrelated to this fix. I can look into it and fix here or in another PR soon but need to ask someone (@iwilltry42 probably):
An alternative is to completely ignore |
👍 I think it is reasonable to have it be a separate issue and can be addressed in a separate PR. I haven't tested this specifically, but I think you can easily work around it if you include |
Hi @konradmalik , thanks for this PR! For the other issue mentioned in the comments, please open a separate issue (or even PR :)). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Thank you :)
Sure, I think I tried to overcomplicate things. I'll do this in a separate PR soon ;) |
Fix for: #470 + implemented the suggestion that docker host injection should not be performed when network=host.