Skip to content
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

健康检查结果不对 #25

Closed
idmaoning opened this issue Jan 9, 2020 · 5 comments
Closed

健康检查结果不对 #25

idmaoning opened this issue Jan 9, 2020 · 5 comments
Labels
bug Something isn't working patch ready patch ready and merged, if the further feedback is resolved, we can close it.

Comments

@idmaoning
Copy link

监控了两个udp服务,然后把两个服务都停掉,状态还是up

@veduco
Copy link

veduco commented Feb 5, 2020

UDP health check always returns true.
Any upstreams monitored with UDP don't accurately reflect the endpoints status.

Tested on v1.17.8

@ice2lin
Copy link

ice2lin commented Mar 20, 2020

udp 的健康检查还是有问题,随便填一个 udp 结果都是 up

@zhouchangxun
Copy link
Owner

zhouchangxun commented Jun 13, 2020

@idmaoning @veduco @ice2lin
The root cause is that I hadn't set default check data which cause the checker didn't send check packet.
I fixed it in this commit: 6188804

Please re-pull the master branch and re-compile & test it again if needed.

Anyway, thank you for you guys' feedback :D

@zhouchangxun zhouchangxun added the bug Something isn't working label Jun 13, 2020
@zhouchangxun
Copy link
Owner

Please write here if the patch works and I will close this issue after that.

@zhouchangxun zhouchangxun added the patch ready patch ready and merged, if the further feedback is resolved, we can close it. label Jun 14, 2020
@zhouchangxun
Copy link
Owner

UDP check depends on icmp reply(port/host unreachable).
So please make sure icmp can work between nginx and backend server otherwise it will assume backend is alive if no any icmp reply.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working patch ready patch ready and merged, if the further feedback is resolved, we can close it.
Projects
None yet
Development

No branches or pull requests

4 participants