Skip to content
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

use natively packaged flannel in CoreOS' cloud-configs #5455

Merged

Conversation

AntonioMeireles
Copy link
Contributor

in the footsteps of #5254 merge earlier today and per @brendandburns comments
in #5254 's review thread...

@AntonioMeireles
Copy link
Contributor Author

@AntonioMeireles AntonioMeireles force-pushed the use_native_coreos_flannel branch 6 times, most recently from 6aba307 to 68c9368 Compare March 16, 2015 19:37
in the footsteps of kubernetes#5254 merge earlier today and per @brendandburns comments
in the PR review thread...

Signed-off-by: António Meireles <antonio.meireles@reformi.st>
…unning.

Signed-off-by: António Meireles <antonio.meireles@reformi.st>
@AntonioMeireles AntonioMeireles force-pushed the use_native_coreos_flannel branch from 68c9368 to 1891ba0 Compare March 17, 2015 08:38
brendandburns added a commit that referenced this pull request Mar 17, 2015
use natively packaged flannel in CoreOS' cloud-configs
@brendandburns brendandburns merged commit 3462f68 into kubernetes:master Mar 17, 2015
@AntonioMeireles AntonioMeireles deleted the use_native_coreos_flannel branch March 17, 2015 14:34
@hobti01
Copy link

hobti01 commented Mar 19, 2015

@AntonioMeireles do you have a reference for ExecStart= in the dropin? systemd does not process the dropin on my CoreOS.

re: --iface=eth1 - I would expect eth1 to exist on vagrant due to https://coreos.com/releases/#379.0.0 But this configuration is for more than vagrant, surely?

@hobti01
Copy link

hobti01 commented Mar 19, 2015

Found a reference for ExecStart= here: https://bugzilla.redhat.com/show_bug.cgi?id=756787#c12

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants