Add tunnel mode healthcheck

google / seesaw

Seesaw v2 is a Linux Virtual Server (LVS) based load balancing platform.

Apache License 2.0

5.65k stars 511 forks source link

Add tunnel mode healthcheck #98

Closed unicell closed 4 years ago

unicell commented 4 years ago

This PR adds the tunnel mode healthcheck support. It consists of following commits:

Add tunneling mode healthcheck in config.proto
Allow tunnel mode healthcheck to use specified source IP
Use engine node address for tunnel mode healthcheck source address
Let healthcheckManager aware of TUN mode healthcheck
Add test coverage for DSR/TUN healthchecks in dedup() logic

liuyuan10 commented 4 years ago

Before I dive into it, I'm curious about how you are using the tunnel mode. The return packet is still bypassing the LBs right? Are you using vlans or anyIP?

unicell commented 4 years ago

Before I dive into it, I'm curious about how you are using the tunnel mode. The return packet is still bypassing the LBs right? Are you using vlans or anyIP?

The returning packet will bypass the LB (ipvs), and directly goes to the client IP. VLAN is not required. Since tunnel mode traffic can go across L2 domains, so as long as backend IPs are reachable from ipvs.

unicell commented 4 years ago

@liuyuan10 Sorry for the late update. So the address the earlier comment, several commits are added.

additional test case added to cover the h.marks / (un)markBackend / pruneMarks path (also found an issue with existing pruneMarks() logic, fixed it along the way)
ensure markBackend logic also works for TUN mode healthcheck

Each commit is self contained. Let me know if any one of them doesn't make sense, or if you wanna break it up into several PRs. Thanks!