From 8feb49876771c62d10b81232c8cd63b873eb1bcb Mon Sep 17 00:00:00 2001 From: Rob Scott Date: Thu, 17 Nov 2022 00:19:44 +0000 Subject: [PATCH] Some small cleanup from v0.6.0 review --- apis/v1alpha2/gateway_types.go | 10 +- apis/v1beta1/gateway_types.go | 8 +- apis/v1beta1/httproute_types.go | 31 +++-- apis/v1beta1/referencegrant_types.go | 8 +- apis/v1beta1/shared_types.go | 6 +- .../gateway.networking.k8s.io_grpcroutes.yaml | 4 +- .../gateway.networking.k8s.io_httproutes.yaml | 124 +++++++++--------- ...way.networking.k8s.io_referencegrants.yaml | 16 +-- .../gateway.networking.k8s.io_tlsroutes.yaml | 4 +- .../gateway.networking.k8s.io_httproutes.yaml | 124 +++++++++--------- ...way.networking.k8s.io_referencegrants.yaml | 16 +-- 11 files changed, 180 insertions(+), 171 deletions(-) diff --git a/apis/v1alpha2/gateway_types.go b/apis/v1alpha2/gateway_types.go index ea4db11d40..ef5f5739fb 100644 --- a/apis/v1alpha2/gateway_types.go +++ b/apis/v1alpha2/gateway_types.go @@ -60,10 +60,10 @@ type GatewaySpec = v1beta1.GatewaySpec type Listener = v1beta1.Listener // ProtocolType defines the application protocol accepted by a Listener. -// Implementations are not required to accept all the defined protocols. -// If an implementation does not support a specified protocol, it -// should set the "Accepted" condition to "False" for the affected Listener with -// a reason of "UnsupportedProtocol". +// Implementations are not required to accept all the defined protocols. If an +// implementation does not support a specified protocol, it MUST set the +// "Accepted" condition to False for the affected Listener with a reason of +// "UnsupportedProtocol". // // Core ProtocolType values are listed in the table below. // @@ -470,9 +470,11 @@ const ( // This reason is used with the "ResolvedRefs" condition when the // Listener has a TLS configuration with at least one TLS CertificateRef // that is invalid or does not exist. + // // A CertificateRef is considered invalid when it refers to a nonexistent // or unsupported resource or kind, or when the data within that resource // is malformed. + // // This reason must be used only when the reference is allowed, either by // referencing an object in the same namespace as the Gateway, or when // a cross-namespace reference has been explicitly allowed by a ReferenceGrant. diff --git a/apis/v1beta1/gateway_types.go b/apis/v1beta1/gateway_types.go index bf26bacc36..a4e50d4a95 100644 --- a/apis/v1beta1/gateway_types.go +++ b/apis/v1beta1/gateway_types.go @@ -237,10 +237,10 @@ type Listener struct { } // ProtocolType defines the application protocol accepted by a Listener. -// Implementations are not required to accept all the defined protocols. -// If an implementation does not support a specified protocol, it -// should set the "Accepted" condition for the affected Listener with -// a reason of "UnsupportedProtocol". +// Implementations are not required to accept all the defined protocols. If an +// implementation does not support a specified protocol, it MUST set the +// "Accepted" condition to False for the affected Listener with a reason of +// "UnsupportedProtocol". // // Core ProtocolType values are listed in the table below. // diff --git a/apis/v1beta1/httproute_types.go b/apis/v1beta1/httproute_types.go index f31622fc79..bda917776d 100644 --- a/apis/v1beta1/httproute_types.go +++ b/apis/v1beta1/httproute_types.go @@ -439,13 +439,14 @@ type HTTPQueryParamMatch struct { // entry with an equivalent name MUST be considered for a match. Subsequent // entries with an equivalent query param name MUST be ignored. // - // If a query param is repeated in an HTTP request, the behavior - // is purposely left undefined, since different data planes have different - // capabilities. However, it's *recommended* that implementations should + // If a query param is repeated in an HTTP request, the behavior is + // purposely left undefined, since different data planes have different + // capabilities. However, it is *recommended* that implementations should // match against the first value of the param if the data plane supports it, // as this behavior is expected in other load balancing contexts outside of // the Gateway API. - // Users should not route traffic based on repeated query params to guard + // + // Users SHOULD NOT route traffic based on repeated query params to guard // themselves against potential differences in the implementations. // // +kubebuilder:validation:MinLength=1 @@ -722,11 +723,10 @@ type HTTPHeader struct { // HTTPHeaderFilter defines a filter that modifies the headers of an HTTP // request or response. Only one action for a given header name is permitted. -// Filters specifying multiple actions of the same or different type for -// any one header name are invalid and will be rejected by the webhook if -// installed. Configuration to set or add multiple values for a -// header must use RFC 7230 header value formatting, separating each value with -// a comma. +// Filters specifying multiple actions of the same or different type for any one +// header name are invalid and will be rejected by the webhook if installed. +// Configuration to set or add multiple values for a header must use RFC 7230 +// header value formatting, separating each value with a comma. type HTTPHeaderFilter struct { // Set overwrites the request with the given header (name, value) // before the action. @@ -861,11 +861,8 @@ type HTTPPathModifier struct { // HTTPRequestRedirect defines a filter that redirects a request. This filter // MUST NOT be used on the same Route rule as a HTTPURLRewrite filter. type HTTPRequestRedirectFilter struct { - // Scheme is the scheme to be used in the value of the `Location` - // header in the response. - // When empty, the scheme of the request is used. - // - // Support: Extended + // Scheme is the scheme to be used in the value of the `Location` header in + // the response. When empty, the scheme of the request is used. // // Note that values may be added to this enum, implementations // must ensure that unknown values will not cause a crash. @@ -874,6 +871,8 @@ type HTTPRequestRedirectFilter struct { // Accepted Condition for the Route to `status: False`, with a // Reason of `UnsupportedValue`. // + // Support: Extended + // // +optional // +kubebuilder:validation:Enum=http;https Scheme *string `json:"scheme,omitempty"` @@ -908,8 +907,6 @@ type HTTPRequestRedirectFilter struct { // StatusCode is the HTTP status code to be used in response. // - // Support: Core - // // Note that values may be added to this enum, implementations // must ensure that unknown values will not cause a crash. // @@ -917,6 +914,8 @@ type HTTPRequestRedirectFilter struct { // Accepted Condition for the Route to `status: False`, with a // Reason of `UnsupportedValue`. // + // Support: Core + // // +optional // +kubebuilder:default=302 // +kubebuilder:validation:Enum=301;302 diff --git a/apis/v1beta1/referencegrant_types.go b/apis/v1beta1/referencegrant_types.go index 634a8f4d19..abc6ca3475 100644 --- a/apis/v1beta1/referencegrant_types.go +++ b/apis/v1beta1/referencegrant_types.go @@ -63,9 +63,9 @@ type ReferenceGrantList struct { // for Gateway API. type ReferenceGrantSpec struct { // From describes the trusted namespaces and kinds that can reference the - // resources described in "To". Each entry in this list must be considered + // resources described in "To". Each entry in this list MUST be considered // to be an additional place that references can be valid from, or to put - // this another way, entries must be combined using OR. + // this another way, entries MUST be combined using OR. // // Support: Core // @@ -74,9 +74,9 @@ type ReferenceGrantSpec struct { From []ReferenceGrantFrom `json:"from"` // To describes the resources that may be referenced by the resources - // described in "From". Each entry in this list must be considered to be an + // described in "From". Each entry in this list MUST be considered to be an // additional place that references can be valid to, or to put this another - // way, entries must be combined using OR. + // way, entries MUST be combined using OR. // // Support: Core // diff --git a/apis/v1beta1/shared_types.go b/apis/v1beta1/shared_types.go index daf0edd057..997837cad8 100644 --- a/apis/v1beta1/shared_types.go +++ b/apis/v1beta1/shared_types.go @@ -351,9 +351,9 @@ type RouteStatus struct { // Hostname is the fully qualified domain name of a network host. This matches // the RFC 1123 definition of a hostname with 2 notable exceptions: // -// 1. IPs are not allowed. -// 2. A hostname may be prefixed with a wildcard label (`*.`). The wildcard -// label must appear by itself as the first label. +// 1. IPs are not allowed. +// 2. A hostname may be prefixed with a wildcard label (`*.`). The wildcard +// label must appear by itself as the first label. // // Hostname can be "precise" which is a domain name without the terminating // dot of a network host (e.g. "foo.example.com") or "wildcard", which is a diff --git a/config/crd/experimental/gateway.networking.k8s.io_grpcroutes.yaml b/config/crd/experimental/gateway.networking.k8s.io_grpcroutes.yaml index 4931605260..aa37bce7ba 100644 --- a/config/crd/experimental/gateway.networking.k8s.io_grpcroutes.yaml +++ b/config/crd/experimental/gateway.networking.k8s.io_grpcroutes.yaml @@ -109,8 +109,8 @@ spec: items: description: "Hostname is the fully qualified domain name of a network host. This matches the RFC 1123 definition of a hostname with - 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname - may be prefixed with a wildcard label (`*.`). The wildcard label + 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname + may be prefixed with a wildcard label (`*.`). The wildcard label must appear by itself as the first label. \n Hostname can be \"precise\" which is a domain name without the terminating dot of a network host (e.g. \"foo.example.com\") or \"wildcard\", which is a domain diff --git a/config/crd/experimental/gateway.networking.k8s.io_httproutes.yaml b/config/crd/experimental/gateway.networking.k8s.io_httproutes.yaml index 3593630332..614474e835 100644 --- a/config/crd/experimental/gateway.networking.k8s.io_httproutes.yaml +++ b/config/crd/experimental/gateway.networking.k8s.io_httproutes.yaml @@ -90,8 +90,8 @@ spec: items: description: "Hostname is the fully qualified domain name of a network host. This matches the RFC 1123 definition of a hostname with - 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname - may be prefixed with a wildcard label (`*.`). The wildcard label + 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname + may be prefixed with a wildcard label (`*.`). The wildcard label must appear by itself as the first label. \n Hostname can be \"precise\" which is a domain name without the terminating dot of a network host (e.g. \"foo.example.com\") or \"wildcard\", which is a domain @@ -571,13 +571,13 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request - is used. \n Support: Extended \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause - a crash. \n Unknown values here must result - in the implementation setting the Accepted - Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + is used. \n Note that values may be added + to this enum, implementations must ensure + that unknown values will not cause a crash. + \n Unknown values here must result in the + implementation setting the Accepted Condition + for the Route to `status: False`, with a Reason + of `UnsupportedValue`. \n Support: Extended" enum: - http - https @@ -585,13 +585,14 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status - code to be used in response. \n Support: Core - \n Note that values may be added to this enum, - implementations must ensure that unknown values - will not cause a crash. \n Unknown values - here must result in the implementation setting - the Accepted Condition for the Route to `status: - False`, with a Reason of `UnsupportedValue`." + code to be used in response. \n Note that + values may be added to this enum, implementations + must ensure that unknown values will not cause + a crash. \n Unknown values here must result + in the implementation setting the Accepted + Condition for the Route to `status: False`, + with a Reason of `UnsupportedValue`. \n Support: + Core" enum: - 301 - 302 @@ -1193,12 +1194,12 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request is used. \n - Support: Extended \n Note that values may be added - to this enum, implementations must ensure that unknown - values will not cause a crash. \n Unknown values - here must result in the implementation setting the - Accepted Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + Note that values may be added to this enum, implementations + must ensure that unknown values will not cause a + crash. \n Unknown values here must result in the + implementation setting the Accepted Condition for + the Route to `status: False`, with a Reason of `UnsupportedValue`. + \n Support: Extended" enum: - http - https @@ -1206,12 +1207,13 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status code to - be used in response. \n Support: Core \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause a - crash. \n Unknown values here must result in the - implementation setting the Accepted Condition for - the Route to `status: False`, with a Reason of `UnsupportedValue`." + be used in response. \n Note that values may be + added to this enum, implementations must ensure + that unknown values will not cause a crash. \n Unknown + values here must result in the implementation setting + the Accepted Condition for the Route to `status: + False`, with a Reason of `UnsupportedValue`. \n + Support: Core" enum: - 301 - 302 @@ -1578,12 +1580,12 @@ spec: be ignored. \n If a query param is repeated in an HTTP request, the behavior is purposely left undefined, since different data planes have different - capabilities. However, it's *recommended* that + capabilities. However, it is *recommended* that implementations should match against the first value of the param if the data plane supports it, as this behavior is expected in other load balancing contexts outside of the Gateway API. - Users should not route traffic based on repeated + \n Users SHOULD NOT route traffic based on repeated query params to guard themselves against potential differences in the implementations." maxLength: 256 @@ -1932,8 +1934,8 @@ spec: items: description: "Hostname is the fully qualified domain name of a network host. This matches the RFC 1123 definition of a hostname with - 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname - may be prefixed with a wildcard label (`*.`). The wildcard label + 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname + may be prefixed with a wildcard label (`*.`). The wildcard label must appear by itself as the first label. \n Hostname can be \"precise\" which is a domain name without the terminating dot of a network host (e.g. \"foo.example.com\") or \"wildcard\", which is a domain @@ -2413,13 +2415,13 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request - is used. \n Support: Extended \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause - a crash. \n Unknown values here must result - in the implementation setting the Accepted - Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + is used. \n Note that values may be added + to this enum, implementations must ensure + that unknown values will not cause a crash. + \n Unknown values here must result in the + implementation setting the Accepted Condition + for the Route to `status: False`, with a Reason + of `UnsupportedValue`. \n Support: Extended" enum: - http - https @@ -2427,13 +2429,14 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status - code to be used in response. \n Support: Core - \n Note that values may be added to this enum, - implementations must ensure that unknown values - will not cause a crash. \n Unknown values - here must result in the implementation setting - the Accepted Condition for the Route to `status: - False`, with a Reason of `UnsupportedValue`." + code to be used in response. \n Note that + values may be added to this enum, implementations + must ensure that unknown values will not cause + a crash. \n Unknown values here must result + in the implementation setting the Accepted + Condition for the Route to `status: False`, + with a Reason of `UnsupportedValue`. \n Support: + Core" enum: - 301 - 302 @@ -3035,12 +3038,12 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request is used. \n - Support: Extended \n Note that values may be added - to this enum, implementations must ensure that unknown - values will not cause a crash. \n Unknown values - here must result in the implementation setting the - Accepted Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + Note that values may be added to this enum, implementations + must ensure that unknown values will not cause a + crash. \n Unknown values here must result in the + implementation setting the Accepted Condition for + the Route to `status: False`, with a Reason of `UnsupportedValue`. + \n Support: Extended" enum: - http - https @@ -3048,12 +3051,13 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status code to - be used in response. \n Support: Core \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause a - crash. \n Unknown values here must result in the - implementation setting the Accepted Condition for - the Route to `status: False`, with a Reason of `UnsupportedValue`." + be used in response. \n Note that values may be + added to this enum, implementations must ensure + that unknown values will not cause a crash. \n Unknown + values here must result in the implementation setting + the Accepted Condition for the Route to `status: + False`, with a Reason of `UnsupportedValue`. \n + Support: Core" enum: - 301 - 302 @@ -3420,12 +3424,12 @@ spec: be ignored. \n If a query param is repeated in an HTTP request, the behavior is purposely left undefined, since different data planes have different - capabilities. However, it's *recommended* that + capabilities. However, it is *recommended* that implementations should match against the first value of the param if the data plane supports it, as this behavior is expected in other load balancing contexts outside of the Gateway API. - Users should not route traffic based on repeated + \n Users SHOULD NOT route traffic based on repeated query params to guard themselves against potential differences in the implementations." maxLength: 256 diff --git a/config/crd/experimental/gateway.networking.k8s.io_referencegrants.yaml b/config/crd/experimental/gateway.networking.k8s.io_referencegrants.yaml index 8bd6c852ca..234631a6c4 100644 --- a/config/crd/experimental/gateway.networking.k8s.io_referencegrants.yaml +++ b/config/crd/experimental/gateway.networking.k8s.io_referencegrants.yaml @@ -56,8 +56,8 @@ spec: from: description: "From describes the trusted namespaces and kinds that can reference the resources described in \"To\". Each entry in this - list must be considered to be an additional place that references - can be valid from, or to put this another way, entries must be combined + list MUST be considered to be an additional place that references + can be valid from, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantFrom describes trusted namespaces and @@ -97,9 +97,9 @@ spec: type: array to: description: "To describes the resources that may be referenced by - the resources described in \"From\". Each entry in this list must + the resources described in \"From\". Each entry in this list MUST be considered to be an additional place that references can be valid - to, or to put this another way, entries must be combined using OR. + to, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantTo describes what Kinds are allowed as @@ -179,8 +179,8 @@ spec: from: description: "From describes the trusted namespaces and kinds that can reference the resources described in \"To\". Each entry in this - list must be considered to be an additional place that references - can be valid from, or to put this another way, entries must be combined + list MUST be considered to be an additional place that references + can be valid from, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantFrom describes trusted namespaces and @@ -220,9 +220,9 @@ spec: type: array to: description: "To describes the resources that may be referenced by - the resources described in \"From\". Each entry in this list must + the resources described in \"From\". Each entry in this list MUST be considered to be an additional place that references can be valid - to, or to put this another way, entries must be combined using OR. + to, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantTo describes what Kinds are allowed as diff --git a/config/crd/experimental/gateway.networking.k8s.io_tlsroutes.yaml b/config/crd/experimental/gateway.networking.k8s.io_tlsroutes.yaml index 6ca087585c..8c48c3fcf9 100644 --- a/config/crd/experimental/gateway.networking.k8s.io_tlsroutes.yaml +++ b/config/crd/experimental/gateway.networking.k8s.io_tlsroutes.yaml @@ -75,8 +75,8 @@ spec: items: description: "Hostname is the fully qualified domain name of a network host. This matches the RFC 1123 definition of a hostname with - 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname - may be prefixed with a wildcard label (`*.`). The wildcard label + 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname + may be prefixed with a wildcard label (`*.`). The wildcard label must appear by itself as the first label. \n Hostname can be \"precise\" which is a domain name without the terminating dot of a network host (e.g. \"foo.example.com\") or \"wildcard\", which is a domain diff --git a/config/crd/standard/gateway.networking.k8s.io_httproutes.yaml b/config/crd/standard/gateway.networking.k8s.io_httproutes.yaml index 4171ccbe17..779a6b7e03 100644 --- a/config/crd/standard/gateway.networking.k8s.io_httproutes.yaml +++ b/config/crd/standard/gateway.networking.k8s.io_httproutes.yaml @@ -90,8 +90,8 @@ spec: items: description: "Hostname is the fully qualified domain name of a network host. This matches the RFC 1123 definition of a hostname with - 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname - may be prefixed with a wildcard label (`*.`). The wildcard label + 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname + may be prefixed with a wildcard label (`*.`). The wildcard label must appear by itself as the first label. \n Hostname can be \"precise\" which is a domain name without the terminating dot of a network host (e.g. \"foo.example.com\") or \"wildcard\", which is a domain @@ -494,13 +494,13 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request - is used. \n Support: Extended \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause - a crash. \n Unknown values here must result - in the implementation setting the Accepted - Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + is used. \n Note that values may be added + to this enum, implementations must ensure + that unknown values will not cause a crash. + \n Unknown values here must result in the + implementation setting the Accepted Condition + for the Route to `status: False`, with a Reason + of `UnsupportedValue`. \n Support: Extended" enum: - http - https @@ -508,13 +508,14 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status - code to be used in response. \n Support: Core - \n Note that values may be added to this enum, - implementations must ensure that unknown values - will not cause a crash. \n Unknown values - here must result in the implementation setting - the Accepted Condition for the Route to `status: - False`, with a Reason of `UnsupportedValue`." + code to be used in response. \n Note that + values may be added to this enum, implementations + must ensure that unknown values will not cause + a crash. \n Unknown values here must result + in the implementation setting the Accepted + Condition for the Route to `status: False`, + with a Reason of `UnsupportedValue`. \n Support: + Core" enum: - 301 - 302 @@ -898,12 +899,12 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request is used. \n - Support: Extended \n Note that values may be added - to this enum, implementations must ensure that unknown - values will not cause a crash. \n Unknown values - here must result in the implementation setting the - Accepted Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + Note that values may be added to this enum, implementations + must ensure that unknown values will not cause a + crash. \n Unknown values here must result in the + implementation setting the Accepted Condition for + the Route to `status: False`, with a Reason of `UnsupportedValue`. + \n Support: Extended" enum: - http - https @@ -911,12 +912,13 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status code to - be used in response. \n Support: Core \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause a - crash. \n Unknown values here must result in the - implementation setting the Accepted Condition for - the Route to `status: False`, with a Reason of `UnsupportedValue`." + be used in response. \n Note that values may be + added to this enum, implementations must ensure + that unknown values will not cause a crash. \n Unknown + values here must result in the implementation setting + the Accepted Condition for the Route to `status: + False`, with a Reason of `UnsupportedValue`. \n + Support: Core" enum: - 301 - 302 @@ -1123,12 +1125,12 @@ spec: be ignored. \n If a query param is repeated in an HTTP request, the behavior is purposely left undefined, since different data planes have different - capabilities. However, it's *recommended* that + capabilities. However, it is *recommended* that implementations should match against the first value of the param if the data plane supports it, as this behavior is expected in other load balancing contexts outside of the Gateway API. - Users should not route traffic based on repeated + \n Users SHOULD NOT route traffic based on repeated query params to guard themselves against potential differences in the implementations." maxLength: 256 @@ -1449,8 +1451,8 @@ spec: items: description: "Hostname is the fully qualified domain name of a network host. This matches the RFC 1123 definition of a hostname with - 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname - may be prefixed with a wildcard label (`*.`). The wildcard label + 2 notable exceptions: \n 1. IPs are not allowed. 2. A hostname + may be prefixed with a wildcard label (`*.`). The wildcard label must appear by itself as the first label. \n Hostname can be \"precise\" which is a domain name without the terminating dot of a network host (e.g. \"foo.example.com\") or \"wildcard\", which is a domain @@ -1853,13 +1855,13 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request - is used. \n Support: Extended \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause - a crash. \n Unknown values here must result - in the implementation setting the Accepted - Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + is used. \n Note that values may be added + to this enum, implementations must ensure + that unknown values will not cause a crash. + \n Unknown values here must result in the + implementation setting the Accepted Condition + for the Route to `status: False`, with a Reason + of `UnsupportedValue`. \n Support: Extended" enum: - http - https @@ -1867,13 +1869,14 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status - code to be used in response. \n Support: Core - \n Note that values may be added to this enum, - implementations must ensure that unknown values - will not cause a crash. \n Unknown values - here must result in the implementation setting - the Accepted Condition for the Route to `status: - False`, with a Reason of `UnsupportedValue`." + code to be used in response. \n Note that + values may be added to this enum, implementations + must ensure that unknown values will not cause + a crash. \n Unknown values here must result + in the implementation setting the Accepted + Condition for the Route to `status: False`, + with a Reason of `UnsupportedValue`. \n Support: + Core" enum: - 301 - 302 @@ -2257,12 +2260,12 @@ spec: description: "Scheme is the scheme to be used in the value of the `Location` header in the response. When empty, the scheme of the request is used. \n - Support: Extended \n Note that values may be added - to this enum, implementations must ensure that unknown - values will not cause a crash. \n Unknown values - here must result in the implementation setting the - Accepted Condition for the Route to `status: False`, - with a Reason of `UnsupportedValue`." + Note that values may be added to this enum, implementations + must ensure that unknown values will not cause a + crash. \n Unknown values here must result in the + implementation setting the Accepted Condition for + the Route to `status: False`, with a Reason of `UnsupportedValue`. + \n Support: Extended" enum: - http - https @@ -2270,12 +2273,13 @@ spec: statusCode: default: 302 description: "StatusCode is the HTTP status code to - be used in response. \n Support: Core \n Note that - values may be added to this enum, implementations - must ensure that unknown values will not cause a - crash. \n Unknown values here must result in the - implementation setting the Accepted Condition for - the Route to `status: False`, with a Reason of `UnsupportedValue`." + be used in response. \n Note that values may be + added to this enum, implementations must ensure + that unknown values will not cause a crash. \n Unknown + values here must result in the implementation setting + the Accepted Condition for the Route to `status: + False`, with a Reason of `UnsupportedValue`. \n + Support: Core" enum: - 301 - 302 @@ -2482,12 +2486,12 @@ spec: be ignored. \n If a query param is repeated in an HTTP request, the behavior is purposely left undefined, since different data planes have different - capabilities. However, it's *recommended* that + capabilities. However, it is *recommended* that implementations should match against the first value of the param if the data plane supports it, as this behavior is expected in other load balancing contexts outside of the Gateway API. - Users should not route traffic based on repeated + \n Users SHOULD NOT route traffic based on repeated query params to guard themselves against potential differences in the implementations." maxLength: 256 diff --git a/config/crd/standard/gateway.networking.k8s.io_referencegrants.yaml b/config/crd/standard/gateway.networking.k8s.io_referencegrants.yaml index 207f1c89ba..1ea14fe7a3 100644 --- a/config/crd/standard/gateway.networking.k8s.io_referencegrants.yaml +++ b/config/crd/standard/gateway.networking.k8s.io_referencegrants.yaml @@ -56,8 +56,8 @@ spec: from: description: "From describes the trusted namespaces and kinds that can reference the resources described in \"To\". Each entry in this - list must be considered to be an additional place that references - can be valid from, or to put this another way, entries must be combined + list MUST be considered to be an additional place that references + can be valid from, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantFrom describes trusted namespaces and @@ -97,9 +97,9 @@ spec: type: array to: description: "To describes the resources that may be referenced by - the resources described in \"From\". Each entry in this list must + the resources described in \"From\". Each entry in this list MUST be considered to be an additional place that references can be valid - to, or to put this another way, entries must be combined using OR. + to, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantTo describes what Kinds are allowed as @@ -179,8 +179,8 @@ spec: from: description: "From describes the trusted namespaces and kinds that can reference the resources described in \"To\". Each entry in this - list must be considered to be an additional place that references - can be valid from, or to put this another way, entries must be combined + list MUST be considered to be an additional place that references + can be valid from, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantFrom describes trusted namespaces and @@ -220,9 +220,9 @@ spec: type: array to: description: "To describes the resources that may be referenced by - the resources described in \"From\". Each entry in this list must + the resources described in \"From\". Each entry in this list MUST be considered to be an additional place that references can be valid - to, or to put this another way, entries must be combined using OR. + to, or to put this another way, entries MUST be combined using OR. \n Support: Core" items: description: ReferenceGrantTo describes what Kinds are allowed as