Fix an issue with default VolmeStorage not being accounted for in Quo…

…taRequests

Signed-off-by: tylerslaton <[email protected]>

pkg/controller/quota/quota.go

@@ -146,16 +146,28 @@ func addStorage(appInstance *v1.AppInstance, quotaRequest *adminv1.QuotaRequestI
 			size = boundSize
 		}
 
-		// No need to proceed if the size is empty
-		if size == "" || size == "0" {
+		// Handle three cases:
+		// 1. The volume's size is explicitly set to 0. This means there is nothing to count.
+		// 2. The volume's size is not set. This means we should assume the default size.
+		// 3. The volume's size is set to a specific value. This means we should use that value.
+		var sizeQuantity resource.Quantity
+		switch size {
+		case "0":
 			continue
+		case "":
+			sizeQuantity = *v1.DefaultSize // Safe to dereference because it is statically set in the v1 package.
+			if appInstance.Status.Defaults.VolumeSize != nil {
+				sizeQuantity = *appInstance.Status.Defaults.VolumeSize
+			}
+		default:
+			parsedQuantity, err := resource.ParseQuantity(string(size))
+			if err != nil {
+				return err
+			}
+			sizeQuantity = parsedQuantity
 		}
 
-		parsedSize, err := resource.ParseQuantity(string(size))
-		if err != nil {
-			return err
-		}
-		quotaRequest.Spec.Resources.VolumeStorage.Add(parsedSize)
+		quotaRequest.Spec.Resources.VolumeStorage.Add(sizeQuantity)
 	}
 
 	// Add the secrets needed to the quota request. We only parse net new secrets, not