make sure reserved sandboxes are excluded

When reservation == "" (not specified)
rhpds · Oct 10, 2023 · b85406c · b85406c
1 parent f23abe8
commit b85406c
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/internal/dynamodb/accounts.go b/internal/dynamodb/accounts.go
@@ -430,6 +430,9 @@ func (a *AwsAccountDynamoDBProvider) Request(service_uuid string, reservation st
 
 	if reservation != "" {
 		filter = filter.And(expression.Name("reservation").Equal(expression.Value(reservation)))
+	} else {
+		filter = filter.And(expression.Name("reservation").AttributeNotExists().
+			Or(expression.Name("reservation").Equal(expression.Value(""))))
 	}
 
 	// get 10 spare accounts in case of concurrency doublebooking
@@ -450,6 +453,9 @@ func (a *AwsAccountDynamoDBProvider) Request(service_uuid string, reservation st
 
 		if reservation != "" {
 			filter = filter.And(expression.Name("reservation").Equal(expression.Value(reservation)))
+		} else {
+			filter = filter.And(expression.Name("reservation").AttributeNotExists().
+				Or(expression.Name("reservation").Equal(expression.Value(""))))
 		}
 
 		accounts, err = GetAccounts(a.Svc, filter, count+10)