Added max_requests_per_connection for downstream connection. (#14936)

Signed-off-by: Manish Kumar <manish.kumar1@india.nec.com>

Mirrored from https://github.com/envoyproxy/envoy @ 364c821f85aff8a42b0bf7fc0e3e5e4b6c98c86c

envoy/config/cluster/v3/cluster.proto

@@ -746,7 +746,11 @@ message Cluster {
   // is respected by both the HTTP/1.1 and HTTP/2 connection pool
   // implementations. If not specified, there is no limit. Setting this
   // parameter to 1 will effectively disable keep alive.
-  google.protobuf.UInt32Value max_requests_per_connection = 9;
+  //
+  // .. attention::
+  //   This field has been deprecated in favor of the :ref:`max_requests_per_connection <envoy_v3_api_field_config.core.v3.HttpProtocolOptions.max_requests_per_connection>` field.
+  google.protobuf.UInt32Value max_requests_per_connection = 9
+      [deprecated = true, (envoy.annotations.deprecated_at_minor_version) = "3.0"];
 
   // Optional :ref:`circuit breaking <arch_overview_circuit_break>` for the cluster.
   CircuitBreakers circuit_breakers = 10;

envoy/config/cluster/v4alpha/cluster.proto

@@ -631,11 +631,12 @@ message Cluster {
         [(validate.rules).double = {lte: 3.0 gte: 1.0}];
   }
 
-  reserved 12, 15, 7, 11, 35, 46, 29, 13, 14, 18, 45, 26, 47;
+  reserved 12, 15, 7, 11, 35, 9, 46, 29, 13, 14, 18, 45, 26, 47;
 
-  reserved "hosts", "tls_context", "extension_protocol_options", "upstream_http_protocol_options",
-      "common_http_protocol_options", "http_protocol_options", "http2_protocol_options",
-      "dns_resolvers", "use_tcp_for_dns_lookups", "protocol_selection", "track_timeout_budgets";
+  reserved "hosts", "tls_context", "extension_protocol_options", "max_requests_per_connection",
+      "upstream_http_protocol_options", "common_http_protocol_options", "http_protocol_options",
+      "http2_protocol_options", "dns_resolvers", "use_tcp_for_dns_lookups", "protocol_selection",
+      "track_timeout_budgets";
 
   // Configuration to use different transport sockets for different endpoints.
   // The entry of *envoy.transport_socket_match* in the
@@ -751,12 +752,6 @@ message Cluster {
   // members will be considered healthy at all times.
   repeated core.v4alpha.HealthCheck health_checks = 8;
 
-  // Optional maximum requests for a single upstream connection. This parameter
-  // is respected by both the HTTP/1.1 and HTTP/2 connection pool
-  // implementations. If not specified, there is no limit. Setting this
-  // parameter to 1 will effectively disable keep alive.
-  google.protobuf.UInt32Value max_requests_per_connection = 9;
-
   // Optional :ref:`circuit breaking <arch_overview_circuit_break>` for the cluster.
   CircuitBreakers circuit_breakers = 10;
 

envoy/config/core/v3/protocol.proto

@@ -93,7 +93,7 @@ message AlternateProtocolsCacheOptions {
   google.protobuf.UInt32Value max_entries = 2 [(validate.rules).uint32 = {gt: 0}];
 }
 
-// [#next-free-field: 6]
+// [#next-free-field: 7]
 message HttpProtocolOptions {
   option (udpa.annotations.versioning).previous_message_type =
       "envoy.api.v2.core.HttpProtocolOptions";
@@ -157,6 +157,12 @@ message HttpProtocolOptions {
   // If this setting is not specified, the value defaults to ALLOW.
   // Note: upstream responses are not affected by this setting.
   HeadersWithUnderscoresAction headers_with_underscores_action = 5;
+
+  // Optional maximum requests for both upstream and downstream connections.
+  // If not specified, there is no limit.
+  // Setting this parameter to 1 will effectively disable keep alive.
+  // For HTTP/2 and HTTP/3, due to concurrent stream processing, the limit is approximate.
+  google.protobuf.UInt32Value max_requests_per_connection = 6;
 }
 
 // [#next-free-field: 8]

envoy/config/core/v4alpha/protocol.proto

@@ -98,7 +98,7 @@ message AlternateProtocolsCacheOptions {
   google.protobuf.UInt32Value max_entries = 2 [(validate.rules).uint32 = {gt: 0}];
 }
 
-// [#next-free-field: 6]
+// [#next-free-field: 7]
 message HttpProtocolOptions {
   option (udpa.annotations.versioning).previous_message_type =
       "envoy.config.core.v3.HttpProtocolOptions";
@@ -162,6 +162,12 @@ message HttpProtocolOptions {
   // If this setting is not specified, the value defaults to ALLOW.
   // Note: upstream responses are not affected by this setting.
   HeadersWithUnderscoresAction headers_with_underscores_action = 5;
+
+  // Optional maximum requests for both upstream and downstream connections.
+  // If not specified, there is no limit.
+  // Setting this parameter to 1 will effectively disable keep alive.
+  // For HTTP/2 and HTTP/3, due to concurrent stream processing, the limit is approximate.
+  google.protobuf.UInt32Value max_requests_per_connection = 6;
 }
 
 // [#next-free-field: 8]