fix

Signed-off-by: Britania Rodriguez Reyes <britaniar@microsoft.com>

Author: britaniar

pkg/propertychecker/azure/checker.go

@@ -147,17 +147,13 @@ func extractCapacityRequirements(req placementv1beta1.PropertySelectorRequiremen
 // validateCapacity checks if the provided capacity value is valid.
 func validateCapacity(value string, operator placementv1beta1.PropertySelectorOperator) (uint32, error) {
 	// Parse directly as uint32 to avoid integer overflow issues
-	capacity, err := strconv.ParseUint(value, 10, 32)
+	valueUint, err := strconv.ParseUint(value, 10, 32)
 	if err != nil {
 		return 0, fmt.Errorf("invalid capacity value %q: %w", value, err)
 	}
+	capacity := uint32(valueUint)
 
-	// Validate capacity is non-negative
-	if capacity < 0 {
-		return 0, fmt.Errorf("capacity value %d cannot be negative", capacity)
-	}
-
-	// Validate capacity bounds
+	// Validate capacity bounds (capacity is already >= 0 since it's parsed as uint)
 	if capacity < minVMInstanceCapacity {
 		return 0, fmt.Errorf("capacity value %d is below minimum allowed value of %d", capacity, minVMInstanceCapacity)
 	}
@@ -169,6 +165,5 @@ func validateCapacity(value string, operator placementv1beta1.PropertySelectorOp
 		// For GreaterThanOrEqualTo, subtract 1 as client requests only support GreaterThan
 		capacity -= 1
 	}
-
-	return uint32(capacity), nil
+	return capacity, nil
 }