From eba8217232f557a383b5603123dd5b995d55cc89 Mon Sep 17 00:00:00 2001 From: Rama Chavali Date: Sat, 23 Jul 2022 15:45:19 +0530 Subject: [PATCH 1/3] clarify destination rule docs Signed-off-by: Rama Chavali --- networking/v1alpha3/destination_rule.gen.json | 4 ++-- networking/v1alpha3/destination_rule.pb.go | 2 ++ networking/v1alpha3/destination_rule.pb.html | 6 ++++-- networking/v1alpha3/destination_rule.proto | 2 ++ networking/v1beta1/destination_rule.gen.json | 4 ++-- networking/v1beta1/destination_rule.pb.go | 2 ++ networking/v1beta1/destination_rule.proto | 2 ++ 7 files changed, 16 insertions(+), 6 deletions(-) diff --git a/networking/v1alpha3/destination_rule.gen.json b/networking/v1alpha3/destination_rule.gen.json index de1ee07028..4ba81ca547 100644 --- a/networking/v1alpha3/destination_rule.gen.json +++ b/networking/v1alpha3/destination_rule.gen.json @@ -74,12 +74,12 @@ "type": "object", "properties": { "http1MaxPendingRequests": { - "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1.", + "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", "type": "integer", "format": "int32" }, "http2MaxRequests": { - "description": "Maximum number of requests to a backend. Default 2^32-1.", + "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", "type": "integer", "format": "int32" }, diff --git a/networking/v1alpha3/destination_rule.pb.go b/networking/v1alpha3/destination_rule.pb.go index 5fa45584b0..43c7bab2bc 100644 --- a/networking/v1alpha3/destination_rule.pb.go +++ b/networking/v1alpha3/destination_rule.pb.go @@ -2220,8 +2220,10 @@ type ConnectionPoolSettings_HTTPSettings struct { unknownFields protoimpl.UnknownFields // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. Http1MaxPendingRequests int32 `protobuf:"varint,1,opt,name=http1_max_pending_requests,json=http1MaxPendingRequests,proto3" json:"http1_max_pending_requests,omitempty"` // Maximum number of requests to a backend. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. Http2MaxRequests int32 `protobuf:"varint,2,opt,name=http2_max_requests,json=http2MaxRequests,proto3" json:"http2_max_requests,omitempty"` // Maximum number of requests per connection to a backend. Setting this // parameter to 1 disables keep alive. Default 0, meaning "unlimited", diff --git a/networking/v1alpha3/destination_rule.pb.html b/networking/v1alpha3/destination_rule.pb.html index bb3decf5e6..70a92988d3 100644 --- a/networking/v1alpha3/destination_rule.pb.html +++ b/networking/v1alpha3/destination_rule.pb.html @@ -1723,7 +1723,8 @@

ConnectionPoolSettings.HTTPSettings http1MaxPendingRequests int32 -

Maximum number of pending HTTP requests to a destination. Default 2^32-1.

+

Maximum number of pending HTTP requests to a destination. Default 2^32-1. +Please note that this is applicable to both HTTP1.1/ and HTTP2.

@@ -1734,7 +1735,8 @@

ConnectionPoolSettings.HTTPSettings http2MaxRequests int32 -

Maximum number of requests to a backend. Default 2^32-1.

+

Maximum number of requests to a backend. Default 2^32-1. +Please note that this is applicable to both HTTP1.1/ and HTTP2.

diff --git a/networking/v1alpha3/destination_rule.proto b/networking/v1alpha3/destination_rule.proto index b0a60447fe..80be5750d2 100644 --- a/networking/v1alpha3/destination_rule.proto +++ b/networking/v1alpha3/destination_rule.proto @@ -710,9 +710,11 @@ message ConnectionPoolSettings { // Settings applicable to HTTP1.1/HTTP2/GRPC connections. message HTTPSettings { // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. int32 http1_max_pending_requests = 1; // Maximum number of requests to a backend. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. int32 http2_max_requests = 2; // Maximum number of requests per connection to a backend. Setting this diff --git a/networking/v1beta1/destination_rule.gen.json b/networking/v1beta1/destination_rule.gen.json index 0e74908a87..2fbded9545 100644 --- a/networking/v1beta1/destination_rule.gen.json +++ b/networking/v1beta1/destination_rule.gen.json @@ -74,12 +74,12 @@ "type": "object", "properties": { "http1MaxPendingRequests": { - "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1.", + "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", "type": "integer", "format": "int32" }, "http2MaxRequests": { - "description": "Maximum number of requests to a backend. Default 2^32-1.", + "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", "type": "integer", "format": "int32" }, diff --git a/networking/v1beta1/destination_rule.pb.go b/networking/v1beta1/destination_rule.pb.go index d5099508b1..865cf35b6c 100644 --- a/networking/v1beta1/destination_rule.pb.go +++ b/networking/v1beta1/destination_rule.pb.go @@ -2169,8 +2169,10 @@ type ConnectionPoolSettings_HTTPSettings struct { unknownFields protoimpl.UnknownFields // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. Http1MaxPendingRequests int32 `protobuf:"varint,1,opt,name=http1_max_pending_requests,json=http1MaxPendingRequests,proto3" json:"http1_max_pending_requests,omitempty"` // Maximum number of requests to a backend. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. Http2MaxRequests int32 `protobuf:"varint,2,opt,name=http2_max_requests,json=http2MaxRequests,proto3" json:"http2_max_requests,omitempty"` // Maximum number of requests per connection to a backend. Setting this // parameter to 1 disables keep alive. Default 0, meaning "unlimited", diff --git a/networking/v1beta1/destination_rule.proto b/networking/v1beta1/destination_rule.proto index 4bf4f22a05..344ca5f32e 100644 --- a/networking/v1beta1/destination_rule.proto +++ b/networking/v1beta1/destination_rule.proto @@ -659,9 +659,11 @@ message ConnectionPoolSettings { // Settings applicable to HTTP1.1/HTTP2/GRPC connections. message HTTPSettings { // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. int32 http1_max_pending_requests = 1; // Maximum number of requests to a backend. Default 2^32-1. + // Please note that this is applicable to both HTTP1.1/ and HTTP2. int32 http2_max_requests = 2; // Maximum number of requests per connection to a backend. Setting this From 915cfd10d8ff50bb2b5b9aaaa69f67c0f440bbc3 Mon Sep 17 00:00:00 2001 From: Rama Chavali Date: Mon, 25 Jul 2022 20:31:17 +0530 Subject: [PATCH 2/3] review comments Signed-off-by: Rama Chavali --- networking/v1alpha3/destination_rule.gen.json | 4 ++-- networking/v1alpha3/destination_rule.pb.go | 4 ++-- networking/v1alpha3/destination_rule.pb.html | 4 ++-- networking/v1alpha3/destination_rule.proto | 4 ++-- networking/v1beta1/destination_rule.gen.json | 4 ++-- networking/v1beta1/destination_rule.pb.go | 4 ++-- networking/v1beta1/destination_rule.proto | 4 ++-- 7 files changed, 14 insertions(+), 14 deletions(-) diff --git a/networking/v1alpha3/destination_rule.gen.json b/networking/v1alpha3/destination_rule.gen.json index 4ba81ca547..072044701f 100644 --- a/networking/v1alpha3/destination_rule.gen.json +++ b/networking/v1alpha3/destination_rule.gen.json @@ -74,12 +74,12 @@ "type": "object", "properties": { "http1MaxPendingRequests": { - "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", + "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, "http2MaxRequests": { - "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", + "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, diff --git a/networking/v1alpha3/destination_rule.pb.go b/networking/v1alpha3/destination_rule.pb.go index 43c7bab2bc..95fdce698e 100644 --- a/networking/v1alpha3/destination_rule.pb.go +++ b/networking/v1alpha3/destination_rule.pb.go @@ -2220,10 +2220,10 @@ type ConnectionPoolSettings_HTTPSettings struct { unknownFields protoimpl.UnknownFields // Maximum number of pending HTTP requests to a destination. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http1MaxPendingRequests int32 `protobuf:"varint,1,opt,name=http1_max_pending_requests,json=http1MaxPendingRequests,proto3" json:"http1_max_pending_requests,omitempty"` // Maximum number of requests to a backend. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http2MaxRequests int32 `protobuf:"varint,2,opt,name=http2_max_requests,json=http2MaxRequests,proto3" json:"http2_max_requests,omitempty"` // Maximum number of requests per connection to a backend. Setting this // parameter to 1 disables keep alive. Default 0, meaning "unlimited", diff --git a/networking/v1alpha3/destination_rule.pb.html b/networking/v1alpha3/destination_rule.pb.html index 70a92988d3..15e1277a50 100644 --- a/networking/v1alpha3/destination_rule.pb.html +++ b/networking/v1alpha3/destination_rule.pb.html @@ -1724,7 +1724,7 @@

ConnectionPoolSettings.HTTPSettings int32

Maximum number of pending HTTP requests to a destination. Default 2^32-1. -Please note that this is applicable to both HTTP1.1/ and HTTP2.

+Please note that this is applicable to both HTTP/1.1 and HTTP2.

@@ -1736,7 +1736,7 @@

ConnectionPoolSettings.HTTPSettings int32

Maximum number of requests to a backend. Default 2^32-1. -Please note that this is applicable to both HTTP1.1/ and HTTP2.

+Please note that this is applicable to both HTTP/1.1 and HTTP2.

diff --git a/networking/v1alpha3/destination_rule.proto b/networking/v1alpha3/destination_rule.proto index 80be5750d2..ab09960812 100644 --- a/networking/v1alpha3/destination_rule.proto +++ b/networking/v1alpha3/destination_rule.proto @@ -710,11 +710,11 @@ message ConnectionPoolSettings { // Settings applicable to HTTP1.1/HTTP2/GRPC connections. message HTTPSettings { // Maximum number of pending HTTP requests to a destination. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http1_max_pending_requests = 1; // Maximum number of requests to a backend. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http2_max_requests = 2; // Maximum number of requests per connection to a backend. Setting this diff --git a/networking/v1beta1/destination_rule.gen.json b/networking/v1beta1/destination_rule.gen.json index 2fbded9545..ab0f8ca8fa 100644 --- a/networking/v1beta1/destination_rule.gen.json +++ b/networking/v1beta1/destination_rule.gen.json @@ -74,12 +74,12 @@ "type": "object", "properties": { "http1MaxPendingRequests": { - "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", + "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, "http2MaxRequests": { - "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP1.1/ and HTTP2.", + "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, diff --git a/networking/v1beta1/destination_rule.pb.go b/networking/v1beta1/destination_rule.pb.go index 865cf35b6c..a0712bab25 100644 --- a/networking/v1beta1/destination_rule.pb.go +++ b/networking/v1beta1/destination_rule.pb.go @@ -2169,10 +2169,10 @@ type ConnectionPoolSettings_HTTPSettings struct { unknownFields protoimpl.UnknownFields // Maximum number of pending HTTP requests to a destination. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http1MaxPendingRequests int32 `protobuf:"varint,1,opt,name=http1_max_pending_requests,json=http1MaxPendingRequests,proto3" json:"http1_max_pending_requests,omitempty"` // Maximum number of requests to a backend. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http2MaxRequests int32 `protobuf:"varint,2,opt,name=http2_max_requests,json=http2MaxRequests,proto3" json:"http2_max_requests,omitempty"` // Maximum number of requests per connection to a backend. Setting this // parameter to 1 disables keep alive. Default 0, meaning "unlimited", diff --git a/networking/v1beta1/destination_rule.proto b/networking/v1beta1/destination_rule.proto index 344ca5f32e..2cd93c86a5 100644 --- a/networking/v1beta1/destination_rule.proto +++ b/networking/v1beta1/destination_rule.proto @@ -659,11 +659,11 @@ message ConnectionPoolSettings { // Settings applicable to HTTP1.1/HTTP2/GRPC connections. message HTTPSettings { // Maximum number of pending HTTP requests to a destination. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http1_max_pending_requests = 1; // Maximum number of requests to a backend. Default 2^32-1. - // Please note that this is applicable to both HTTP1.1/ and HTTP2. + // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http2_max_requests = 2; // Maximum number of requests per connection to a backend. Setting this From a81990bd089e1c11bcad53ddbcda84ef919b271b Mon Sep 17 00:00:00 2001 From: Rama Chavali Date: Wed, 27 Jul 2022 09:45:19 +0530 Subject: [PATCH 3/3] update docs Signed-off-by: Rama Chavali --- kubernetes/customresourcedefinitions.gen.yaml | 40 +++++++------------ networking/v1alpha3/destination_rule.gen.json | 4 +- networking/v1alpha3/destination_rule.pb.go | 7 +++- networking/v1alpha3/destination_rule.pb.html | 7 +++- networking/v1alpha3/destination_rule.proto | 7 +++- networking/v1beta1/destination_rule.gen.json | 4 +- networking/v1beta1/destination_rule.pb.go | 7 +++- networking/v1beta1/destination_rule.proto | 7 +++- 8 files changed, 43 insertions(+), 40 deletions(-) diff --git a/kubernetes/customresourcedefinitions.gen.yaml b/kubernetes/customresourcedefinitions.gen.yaml index c6753f4379..a403aa737f 100644 --- a/kubernetes/customresourcedefinitions.gen.yaml +++ b/kubernetes/customresourcedefinitions.gen.yaml @@ -197,12 +197,11 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP requests - to a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to a backend. + description: Maximum number of active requests to + a destination. format: int32 type: integer idleTimeout: @@ -448,13 +447,11 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP - requests to a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to - a backend. + description: Maximum number of active requests + to a destination. format: int32 type: integer idleTimeout: @@ -793,12 +790,10 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP requests to - a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to a backend. + description: Maximum number of active requests to a destination. format: int32 type: integer idleTimeout: @@ -1040,12 +1035,11 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP requests - to a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to a backend. + description: Maximum number of active requests to + a destination. format: int32 type: integer idleTimeout: @@ -1434,12 +1428,11 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP requests - to a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to a backend. + description: Maximum number of active requests to + a destination. format: int32 type: integer idleTimeout: @@ -1685,13 +1678,11 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP - requests to a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to - a backend. + description: Maximum number of active requests + to a destination. format: int32 type: integer idleTimeout: @@ -2030,12 +2021,10 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP requests to - a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to a backend. + description: Maximum number of active requests to a destination. format: int32 type: integer idleTimeout: @@ -2277,12 +2266,11 @@ spec: - UPGRADE type: string http1MaxPendingRequests: - description: Maximum number of pending HTTP requests - to a destination. format: int32 type: integer http2MaxRequests: - description: Maximum number of requests to a backend. + description: Maximum number of active requests to + a destination. format: int32 type: integer idleTimeout: diff --git a/networking/v1alpha3/destination_rule.gen.json b/networking/v1alpha3/destination_rule.gen.json index 072044701f..ccb4f20658 100644 --- a/networking/v1alpha3/destination_rule.gen.json +++ b/networking/v1alpha3/destination_rule.gen.json @@ -74,12 +74,12 @@ "type": "object", "properties": { "http1MaxPendingRequests": { - "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", + "description": "Maximum number of requests that will be queued while waiting for a ready connection pool connection. Default 1024. Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking under which conditions a new connection is created for HTTP2. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, "http2MaxRequests": { - "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", + "description": "Maximum number of active requests to a destination. Default 1024. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, diff --git a/networking/v1alpha3/destination_rule.pb.go b/networking/v1alpha3/destination_rule.pb.go index 95fdce698e..aa13c0ce1b 100644 --- a/networking/v1alpha3/destination_rule.pb.go +++ b/networking/v1alpha3/destination_rule.pb.go @@ -2219,10 +2219,13 @@ type ConnectionPoolSettings_HTTPSettings struct { sizeCache protoimpl.SizeCache unknownFields protoimpl.UnknownFields - // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Maximum number of requests that will be queued while waiting for + // a ready connection pool connection. Default 1024. + // Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking + // under which conditions a new connection is created for HTTP2. // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http1MaxPendingRequests int32 `protobuf:"varint,1,opt,name=http1_max_pending_requests,json=http1MaxPendingRequests,proto3" json:"http1_max_pending_requests,omitempty"` - // Maximum number of requests to a backend. Default 2^32-1. + // Maximum number of active requests to a destination. Default 1024. // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http2MaxRequests int32 `protobuf:"varint,2,opt,name=http2_max_requests,json=http2MaxRequests,proto3" json:"http2_max_requests,omitempty"` // Maximum number of requests per connection to a backend. Setting this diff --git a/networking/v1alpha3/destination_rule.pb.html b/networking/v1alpha3/destination_rule.pb.html index 15e1277a50..bc52ee5568 100644 --- a/networking/v1alpha3/destination_rule.pb.html +++ b/networking/v1alpha3/destination_rule.pb.html @@ -1723,7 +1723,10 @@

ConnectionPoolSettings.HTTPSettings http1MaxPendingRequests int32 -

Maximum number of pending HTTP requests to a destination. Default 2^32-1. +

Maximum number of requests that will be queued while waiting for +a ready connection pool connection. Default 1024. +Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking +under which conditions a new connection is created for HTTP2. Please note that this is applicable to both HTTP/1.1 and HTTP2.

@@ -1735,7 +1738,7 @@

ConnectionPoolSettings.HTTPSettings http2MaxRequests int32 -

Maximum number of requests to a backend. Default 2^32-1. +

Maximum number of active requests to a destination. Default 1024. Please note that this is applicable to both HTTP/1.1 and HTTP2.

diff --git a/networking/v1alpha3/destination_rule.proto b/networking/v1alpha3/destination_rule.proto index ab09960812..68b7e4910d 100644 --- a/networking/v1alpha3/destination_rule.proto +++ b/networking/v1alpha3/destination_rule.proto @@ -709,11 +709,14 @@ message ConnectionPoolSettings { // Settings applicable to HTTP1.1/HTTP2/GRPC connections. message HTTPSettings { - // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Maximum number of requests that will be queued while waiting for + // a ready connection pool connection. Default 1024. + // Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking + // under which conditions a new connection is created for HTTP2. // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http1_max_pending_requests = 1; - // Maximum number of requests to a backend. Default 2^32-1. + // Maximum number of active requests to a destination. Default 1024. // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http2_max_requests = 2; diff --git a/networking/v1beta1/destination_rule.gen.json b/networking/v1beta1/destination_rule.gen.json index ab0f8ca8fa..030400f974 100644 --- a/networking/v1beta1/destination_rule.gen.json +++ b/networking/v1beta1/destination_rule.gen.json @@ -74,12 +74,12 @@ "type": "object", "properties": { "http1MaxPendingRequests": { - "description": "Maximum number of pending HTTP requests to a destination. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", + "description": "Maximum number of requests that will be queued while waiting for a ready connection pool connection. Default 1024. Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking under which conditions a new connection is created for HTTP2. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, "http2MaxRequests": { - "description": "Maximum number of requests to a backend. Default 2^32-1. Please note that this is applicable to both HTTP/1.1 and HTTP2.", + "description": "Maximum number of active requests to a destination. Default 1024. Please note that this is applicable to both HTTP/1.1 and HTTP2.", "type": "integer", "format": "int32" }, diff --git a/networking/v1beta1/destination_rule.pb.go b/networking/v1beta1/destination_rule.pb.go index a0712bab25..dd41540db5 100644 --- a/networking/v1beta1/destination_rule.pb.go +++ b/networking/v1beta1/destination_rule.pb.go @@ -2168,10 +2168,13 @@ type ConnectionPoolSettings_HTTPSettings struct { sizeCache protoimpl.SizeCache unknownFields protoimpl.UnknownFields - // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Maximum number of requests that will be queued while waiting for + // a ready connection pool connection. Default 1024. + // Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking + // under which conditions a new connection is created for HTTP2. // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http1MaxPendingRequests int32 `protobuf:"varint,1,opt,name=http1_max_pending_requests,json=http1MaxPendingRequests,proto3" json:"http1_max_pending_requests,omitempty"` - // Maximum number of requests to a backend. Default 2^32-1. + // Maximum number of active requests to a destination. Default 1024. // Please note that this is applicable to both HTTP/1.1 and HTTP2. Http2MaxRequests int32 `protobuf:"varint,2,opt,name=http2_max_requests,json=http2MaxRequests,proto3" json:"http2_max_requests,omitempty"` // Maximum number of requests per connection to a backend. Setting this diff --git a/networking/v1beta1/destination_rule.proto b/networking/v1beta1/destination_rule.proto index 2cd93c86a5..1cd55a9aa3 100644 --- a/networking/v1beta1/destination_rule.proto +++ b/networking/v1beta1/destination_rule.proto @@ -658,11 +658,14 @@ message ConnectionPoolSettings { // Settings applicable to HTTP1.1/HTTP2/GRPC connections. message HTTPSettings { - // Maximum number of pending HTTP requests to a destination. Default 2^32-1. + // Maximum number of requests that will be queued while waiting for + // a ready connection pool connection. Default 1024. + // Refer to https://www.envoyproxy.io/docs/envoy/latest/intro/arch_overview/upstream/circuit_breaking + // under which conditions a new connection is created for HTTP2. // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http1_max_pending_requests = 1; - // Maximum number of requests to a backend. Default 2^32-1. + // Maximum number of active requests to a destination. Default 1024. // Please note that this is applicable to both HTTP/1.1 and HTTP2. int32 http2_max_requests = 2;