-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
sctp: Handle SCTP when correlating Endpoints to services. #21490
Conversation
c8ae112
to
fc508e3
Compare
/test Job 'Cilium-PR-K8s-1.25-kernel-net-next' failed: Click to show.Test Name
Failure Output
If it is a flake and a GitHub issue doesn't already exist to track it, comment |
fc508e3
to
0cc02ff
Compare
Otherwise, Services with SCTP ports and pods with SCTP ports will not have their backends correctly populated. This was missed in the original SCTP PR (b1d9fcd). Signed-off-by: Harsh Modi <harshmodi@google.com>
0cc02ff
to
ab89ebc
Compare
The failure seems unrelated to my change :-/ |
@DolceTriade yes, if you search the issues for this test title you can find an issue filed for this already: #21120 |
/test |
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.
LGTM 💯
Otherwise, Services with SCTP ports and pods with SCTP ports will not have their backends correctly populated.
This was missed in the original SCTP PR.
Signed-off-by: Harsh Modi harshmodi@google.com