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
v1.14 Backports 2024-05-11 #32483
v1.14 Backports 2024-05-11 #32483
Conversation
d782860
to
7878a97
Compare
[ upstream commit 32b2784 ] This commit is to re-enable proxy rule installation in native mode if cilium envoy config or ipsec is enabled. The reason is to handle the reply packet of hair-pinning traffic in Ingress L7 proxy (i.e. backend is in the same node). Relates: 0ebe516 Relates: #29530, #29864 Signed-off-by: Tam Mach <tam.mach@cilium.io>
[ upstream commit 4ce4bb1 ] This is to make sure that we have a coverage for hair pinning traffic i.e. backend is on the same node with client. Additionally, this test will provide some coverage with cloud provider (e.g. GKE), as all other Ingress/Gateway API/L7 LB tests are running with kind clusters. Signed-off-by: Tam Mach <tam.mach@cilium.io>
[ upstream commit bc41564 ] This is to pick up the changes for Ingress controller connectivity tests Signed-off-by: Tam Mach <tam.mach@cilium.io>
7878a97
to
13ebecb
Compare
/test-backport-1.14 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
init.sh changes look fine to me 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
bpf/init.sh looks good.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:rubber-stamp:
Once this PR is merged, a GitHub action will update the labels of these PRs: