-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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.13 Backports 2023-06-28 #26528
v1.13 Backports 2023-06-28 #26528
Conversation
[ upstream commit b8786b3 ] This commit is to add loadBalancerIP and loadBalancerClass for shared Ingress service. Kindly note the below points: - loadbalancerIP feature is depending on cloud provider controller - loadBalancerClass is only supported for k8s 1.24 and later. Signed-off-by: Oliver Ni <oliver.ni@gmail.com> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
[ upstream commit ee5f473 ] While refactoring the CT lookup helpers, I fat-fingered the order of parameters in the ct_lookup6() helper (which is used by eg. bpf_lxc). But as both parameters are primarily used for tracing policy, this didn't completely break things - we just don't get the expected trace output for packets, and have problems detecting closed / reopened connections. Fix the parameters up to make things work again. [ backporter's note: encountered merge conflict, modified to only switch l4_off and action in the __ct_lookup6 call. ] Fixes: eee902f ("bpf: conntrack: introduce an optimized CT lookup for LB") Reported-by: Marco Iorio <marco.iorio@isovalent.com> Signed-off-by: Julian Wiedmann <jwi@isovalent.com> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
[ upstream commit fd3814e ] before the fix 2 content headers are not returned ```release-note Fix: Return "Content-Type" and "X-Content-Type-Options" headers from Health Check Node Port ``` Signed-off-by: Cezary Zawadka <czawadka@google.com> Signed-off-by: Tobias Klauser <tklauser@distanz.ch>
/test-backport-1.13 Job 'Cilium-PR-K8s-1.25-kernel-4.19' failed: Click to show.Test Name
Failure Output
Jenkins URL: https://jenkins.cilium.io/job/Cilium-PR-K8s-1.25-kernel-4.19/766/ If it is a flake and a GitHub issue doesn't already exist to track it, comment Then please upload the Jenkins artifacts to that issue. |
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.
ta Tobias for resolving the conflict!
/test-1.25-4.19 |
PRs skipped due to conflicts:
Once this PR is merged, you can update the PR labels via:
or with