fix: lint

garrettladley · Oct 23, 2024 · 2a6c0e0 · 2a6c0e0
1 parent d2a2968
commit 2a6c0e0
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/internal/agent/agent_test.go b/internal/agent/agent_test.go
@@ -134,7 +134,7 @@ func client(t *testing.T, agent *agent.Agent, tlsConfig *tls.Config) api.LogClie
 	opts := []grpc.DialOption{grpc.WithTransportCredentials(tlsCreds)}
 	rpcAddr, err := agent.Config.RPCAddr()
 	require.NoError(t, err)
-	conn, err := grpc.Dial(fmt.Sprintf(
+	conn, err := grpc.NewClient(fmt.Sprintf(
 		"%s:///%s",
 		loadbalance.Name,
 		rpcAddr,

diff --git a/internal/loadbalance/resolver.go b/internal/loadbalance/resolver.go
@@ -87,10 +87,15 @@ func (r *Resolver) ResolveNow(resolver.ResolveNowOptions) {
 			),
 		})
 	}
-	r.clientConn.UpdateState(resolver.State{
+	if err := r.clientConn.UpdateState(resolver.State{
 		Addresses:     addrs,
 		ServiceConfig: r.serviceConfig,
-	})
+	}); err != nil {
+		r.logger.Error(
+			"failed to update state",
+			zap.Error(err),
+		)
+	}
 }
 
 func (r *Resolver) Close() {

diff --git a/internal/loadbalance/resolver_test.go b/internal/loadbalance/resolver_test.go
@@ -1,6 +1,7 @@
 package loadbalance_test
 
 import (
+	"fmt"
 	"net"
 	"net/url"
 	"testing"
@@ -37,7 +38,11 @@ func TestResolver(t *testing.T) {
 	}, grpc.Creds(serverCreds))
 	require.NoError(t, err)
 
-	go srv.Serve(l)
+	go func() {
+		if err := srv.Serve(l); err != nil {
+			panic(fmt.Errorf("failed to serve: %w", err))
+		}
+	}()
 
 	conn := &clientConn{}
 	tlsConfig, err = config.SetupTLSConfig(config.TLSConfig{