-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
destination return expired ip when endpoints in namespace kube-system #4055
Comments
The service IP would break load balancing, which would be a weird gotcha. Do you know why the endpoint stale in k8s for this? Also, the service IP would try to load balance to the wrong IP anyways, resulting in the same problem =/ |
diff,see the code ,
the endpoint ip never update until restart destination |
It sounds like we just need to remove those lines? Why should we be using the service ip and how would any of this help the invalid output that you're getting from |
the kubectl get ip is valid. linkerd return ip 172.20.14.124 is invalid |
From your paste:
That contains 172.20.14.124, right? |
sorry, i make a mistake on the issue, from my paste ,i stop the mesh ,so we get right on linkerd return follow the step below,you can get the same:
|
Ahha, that makes way more sense! Up for a PR? @adleong any ideas why we'd just not bother doing discovery on |
I did some digging into the history and that kube-system exclusion has been there from the beginning. I can't recall why we put it in but as far as I can tell it should be safe to remove. |
@humboldt-xie are you interested in working on this? |
…stem linkerd#4055 Signed-off-by: humboldt <humboldt_xie@163.com>
i had remove the code when namespace is kube-system . review please |
We were being affected by this issue. We had two services running in kube-system namespace talking to each other (via Linkerd proxy), and whenever some pods restarted, they stopped communicating each other because linkerd could not find proper endpoints. Logs from Linkerd-proxy: Simply moving services out of kube-system namespace resolved this issue. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed in 14 days if no further activity occurs. Thank you for your contributions. |
the key code:
full step:
the 172.20.3.140 is invalid
I recommend using svc ip when encpoints in kube-system
The text was updated successfully, but these errors were encountered: