feat: adds auto scale padding

strangelove-ventures · Sep 26, 2023 · d431d79 · d431d79
1 parent f240661
commit d431d79
Show file tree

Hide file tree

Showing 2 changed files with 95 additions and 23 deletions.
diff --git a/internal/fullnode/pvc_builder.go b/internal/fullnode/pvc_builder.go
@@ -6,10 +6,16 @@ import (
 	cosmosv1 "github.com/strangelove-ventures/cosmos-operator/api/v1"
 	"github.com/strangelove-ventures/cosmos-operator/internal/diff"
 	"github.com/strangelove-ventures/cosmos-operator/internal/kube"
+	"gopkg.in/inf.v0"
 	corev1 "k8s.io/api/core/v1"
+	"k8s.io/apimachinery/pkg/api/resource"
 	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
+const (
+	SnapshotGrowthFactor = 102
+)
+
 var (
 	defaultAccessModes = []corev1.PersistentVolumeAccessMode{corev1.ReadWriteOnce}
 )
@@ -79,9 +85,23 @@ func pvcResources(crd *cosmosv1.CosmosFullNode) corev1.ResourceRequirements {
 		reqs = crd.Spec.VolumeClaimTemplate.Resources
 		size = reqs.Requests[corev1.ResourceStorage]
 	)
+
 	if autoScale := crd.Status.SelfHealing.PVCAutoScale; autoScale != nil {
-		if autoScale.RequestedSize.Cmp(size) > 0 {
-			reqs.Requests[corev1.ResourceStorage] = autoScale.RequestedSize
+		// This is the implementation using Int64 but it does not support fractions
+		//requestSize, unableToConvert := autoScale.RequestedSize.AsInt64()
+		//if unableToConvert == true {
+		//	fmt.Errorf("Unable to convert auto scale request")
+		//} else {
+		//	sizeWithPadding := resource.NewQuantity(int64(float64(requestSize)*SnapshotScalingFactor), resource.DecimalSI)
+		//	if sizeWithPadding.Cmp(size) > 0 {
+		//		reqs.Requests[corev1.ResourceStorage] = *sizeWithPadding
+		//	}
+		//}
+		requestedSize := autoScale.RequestedSize.DeepCopy()
+		newSize := requestedSize.AsDec()
+		sizeWithPadding := resource.NewDecimalQuantity(*newSize.Mul(newSize, inf.NewDec(SnapshotGrowthFactor, 2)), resource.DecimalSI)
+		if sizeWithPadding.Cmp(size) > 0 {
+			reqs.Requests[corev1.ResourceStorage] = *sizeWithPadding
 		}
 	}
 	return reqs

diff --git a/internal/fullnode/pvc_builder_test.go b/internal/fullnode/pvc_builder_test.go
@@ -28,6 +28,7 @@ func TestBuildPVCs(t *testing.T) {
 				Requests: corev1.ResourceList{corev1.ResourceStorage: resource.MustParse("100G")},
 			},
 		}
+
 		crd.Spec.InstanceOverrides = map[string]cosmosv1.InstanceOverridesSpec{
 			"juno-0": {},
 		}
@@ -148,30 +149,81 @@ func TestBuildPVCs(t *testing.T) {
 		}
 	})
 
-	t.Run("pvc auto scale", func(t *testing.T) {
-		for _, tt := range []struct {
-			SpecQuant, AutoScaleQuant, WantQuant string
-		}{
-			{"100G", "99G", "100G"},
-			{"100G", "101G", "101G"},
-		} {
-			crd := defaultCRD()
-			crd.Spec.Replicas = 1
-			crd.Spec.VolumeClaimTemplate = cosmosv1.PersistentVolumeClaimSpec{
-				Resources: corev1.ResourceRequirements{
-					Requests: corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.SpecQuant)},
-				},
-			}
-			crd.Status.SelfHealing.PVCAutoScale = &cosmosv1.PVCAutoScaleStatus{
-				RequestedSize: resource.MustParse(tt.AutoScaleQuant),
+	t.Run("pvc auto scale with padding", func(t *testing.T) {
+		t.Run("given auto scale size less then current size", func(t *testing.T) {
+			for _, tt := range []struct {
+				SpecQuant, AutoScaleQuant, WantQuant string
+			}{
+				{"100G", "97G", "100G"},
+			} {
+				crd := defaultCRD()
+				crd.Spec.Replicas = 1
+				crd.Spec.VolumeClaimTemplate = cosmosv1.PersistentVolumeClaimSpec{
+					Resources: corev1.ResourceRequirements{
+						Requests: corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.SpecQuant)},
+					},
+				}
+				crd.Status.SelfHealing.PVCAutoScale = &cosmosv1.PVCAutoScaleStatus{
+					RequestedSize: resource.MustParse(tt.AutoScaleQuant),
+				}
+
+				pvcs := BuildPVCs(&crd)
+				require.Len(t, pvcs, 1, tt)
+
+				want := corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.WantQuant)}
+				require.Equal(t, want.Storage().Value(), pvcs[0].Object().Spec.Resources.Requests.Storage().Value(), tt)
 			}
+		})
 
-			pvcs := BuildPVCs(&crd)
-			require.Len(t, pvcs, 1, tt)
+		t.Run("given auto scale size equal to current size", func(t *testing.T) {
+			for _, tt := range []struct {
+				SpecQuant, AutoScaleQuant, WantQuant string
+			}{
+				{"102G", "100G", "102G"},
+			} {
+				crd := defaultCRD()
+				crd.Spec.Replicas = 1
+				crd.Spec.VolumeClaimTemplate = cosmosv1.PersistentVolumeClaimSpec{
+					Resources: corev1.ResourceRequirements{
+						Requests: corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.SpecQuant)},
+					},
+				}
+				crd.Status.SelfHealing.PVCAutoScale = &cosmosv1.PVCAutoScaleStatus{
+					RequestedSize: resource.MustParse(tt.AutoScaleQuant),
+				}
+
+				pvcs := BuildPVCs(&crd)
+				require.Len(t, pvcs, 1, tt)
+
+				want := corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.WantQuant)}
+				require.Equal(t, want, pvcs[0].Object().Spec.Resources.Requests, tt)
+			}
+		})
 
-			want := corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.WantQuant)}
-			require.Equal(t, want, pvcs[0].Object().Spec.Resources.Requests, tt)
-		}
+		t.Run("given auto scale size greater then current size", func(t *testing.T) {
+			for _, tt := range []struct {
+				SpecQuant, AutoScaleQuant, WantQuant string
+			}{
+				{"100G", "100G", "102G"},
+			} {
+				crd := defaultCRD()
+				crd.Spec.Replicas = 1
+				crd.Spec.VolumeClaimTemplate = cosmosv1.PersistentVolumeClaimSpec{
+					Resources: corev1.ResourceRequirements{
+						Requests: corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.SpecQuant)},
+					},
+				}
+				crd.Status.SelfHealing.PVCAutoScale = &cosmosv1.PVCAutoScaleStatus{
+					RequestedSize: resource.MustParse(tt.AutoScaleQuant),
+				}
+
+				pvcs := BuildPVCs(&crd)
+				require.Len(t, pvcs, 1, tt)
+
+				want := corev1.ResourceList{corev1.ResourceStorage: resource.MustParse(tt.WantQuant)}
+				require.Equal(t, want.Storage().Value(), pvcs[0].Object().Spec.Resources.Requests.Storage().Value(), tt)
+			}
+		})
 	})
 
 	test.HasTypeLabel(t, func(crd cosmosv1.CosmosFullNode) []map[string]string {