Skip to content

Commit

Permalink
fix: allow setting overprovision ratio to 1
Browse files Browse the repository at this point in the history
Signed-off-by: Suleyman Akbas <[email protected]>
  • Loading branch information
suleymanakbas91 committed Dec 5, 2023
1 parent d27e8ef commit 61591e0
Show file tree
Hide file tree
Showing 6 changed files with 6 additions and 6 deletions.
2 changes: 1 addition & 1 deletion api/v1alpha1/lvmcluster_types.go
Original file line number Diff line number Diff line change
Expand Up @@ -52,7 +52,7 @@ type ThinPoolConfig struct {

// OverProvisionRatio is the factor by which additional storage can be provisioned compared to
// the available storage in the thin pool.
// +kubebuilder:validation:Minimum=2
// +kubebuilder:validation:Minimum=1
// +kubebuilder:validation:Required
// +required
OverprovisionRatio int `json:"overprovisionRatio"`
Expand Down
2 changes: 1 addition & 1 deletion bundle/manifests/lvm.topolvm.io_lvmclusters.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -170,7 +170,7 @@ spec:
description: OverProvisionRatio is the factor by which
additional storage can be provisioned compared to
the available storage in the thin pool.
minimum: 2
minimum: 1
type: integer
sizePercent:
default: 90
Expand Down
2 changes: 1 addition & 1 deletion bundle/manifests/lvm.topolvm.io_lvmvolumegroups.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ spec:
description: OverProvisionRatio is the factor by which additional
storage can be provisioned compared to the available storage
in the thin pool.
minimum: 2
minimum: 1
type: integer
sizePercent:
default: 90
Expand Down
2 changes: 1 addition & 1 deletion config/crd/bases/lvm.topolvm.io_lvmclusters.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ spec:
description: OverProvisionRatio is the factor by which
additional storage can be provisioned compared to
the available storage in the thin pool.
minimum: 2
minimum: 1
type: integer
sizePercent:
default: 90
Expand Down
2 changes: 1 addition & 1 deletion config/crd/bases/lvm.topolvm.io_lvmvolumegroups.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -145,7 +145,7 @@ spec:
description: OverProvisionRatio is the factor by which additional
storage can be provisioned compared to the available storage
in the thin pool.
minimum: 2
minimum: 1
type: integer
sizePercent:
default: 90
Expand Down
2 changes: 1 addition & 1 deletion doc/design/thin_pool.md
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ The LVMO will create a thin-pool LV in the volume group in order to create thinl
+ // OverProvisionRatio represents the ratio of overprovision that can
+ // be allowed on thin pools
+ // +kubebuilder:validation:Minimum=2
+ // +kubebuilder:validation:Minimum=1
+ OverprovisionRatio int `json:"overprovisionRatio,omitempty"`
}
Expand Down

0 comments on commit 61591e0

Please sign in to comment.