Don't start up multiple apiserver load balancers

get() is called in a loop until client configuration is successfully retrieved. Each iteration will try to configure the apiserver proxy, which will in turn create a new load balancer. Skip creating a new load balancer if we already have one. Signed-off-by: Brad Davidson <brad.davidson@rancher.com>

Don't start up multiple apiserver load balancers
8ace8975 · Brad Davidson · Brad Davidson · c0d12900 · 8ace8975
Commit 8ace8975 authored Mar 08, 2021 by Brad Davidson Committed by Brad Davidson Mar 08, 2021
Hide whitespace changes
Inline Side-by-side

Showing with 1 addition and 1 deletion

apiproxy.go pkg/agent/proxy/apiproxy.go +1 -1

No files found.
--- a/pkg/agent/proxy/apiproxy.go
+++ b/pkg/agent/proxy/apiproxy.go
@@ -119,7 +119,7 @@ func (p *proxy) SetAPIServerPort(ctx context.Context, port int) error {
 	p.apiServerURL = u.String()
 	p.apiServerEnabled = true

-	if p.lbEnabled {
+	if p.lbEnabled && p.apiServerLB == nil {
 		lbServerPort := p.lbServerPort
 		if lbServerPort != 0 {
 			lbServerPort = lbServerPort - 1