tkestack / tke

Native Kubernetes container management platform supporting multi-tenant and multi-cluster
Other
1.47k stars 334 forks source link

fix(health check): check cluster status for multiple times in health check go get rid of socket connection failure #2310 #2311

Closed pikehuang closed 1 year ago

pikehuang commented 1 year ago

What type of PR is this?

Uncomment only one /kind <> line, hit enter to put that in a new line, and remove leading whitespace from that line: /kind bug

What this PR does / why we need it: fix bug #2310 simple description: cluster health check will failed in most case bacause the socket connection to the target cluster is not stable, so we check multiple times when for cluster health check.

Which issue(s) this PR fixes:

Fixes #2310

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

leoryu commented 1 year ago

LGTM