Skip to content

Commit

Permalink
Don't set default for optional CtlplaneNetmask
Browse files Browse the repository at this point in the history
Signed-off-by: Brendan Shephard <[email protected]>
  • Loading branch information
bshephar committed Dec 9, 2024
1 parent caecc88 commit fc75931
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,6 @@ spec:
to use for ctlplane network
type: string
ctlplaneNetmask:
default: 255.255.255.0
description: 'CtlplaneNetmask - Netmask to use for ctlplane network
(TODO: acquire this is another manner?)'
type: string
Expand Down
1 change: 0 additions & 1 deletion api/v1beta1/openstackbaremetalset_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -88,7 +88,6 @@ type OpenStackBaremetalSetSpec struct {
// +kubebuilder:validation:Optional
CtlplaneGateway string `json:"ctlplaneGateway,omitempty"`
// +kubebuilder:validation:Optional
// +kubebuilder:default="255.255.255.0"
// CtlplaneNetmask - Netmask to use for ctlplane network (TODO: acquire this is another manner?)
CtlplaneNetmask string `json:"ctlplaneNetmask,omitempty"`
// +kubebuilder:default=openshift-machine-api
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -139,7 +139,6 @@ spec:
to use for ctlplane network
type: string
ctlplaneNetmask:
default: 255.255.255.0
description: 'CtlplaneNetmask - Netmask to use for ctlplane network
(TODO: acquire this is another manner?)'
type: string
Expand Down

0 comments on commit fc75931

Please sign in to comment.