Merge pull request #2388 from frankbu/gep-1742-update

Update GEP-1742 with latest comments from go apis

Author: k8s-ci-robot

geps/gep-1742.md

@@ -390,11 +390,14 @@ type HTTPRouteRule struct {
 }
 
 // HTTPRouteTimeouts defines timeouts that can be configured for an HTTPRoute.
-// Timeout values are formatted like 1h/1m/1s/1ms, as specified in GEP-2257,
-// and MUST BE >= 1ms or "0s" to disable (no timeout).
+// Timeout values are represented with Gateway API Duration formatting.
+// Specifying a zero value such as "0s" is interpreted as no timeout.
+//
+// +kubebuilder:validation:XValidation:message="backendRequest timeout cannot be longer than request timeout",rule="!(has(self.request) && has(self.backendRequest) && duration(self.request) != duration('0s') && duration(self.backendRequest) > duration(self.request))"
 type HTTPRouteTimeouts struct {
-	// Request specifies the duration for processing an HTTP client request after which the
-	// gateway will time out if unable to send a response.
+	// Request specifies the maximum duration for a gateway to respond to an HTTP request.
+	// If the gateway has not been able to respond before this deadline is met, the gateway
+	// MUST return a timeout error.
 	//
 	// For example, setting the `rules.timeouts.request` field to the value `10s` in an
 	// `HTTPRoute` will cause a timeout if a client request is taking longer than 10 seconds
@@ -405,19 +408,19 @@ type HTTPRouteTimeouts struct {
 	// request stream has been received instead of immediately after the transaction is
 	// initiated by the client.
 	//
-	// When this field is unspecified, request timeout behavior is implementation-dependent.
+	// When this field is unspecified, request timeout behavior is implementation-specific.
 	//
 	// Support: Extended
 	//
 	// +optional
 	Request *Duration `json:"request,omitempty"`
 
 	// BackendRequest specifies a timeout for an individual request from the gateway
-	// to a backend service. This covers the time from when the request first starts being
+	// to a backend. This covers the time from when the request first starts being
 	// sent from the gateway to when the full response has been received from the backend.
 	//
 	// An entire client HTTP transaction with a gateway, covered by the Request timeout,
-	// may result in more than one call from the gateway to the destination backend service,
+	// may result in more than one call from the gateway to the destination backend,
 	// for example, if automatic retries are supported.
 	//
 	// Because the Request timeout encompasses the BackendRequest timeout, the value of