-
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
loader : Log upsert and remove route errors #15525
Conversation
Fixes : cilium#15282 Signed-off-by: Gaurav Genani <h3llix.pvt@gmail.com>
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.
now looks great :D Thanks!
test-me-please |
test-1.20-4.19 Previously failed with https://jenkins.cilium.io/job/Cilium-PR-K8s-1.20-kernel-4.19/11/testReport/junit/Suite-k8s-1/20/K8sServicesTest_Checks_service_across_nodes_Tests_NodePort_BPF_Tests_with_vxlan_Tests_NodePort_with_sessionAffinity/. Likely a flake. |
I opened #15572 for the new flake. Merging. |
if ep.RequireEndpointRoute() { | ||
upsertEndpointRoute(ep, *ip.IPNet(32)) | ||
if err := upsertEndpointRoute(ep, *ip.IPNet(32)); err != nil { | ||
scopedLog.WithError(err).Warn("Failed to upsert route") |
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.
Does this log message contain the actual route which could not be added/removed? That would be super helpful details to have when these cases are hit.
Fixes : #15282
Signed-off-by: Gaurav Genani h3llix.pvt@gmail.com