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

Enable IPv4/IPv6 forwarding on demand automatically #5833

Merged
merged 1 commit into from
Jan 4, 2024

Conversation

tnqn
Copy link
Member

@tnqn tnqn commented Jan 2, 2024

Although it has been documented as a prerequisite in [1], there are some platforms not enabling ip forwarding by default. kube-proxy ipvs mode and some CNIs enable it by themselves to ensure Pod networking work properly.

As Antrea needs IP forwarding to be enabled, there seems no reason to not do it by itself, rather than expecting users or other components to do it.

[1] https://kubernetes.io/docs/setup/production-environment/container-runtimes/#install-and-configure-prerequisites

For vmware/antrea-operator-for-kubernetes#98

Although it has been documented as a prerequisite in [1], there are
some platforms not enabling ip forwarding by default. kube-proxy ipvs
mode and some CNIs enable it by themselves to ensure Pod networking
work properly.

As Antrea needs IP forwarding to be enabled, there seems no reason to
not do it by itself, rather than expecting users or other components to
do it.

[1] https://kubernetes.io/docs/setup/production-environment/container-runtimes/#install-and-configure-prerequisites

Signed-off-by: Quan Tian <qtian@vmware.com>
@tnqn tnqn added the action/release-note Indicates a PR that should be included in release notes. label Jan 3, 2024
@antoninbas
Copy link
Contributor

/test-all

@tnqn tnqn merged commit 0204fbc into antrea-io:main Jan 4, 2024
47 of 53 checks passed
@tnqn tnqn deleted the enable-ip-forward branch January 4, 2024 07:32
@jsalatiel
Copy link

Will this be backported to 1.14?

@tnqn
Copy link
Member Author

tnqn commented Jan 10, 2024

Will this be backported to 1.14?

It's being backported to 1.13 and 1.14.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/release-note Indicates a PR that should be included in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants