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

Updates for flannelConfig.md #68

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nalind
Copy link
Contributor

@nalind nalind commented Jun 25, 2015

  • Use rhel-tools instead of soon-to-be-EOL fedora:20.
  • Set now-required ETCD_ADVERTISE_CLIENT_URLS.
  • Add ordering information so that systemd will always start etcd before flannel on the master node.
  • Add whitespace to sample ip a l eth0 output to better match the formatting of a live run.

Add dependencies to the flanneld service configuration for systemd, so
that systemd will make sure the network is up and etcd is running before
it goes to start flannel.
Fedora 20's about to hit EOL, and later versions of the fedora images
don't appear to include iputils.  Switch to using rhel-tools instead,
since it's already the big bag of troubleshooting tools.
In the captured output of the 'ip a' command, include leading whitespace
in the sample output to more closely match the expected output.
As of etcd 2.0.11, etcd requires ETCD_ADVERTISE_CLIENT_URLS to be set if
ETCD_LISTEN_CLIENT_URLS is set, and will fail to start with an error if
it is not set.
# systemctl enable flanneld.service
# mkdir /etc/systemd/system/flanneld.service.wants
# ln -s /usr/lib/systemd/system/etcd.service /etc/systemd/system/flanneld.service.wants/
# ln -s /usr/lib/systemd/system/NetworkManager-wait-online.service /etc/systemd/system/flanneld.service.wants/
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This absolutely should not be necessary... flannel waits until it can find an ip/default route... What problem did you have?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think you may be right, that the startup problems I was seeing could have been caused by not being up yet. This is what happened when etcd was being started later:

Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: E0623 14:06:36.579070 00697 main.go:151] Failed to create SubnetManager: 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]
Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: E0623 14:06:37.581440 00697 main.go:151] Failed to create SubnetManager: 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]
Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: E0623 14:06:38.582929 00697 main.go:151] Failed to create SubnetManager: 501: All the given peers are not reachable (Tried to connect to each peer twice and failed) [0]
Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: I0623 14:06:41.377479 00697 main.go:247] Installing signal handlers
Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: I0623 14:06:41.377529 00697 main.go:118] Determining IP address of default interface
Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: E0623 14:06:41.378311 00697 main.go:190] Failed to get default interface: Unable to find default route
Jun 23 14:06:55 nalin-atomic-1.os1.phx2.redhat.com flanneld[697]: I0623 14:06:41.378335 00697 main.go:264] VXLAN mode exited

They're still being started in the right order on my system, even after removing the .wants symlink, so I'm not seeing this problem any more.

Copy link
Collaborator

@eparis eparis Jun 26, 2015 via email

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

2 participants