Skip to content

Commit 80abd39

Browse files
authored
Merge pull request #12276 from sbueringer/pr-make-crsb-cluster-name-required
⚠️ Make clusterName field in ClusterResourceSetBinding required
2 parents 0d9961e + c59b1f5 commit 80abd39

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

api/addons/v1beta2/clusterresourcesetbinding_types.go

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -141,11 +141,10 @@ type ClusterResourceSetBindingSpec struct {
141141
Bindings []*ResourceSetBinding `json:"bindings,omitempty"`
142142

143143
// clusterName is the name of the Cluster this binding applies to.
144-
// Note: this field mandatory in v1beta2.
145-
// +optional
144+
// +required
146145
// +kubebuilder:validation:MinLength=1
147146
// +kubebuilder:validation:MaxLength=63
148-
ClusterName string `json:"clusterName,omitempty"`
147+
ClusterName string `json:"clusterName"`
149148
}
150149

151150
// ANCHOR_END: ClusterResourceSetBindingSpec

config/crd/bases/addons.cluster.x-k8s.io_clusterresourcesetbindings.yaml

Lines changed: 4 additions & 3 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)