fix(policy): compute desired number of matches before rooms

internal/core/services/autoscaler/policies/roomoccupancy/policy.go

@@ -96,12 +96,12 @@ func (p *Policy) CalculateDesiredNumberOfRooms(policyParameters autoscaling.Poli
 		return -1, errors.New("There are no runningMatches in the currentState")
 	}
 
-	maxMatches := currentState[MaxMatchesKey].(int)
+	maxMatchesPerRoom := currentState[MaxMatchesKey].(int)
 
-	occupiedRooms := math.Ceil(float64(runningMatches) / float64(maxMatches))
-	desiredNumberOfRoom := int(math.Ceil(float64(occupiedRooms) / (float64(1) - readyTarget)))
+	desiredNumberOfMatches := math.Ceil(float64(runningMatches) / (float64(1) - readyTarget))
+	desiredNumberOfRooms := int(math.Ceil(desiredNumberOfMatches / float64(maxMatchesPerRoom)))
 
-	return desiredNumberOfRoom, nil
+	return desiredNumberOfRooms, nil
 }
 
 func (p *Policy) CanDownscale(policyParameters autoscaling.PolicyParameters, currentState policies.CurrentState) (bool, error) {

proto/api/v1/rooms.proto

@@ -98,7 +98,7 @@ message UpdateRoomStatusRequest {
   string status = 4;
   // Timestamp of the status update.
   int64 timestamp = 6;
-  // The number of running_matches in the game room.
+  // The number of running matches in the game room.
   int64 running_matches = 7;
 }
 

proto/apidocs.swagger.json

@@ -800,7 +800,7 @@
         "runningMatches": {
           "type": "string",
           "format": "int64",
-          "description": "The number of running_matches in the game room."
+          "description": "The number of running matches in the game room."
         }
       },
       "description": "The player event request."