Skip to content

Commit 635c18e

Browse files
authored
🐛 Fix MaxLength of worker topology Name fields (#12072)
* Fix MaxLength of worker topology Name fields Due to existing webhook validation, the max length was effectively 63 characters. This updates the annotation (and CRD) to reflect this actual max length. * fixup! Fix MaxLength of worker topology Name fields Adjust to 63 because maxLength is inclusive
1 parent df72551 commit 635c18e

File tree

3 files changed

+8
-8
lines changed

3 files changed

+8
-8
lines changed

Diff for: api/v1beta1/cluster_types.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -686,7 +686,7 @@ type MachineDeploymentTopology struct {
686686
// the values are hashed together.
687687
// +required
688688
// +kubebuilder:validation:MinLength=1
689-
// +kubebuilder:validation:MaxLength=256
689+
// +kubebuilder:validation:MaxLength=63
690690
Name string `json:"name"`
691691

692692
// failureDomain is the failure domain the machines will be created in.
@@ -797,7 +797,7 @@ type MachinePoolTopology struct {
797797
// the values are hashed together.
798798
// +required
799799
// +kubebuilder:validation:MinLength=1
800-
// +kubebuilder:validation:MaxLength=256
800+
// +kubebuilder:validation:MaxLength=63
801801
Name string `json:"name"`
802802

803803
// failureDomains is the list of failure domains the machine pool will be created in.

Diff for: api/v1beta2/cluster_types.go

+2-2
Original file line numberDiff line numberDiff line change
@@ -686,7 +686,7 @@ type MachineDeploymentTopology struct {
686686
// the values are hashed together.
687687
// +required
688688
// +kubebuilder:validation:MinLength=1
689-
// +kubebuilder:validation:MaxLength=256
689+
// +kubebuilder:validation:MaxLength=63
690690
Name string `json:"name"`
691691

692692
// failureDomain is the failure domain the machines will be created in.
@@ -797,7 +797,7 @@ type MachinePoolTopology struct {
797797
// the values are hashed together.
798798
// +required
799799
// +kubebuilder:validation:MinLength=1
800-
// +kubebuilder:validation:MaxLength=256
800+
// +kubebuilder:validation:MaxLength=63
801801
Name string `json:"name"`
802802

803803
// failureDomains is the list of failure domains the machine pool will be created in.

Diff for: config/crd/bases/cluster.x-k8s.io_clusters.yaml

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

0 commit comments

Comments
 (0)