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

bpf: Mark 169.254.254.1 address as link scope #1575

Merged
merged 1 commit into from
Sep 22, 2017
Merged

Conversation

tgraf
Copy link
Member

@tgraf tgraf commented Sep 22, 2017

This should prevent kubeadm from picking it up as InternalIP candidate.

Signed-off-by: Thomas Graf [email protected]

This should prevent kubeadm from picking it up as InternalIP candidate.

Signed-off-by: Thomas Graf <[email protected]>
@tgraf tgraf added wip kind/bug This is a bug in the Cilium logic. labels Sep 22, 2017
GO_BINDATA_SHA1SUM=9531ba86f751cad1eb25fe2b913f13a448aab4e6
GO_VERSION_USED=go1.9
GO_BINDATA_SHA1SUM=4846bd2f17b1bb2c3f3722a631169917b2c29ba3
GO_VERSION_USED=go1.8.3
Copy link
Member

Choose a reason for hiding this comment

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

Will this go version 1.8.3 affect other developers? Is this GO_VERSION_USED being used one a location that I'm not aware of?

Copy link
Member Author

Choose a reason for hiding this comment

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

I'll update to go1.9. It's not used anywhere, it's only used for us.

@tgraf tgraf added pending-review and removed wip labels Sep 22, 2017
@tgraf tgraf merged commit 0c7811b into master Sep 22, 2017
@tgraf tgraf deleted the host-ip-link-scope branch September 22, 2017 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug This is a bug in the Cilium logic.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants