Update the default keep alive params for the grpc client (#20)

* Update the default keep alive params for the grpc client
nginx · Aug 22, 2022 · baf3723 · baf3723
1 parent bcb8b79
commit baf3723
Show file tree

Hide file tree

Showing 6 changed files with 9 additions and 9 deletions.
diff --git a/sdk/examples/README.md b/sdk/examples/README.md
@@ -6,13 +6,13 @@ To run the server example from this context:
 ```
 go run server.go
 
-INFO[0000] http listening at 54790                      
-INFO[0000] grpc listening at 54789
+INFO[0000] http listening                    
+INFO[0000] grpc listening
 ```
 
 The above command should show the following output in the application:
-INFO[0000] http listening at 54790                      
-INFO[0000] grpc listening at 54789    
+INFO[0000] http listening                       
+INFO[0000] grpc listening    
 
 Configuring the Agent to run against the example server
 ### Agent Settings

diff --git a/sdk/grpc/grpc.go b/sdk/grpc/grpc.go
@@ -31,7 +31,7 @@ var (
 		grpc.WithStreamInterceptor(grpc_retry.StreamClientInterceptor()),
 		grpc.WithUnaryInterceptor(grpc_retry.UnaryClientInterceptor()),
 		grpc.WithKeepaliveParams(keepalive.ClientParameters{
-			Time:                10 * time.Second,
+			Time:                120 * time.Second,
 			Timeout:             60 * time.Second,
 			PermitWithoutStream: true,
 		}),

diff --git a/src/core/metrics/sources/cpu.go b/src/core/metrics/sources/cpu.go
@@ -82,7 +82,7 @@ func (c *CPUTimes) Collect(ctx context.Context, wg *sync.WaitGroup, m chan<- *pr
 
 		if err != nil {
 			// linux impl returns zero length without error
-			log.Errorf("Failed to get CPU metrics %v", err)
+			log.Warnf("Error occurred getting CPU metrics, %v", err)
 			return
 		}
 

diff --git a/test/performance/vendor/github.com/nginx/agent/sdk/v2/grpc/grpc.go b/test/performance/vendor/github.com/nginx/agent/sdk/v2/grpc/grpc.go
diff --git a/test/performance/vendor/github.com/nginx/agent/v2/src/core/metrics/sources/cpu.go b/test/performance/vendor/github.com/nginx/agent/v2/src/core/metrics/sources/cpu.go
diff --git a/vendor/github.com/nginx/agent/sdk/v2/grpc/grpc.go b/vendor/github.com/nginx/agent/sdk/v2/grpc/grpc.go