Merge pull request #671 from robscott/require-controller

Making Controller a required field in Route status
kubernetes-sigs · May 18, 2021 · 9792a30 · 9792a30
2 parents cb48d36 + 55b8136
commit 9792a30
Show file tree

Hide file tree

Showing 6 changed files with 11 additions and 8 deletions.
diff --git a/apis/v1alpha2/shared_types.go b/apis/v1alpha2/shared_types.go
@@ -220,9 +220,9 @@ type RouteStatusGatewayReference struct {
 	// valid Kubernetes names
 	// (https://kubernetes.io/docs/concepts/overview/working-with-objects/names/#names).
 	//
+	// +kubebuilder:validation:MinLength=1
 	// +kubebuilder:validation:MaxLength=253
-	// +optional
-	Controller *string `json:"controller"`
+	Controller string `json:"controller"`
 }
 
 // RouteStatus defines the observed state that is required across

diff --git a/apis/v1alpha2/zz_generated.deepcopy.go b/apis/v1alpha2/zz_generated.deepcopy.go
diff --git a/config/crd/bases/networking.x-k8s.io_httproutes.yaml b/config/crd/bases/networking.x-k8s.io_httproutes.yaml
diff --git a/config/crd/bases/networking.x-k8s.io_tcproutes.yaml b/config/crd/bases/networking.x-k8s.io_tcproutes.yaml
diff --git a/config/crd/bases/networking.x-k8s.io_tlsroutes.yaml b/config/crd/bases/networking.x-k8s.io_tlsroutes.yaml
diff --git a/config/crd/bases/networking.x-k8s.io_udproutes.yaml b/config/crd/bases/networking.x-k8s.io_udproutes.yaml