Skip to content

Commit 8d7fb06

Browse files
authored
Merge pull request #1139 from robscott/revert-address-match
Reverting GEP 735 from implemented to provisional
2 parents b5f5cd3 + d5e92da commit 8d7fb06

File tree

9 files changed

+1
-342
lines changed

9 files changed

+1
-342
lines changed

apis/v1alpha2/shared_types.go

-51
Original file line numberDiff line numberDiff line change
@@ -465,57 +465,6 @@ type AnnotationKey string
465465
// +kubebuilder:validation:MaxLength=4096
466466
type AnnotationValue string
467467

468-
// AddressRouteMatches defines AddressMatch rules for inbound traffic according to
469-
// source and/or destination address of a packet or connection.
470-
type AddressRouteMatches struct {
471-
// SourceAddresses indicates the originating (source) network
472-
// addresses which are valid for routing traffic.
473-
//
474-
// Support: Extended
475-
SourceAddresses []AddressMatch `json:"sourceAddresses"`
476-
477-
// DestinationAddresses indicates the destination network addresses
478-
// which are valid for routing traffic.
479-
//
480-
// Support: Extended
481-
DestinationAddresses []AddressMatch `json:"destinationAddresses"`
482-
}
483-
484-
// AddressMatch defines matching rules for network addresses by type.
485-
type AddressMatch struct {
486-
// Type of the address, either IPAddress or NamedAddress.
487-
//
488-
// If NamedAddress is used this is a custom and specific value for each
489-
// implementation to handle (and add validation for) according to their
490-
// own needs.
491-
//
492-
// For IPAddress the implementor may expect either IPv4 or IPv6.
493-
//
494-
// Support: Core (IPAddress)
495-
// Support: Custom (NamedAddress)
496-
//
497-
// +optional
498-
// +kubebuilder:validation:Enum=IPAddress;NamedAddress
499-
// +kubebuilder:default=IPAddress
500-
Type *AddressType `json:"type,omitempty"`
501-
502-
// Value of the address. The validity of the values will depend
503-
// on the type and support by the controller.
504-
//
505-
// If implementations support proxy-protocol (see:
506-
// https://www.haproxy.org/download/1.8/doc/proxy-protocol.txt) they
507-
// must respect the connection metadata from proxy-protocol
508-
// in the match logic implemented for these address values.
509-
//
510-
// Examples: `1.2.3.4`, `128::1`, `my-named-address`.
511-
//
512-
// Support: Core
513-
//
514-
// +kubebuilder:validation:MinLength=1
515-
// +kubebuilder:validation:MaxLength=253
516-
Value string `json:"value"`
517-
}
518-
519468
// AddressType defines how a network address is represented as a text string.
520469
type AddressType string
521470

apis/v1alpha2/tcproute_types.go

-7
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,6 @@ type TCPRouteStatus struct {
5959

6060
// TCPRouteRule is the configuration for a given rule.
6161
type TCPRouteRule struct {
62-
// Matches are rules for routing traffic to backends based on addresses.
63-
//
64-
// +optional
65-
// +kubebuilder:validation:MaxItems=16
66-
// <gateway:experimental>
67-
Matches []AddressRouteMatches `json:"matches,omitempty"`
68-
6962
// BackendRefs defines the backend(s) where matching requests should be
7063
// sent. If unspecified or invalid (refers to a non-existent resource or a
7164
// Service with no endpoints), the underlying implementation MUST actively

apis/v1alpha2/udproute_types.go

-7
Original file line numberDiff line numberDiff line change
@@ -59,13 +59,6 @@ type UDPRouteStatus struct {
5959

6060
// UDPRouteRule is the configuration for a given rule.
6161
type UDPRouteRule struct {
62-
// Matches add rules for filtering traffic to backends based on addresses.
63-
//
64-
// +optional
65-
// +kubebuilder:validation:MaxItems=16
66-
// <gateway:experimental>
67-
Matches []AddressRouteMatches `json:"matches,omitempty"`
68-
6962
// BackendRefs defines the backend(s) where matching requests should be
7063
// sent. If unspecified or invalid (refers to a non-existent resource or a
7164
// Service with no endpoints), the underlying implementation MUST actively

apis/v1alpha2/zz_generated.deepcopy.go

-63
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_tcproutes.yaml

-88
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

config/crd/experimental/gateway.networking.k8s.io_udproutes.yaml

-88
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

examples/experimental/v1alpha2/traffic-matching-tcp.yaml

-15
This file was deleted.

site-src/geps/gep-735.md

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
# GEP-735: TCP and UDP addresses matching
22

33
* Issue: [#735](https://github.com/kubernetes-sigs/gateway-api/issues/735)
4-
* Status: Implemented
4+
* Status: Provisional
55

66
## TLDR
77

0 commit comments

Comments
 (0)