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

iptables: Add extra warning message listing missing IPV6 kernel modules #16842

Merged
merged 1 commit into from Aug 5, 2021

Conversation

oneiro-naut
Copy link
Contributor

This warning is shown only if ipv6 is enabled in cilium.

Fixes: #11162

@oneiro-naut oneiro-naut requested review from a team and jrfastab July 9, 2021 14:46
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 9, 2021
Copy link
Member

@nathanjsweet nathanjsweet left a comment

Choose a reason for hiding this comment

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

Super small nit. Otherwise, I think this is a good warning to print.

Fixes: cilium#11162

Signed-off-by: Ayush Dwivedi <ayush.dwivedi@accuknox.com>
@oneiro-naut
Copy link
Contributor Author

Hi @nathanjsweet. I made the suggested the change.

Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

:shipit:

@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Jul 13, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 13, 2021
@aanm
Copy link
Member

aanm commented Jul 20, 2021

test-me-please

@aanm aanm merged commit bafa03c into cilium:master Aug 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kernel 5.7: could not insert 'ip6_tables'
5 participants