Select random router from map

Author: NickCao

deploy/helm/jumpstarter/charts/jumpstarter-controller/templates/controller-deployment.yaml

@@ -65,14 +65,6 @@ spec:
           {{ else }}
           value: grpc.{{ .Values.global.baseDomain }}:{{ .Values.grpc.tls.port }}
           {{ end }}
-        - name: GRPC_ROUTER_ENDPOINT
-          {{ if .Values.grpc.router.default.endpoint }}
-          value: {{ .Values.grpc.router.default.endpoint }}
-          {{ else if .Values.router.default.hostname }}
-          value: {{ .Values.router.default.hostname }}:{{ .Values.grpc.tls.port }}
-          {{ else }}
-          value: router.{{ .Values.global.baseDomain }}:{{ .Values.grpc.tls.port }}
-          {{ end }}
         - name: CONTROLLER_KEY
           valueFrom:
             secretKeyRef:

internal/service/controller_service.go

@@ -20,6 +20,7 @@ import (
 	"context"
 	"crypto/tls"
 	"fmt"
+	"math/rand"
 	"net/http"
 	"os"
 	"strings"
@@ -413,8 +414,19 @@ func (s *ControllerService) Dial(ctx context.Context, req *pb.DialRequest) (*pb.
 		return nil, status.Errorf(codes.Internal, "unable to sign token")
 	}
 
-	// TODO: find best router from list
-	endpoint := routerEndpoint()
+	var endpoint string
+	// Current go map implementation guarantees a random ordering
+	for name, v := range s.Router {
+		endpoint = v.Endpoint
+		logger.Info("selected router", "name", name, "endpoint", endpoint)
+		break
+	}
+
+	if endpoint == "" {
+		err := fmt.Errorf("no router available")
+		logger.Error(err, "no router available")
+		return nil, err
+	}
 
 	response := &pb.ListenResponse{
 		RouterEndpoint: endpoint,