Backport of Adding support for idleTimeout in Service Router spec int…

…o release/1.0.x (#2198)

* backport of commit b990087

* backport of commit fa3ca5f

* backport of commit 19f05a2

* backport of commit ceeb4b7

---------

Co-authored-by: Kaustubh Phatak <kaustubh.phatak@machinify.com>

charts/consul/templates/crd-servicerouters.yaml

@@ -68,6 +68,10 @@ spec:
                       description: Destination controls how to proxy the matching
                         request(s) to a service.
                       properties:
+                        idleTimeout:
+                          description: IdleTimeout is total amount of time permitted
+                            for the request stream to be idle.
+                          type: string
                         namespace:
                           description: Namespace is the Consul namespace to resolve
                             the service from instead of the current namespace. If

control-plane/api/v1alpha1/servicerouter_types.go

@@ -137,6 +137,9 @@ type ServiceRouteDestination struct {
 	// This requires that either match.http.pathPrefix or match.http.pathExact
 	// be configured on this route.
 	PrefixRewrite string `json:"prefixRewrite,omitempty"`
+	// IdleTimeout is total amount of time permitted
+	// for the request stream to be idle.
+	IdleTimeout metav1.Duration `json:"idleTimeout,omitempty"`
 	// RequestTimeout is the total amount of time permitted for the entire
 	// downstream request (and retries) to be processed.
 	RequestTimeout metav1.Duration `json:"requestTimeout,omitempty"`
@@ -334,6 +337,7 @@ func (in *ServiceRouteDestination) toConsul() *capi.ServiceRouteDestination {
 		Namespace:             in.Namespace,
 		Partition:             in.Partition,
 		PrefixRewrite:         in.PrefixRewrite,
+		IdleTimeout:           in.IdleTimeout.Duration,
 		RequestTimeout:        in.RequestTimeout.Duration,
 		NumRetries:            in.NumRetries,
 		RetryOnConnectFailure: in.RetryOnConnectFailure,

control-plane/api/v1alpha1/servicerouter_types_test.go

@@ -79,6 +79,7 @@ func TestServiceRouter_MatchesConsul(t *testing.T) {
 								ServiceSubset:         "serviceSubset",
 								Namespace:             "namespace",
 								PrefixRewrite:         "prefixRewrite",
+								IdleTimeout:           metav1.Duration{Duration: 1 * time.Second},
 								RequestTimeout:        metav1.Duration{Duration: 1 * time.Second},
 								NumRetries:            1,
 								RetryOnConnectFailure: true,
@@ -155,6 +156,7 @@ func TestServiceRouter_MatchesConsul(t *testing.T) {
 							ServiceSubset:         "serviceSubset",
 							Namespace:             "namespace",
 							PrefixRewrite:         "prefixRewrite",
+							IdleTimeout:           1 * time.Second,
 							RequestTimeout:        1 * time.Second,
 							NumRetries:            1,
 							RetryOnConnectFailure: true,
@@ -280,6 +282,7 @@ func TestServiceRouter_ToConsul(t *testing.T) {
 								ServiceSubset:         "serviceSubset",
 								Namespace:             "namespace",
 								PrefixRewrite:         "prefixRewrite",
+								IdleTimeout:           metav1.Duration{Duration: 1 * time.Second},
 								RequestTimeout:        metav1.Duration{Duration: 1 * time.Second},
 								NumRetries:            1,
 								RetryOnConnectFailure: true,
@@ -356,6 +359,7 @@ func TestServiceRouter_ToConsul(t *testing.T) {
 							ServiceSubset:         "serviceSubset",
 							Namespace:             "namespace",
 							PrefixRewrite:         "prefixRewrite",
+							IdleTimeout:           1 * time.Second,
 							RequestTimeout:        1 * time.Second,
 							NumRetries:            1,
 							RetryOnConnectFailure: true,
@@ -714,7 +718,7 @@ func TestServiceRouter_Validate(t *testing.T) {
 			},
 			namespacesEnabled: false,
 			expectedErrMsgs: []string{
-				`servicerouter.consul.hashicorp.com "foo" is invalid: spec.routes[0]: Invalid value: "{\"match\":{\"http\":{}},\"destination\":{\"prefixRewrite\":\"prefixRewrite\",\"requestTimeout\":\"0s\"}}": destination.prefixRewrite requires that either match.http.pathPrefix or match.http.pathExact be configured on this route`,
+				`servicerouter.consul.hashicorp.com "foo" is invalid: spec.routes[0]: Invalid value: "{\"match\":{\"http\":{}},\"destination\":{\"prefixRewrite\":\"prefixRewrite\",\"idleTimeout\":\"0s\",\"requestTimeout\":\"0s\"}}": destination.prefixRewrite requires that either match.http.pathPrefix or match.http.pathExact be configured on this route`,
 			},
 		},
 		"namespaces disabled: single destination namespace specified": {

control-plane/config/crd/bases/consul.hashicorp.com_servicerouters.yaml

@@ -61,6 +61,10 @@ spec:
                       description: Destination controls how to proxy the matching
                         request(s) to a service.
                       properties:
+                        idleTimeout:
+                          description: IdleTimeout is total amount of time permitted
+                            for the request stream to be idle.
+                          type: string
                         namespace:
                           description: Namespace is the Consul namespace to resolve
                             the service from instead of the current namespace. If