Merge pull request #193 from irek02/local

k8s-ci-robot · web-flow · commit dd3910f2b756 · 2025-12-01T09:38:33.000-08:00
Fix flaky gRPC tests in sidecar
diff --git a/sidecar/internal/test/rpc.go b/sidecar/internal/test/rpc.go
@@ -89,5 +89,9 @@ func ClientConn(tmpSockUri string) (*grpc.ClientConn, error) {
 	return grpc.NewClient(tmpSockUri,
 		// nothing fancy for unit tests
 		grpc.WithTransportCredentials(insecure.NewCredentials()),
+		grpc.WithDefaultCallOptions(
+			// Handle server starting slower than client calls in tests
+			grpc.WaitForReady(true),
+		),
 	)
 }

Original file line number	Diff line number	Diff line change
`@@ -89,5 +89,9 @@ func ClientConn(tmpSockUri string) (*grpc.ClientConn, error) {`
`89`	`89`	`return grpc.NewClient(tmpSockUri,`
`90`	`90`	`// nothing fancy for unit tests`
`91`	`91`	`grpc.WithTransportCredentials(insecure.NewCredentials()),`
	`92`	`+ grpc.WithDefaultCallOptions(`
	`93`	`+ // Handle server starting slower than client calls in tests`
	`94`	`+ grpc.WaitForReady(true),`
	`95`	`+ ),`
`92`	`96`	`)`
`93`	`97`	`}`