Skip to content

cilium: only enable IPv6 forwarding if IPv6 is enabled #9034

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

Merged
merged 1 commit into from
Aug 25, 2019

Conversation

jrfastab
Copy link
Contributor

@jrfastab jrfastab commented Aug 25, 2019

When running with a kernel that has IPv6 compiled out there is no
/proc/net/ipv6/ directory so trying to enable IPv6 fails and the
daemon can not start.

To resolve this only enable IPv6 forwarding when daemon is running with
IPv6 enabled.

Error seen with IPv6 compiled out and IPv6 disabled in config map is

level=fatal msg="Error when enabling sysctl parameters" error="could not open the sysctl file /proc/sys/net/ipv6/conf/all/forwarding: open /proc/sys/net/ipv6/conf/all/forwarding: no such file or directory" subsys=daemon

Fixes: 294cf66 ("daemon: Enable IP forwarding on start")
Signed-off-by: John Fastabend john.fastabend@gmail.com


This change is Reviewable

When running with a kernel that has IPv6 compiled out there is no
/proc/net/ipv6/ directory so trying to enable IPv6 fails and the
daemon can not start.

To resolve this only enable IPv6 forwarding when daemon is running with
IPv6 enabled.

Error seen with IPv6 compiled out and IPv6 disabled in config map is

level=fatal msg="Error when enabling sysctl parameters" error="could not open the sysctl file /proc/sys/net/ipv6/conf/all/forwarding: open /proc/sys/net/ipv6/conf/all/forwarding: no such file or directory" subsys=daemon

Fixes: 294cf66 ("daemon: Enable IP forwarding on start")
Signed-off-by: John Fastabend <john.fastabend@gmail.com>
@jrfastab jrfastab requested a review from a team August 25, 2019 06:15
@jrfastab jrfastab added kind/bug This is a bug in the Cilium logic. area/daemon Impacts operation of the Cilium daemon. labels Aug 25, 2019
@jrfastab
Copy link
Contributor Author

test-me-please

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.01%) to 44.097% when pulling 72757d5 on fix-ipv6-disabled into fb2a99e on master.

@tgraf tgraf merged commit ef01850 into master Aug 25, 2019
@tgraf tgraf deleted the fix-ipv6-disabled branch August 25, 2019 08:28
@aanm aanm added the release-note/bug This PR fixes an issue in a previous release of Cilium. label Nov 14, 2019
@jdef
Copy link

jdef commented Feb 6, 2020

any chance of this being backported to cilium 1.6.x?

@tgraf
Copy link
Member

tgraf commented Feb 6, 2020

Added label, thanks for reporting @jdef

@tgraf tgraf added kind/regression This functionality worked fine before, but was broken in a newer release of Cilium. needs-backport/1.5 labels Feb 10, 2020
@jrfastab
Copy link
Contributor Author

@raybejjani same here update tags ok?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/daemon Impacts operation of the Cilium daemon. kind/bug This is a bug in the Cilium logic. kind/regression This functionality worked fine before, but was broken in a newer release of Cilium. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants