-
Notifications
You must be signed in to change notification settings - Fork 39
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
fix: enable-bpf-masquerade when snat values are not enabled #350
base: master
Are you sure you want to change the base?
Conversation
@hown3d Label kind/todo does not exist. |
Hi @hown3d. Thanks for your PR. I'm waiting for a gardener member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
@hown3d Thank you for your contribution. |
Thank you @hown3d for your contribution. Before I can start building your PR, a member of the organization must set the required label(s) {'reviewed/ok-to-test'}. Once started, you can check the build status in the PR checks section below. |
/ok-to-test |
/test pull-extension-networking-cilium-e2e-kind |
In tests I see an issue with this change together with the apiserver-proxy. |
Signed-off-by: Lukas Hoehl <[email protected]>
50a1e9b
to
8e35b32
Compare
@hown3d: The following test failed, say
Full PR test history. Your PR dashboard. Command help for this repository. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
How to categorize this PR?
/area networking
/kind TODO
What this PR does / why we need it:
Enable bpf-masquerading on direct routing if SNAT masquerades are not enabled.
For more information see the issue.
Which issue(s) this PR fixes:
Fixes #349
Special notes for your reviewer:
Release note: