refactor(BA-5982): rename ReplicaCountData → ReplicaData

Per review feedback. The wrapper used to pair the persisted
``replica_count`` with the optional ``desired_replica_count`` no longer
needs the ``Count`` qualifier now that the owning field on
``DeploymentInfo`` is just ``replica`` — calling the type ``ReplicaData``
matches the field name and is consistent with the rest of the
``…Data`` family in this module. No behavior change.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: jopemachine

src/ai/backend/manager/data/deployment/types.py

@@ -359,7 +359,7 @@ def target_replica_count(self) -> int:
 
 
 @dataclass
-class ReplicaCountData:
+class ReplicaData:
     replica_count: int
     desired_replica_count: int | None
 
@@ -684,7 +684,7 @@ class DeploymentInfo:
     id: DeploymentID
     metadata: DeploymentMetadata
     state: DeploymentState
-    replica: ReplicaCountData
+    replica: ReplicaData
     network: DeploymentNetworkData
     model_revisions: list[ModelRevisionData]
     options: DeploymentOptions

src/ai/backend/manager/models/endpoint/row.py

@@ -73,7 +73,7 @@
     DeploymentSummaryData,
     ModelDeploymentAutoScalingRuleData,
     ModelRevisionData,
-    ReplicaCountData,
+    ReplicaData,
 )
 from ai.backend.manager.data.model_serving.types import (
     EndpointAutoScalingRuleData,
@@ -765,7 +765,7 @@ def _to_deployment_info_with_revisions(
                 scaling_state=self.scaling_state,
                 retry_count=self.retries,
             ),
-            replica=ReplicaCountData(
+            replica=ReplicaData(
                 replica_count=self.replicas,
                 desired_replica_count=self.desired_replicas,
             ),

tests/unit/manager/services/deployment/test_deployment_crud_actions.py

@@ -43,7 +43,7 @@
     ModelMountConfigData,
     ModelRevisionData,
     ModelRuntimeConfigData,
-    ReplicaCountData,
+    ReplicaData,
     ResourceConfigData,
     RouteHealthStatus,
     RouteInfo,
@@ -148,7 +148,7 @@ def endpoint_info(self, endpoint_id: uuid.UUID) -> DeploymentInfo:
                 scaling_state=ScalingState.STABLE,
                 retry_count=0,
             ),
-            replica=ReplicaCountData(replica_count=2, desired_replica_count=None),
+            replica=ReplicaData(replica_count=2, desired_replica_count=None),
             network=DeploymentNetworkData(
                 open_to_public=False, access_token_ids=None, url=None, preferred_domain_name=None
             ),

tests/unit/manager/services/deployment/test_deployment_service.py

@@ -53,7 +53,7 @@
     ModelMountConfigData,
     ModelRevisionData,
     ModelRuntimeConfigData,
-    ReplicaCountData,
+    ReplicaData,
     ResourceConfigData,
     ResourceSpec,
 )
@@ -378,7 +378,7 @@ def deployment_info(self, deployment_id: uuid.UUID) -> DeploymentInfo:
                 scaling_state=ScalingState.STABLE,
                 retry_count=0,
             ),
-            replica=ReplicaCountData(replica_count=1, desired_replica_count=None),
+            replica=ReplicaData(replica_count=1, desired_replica_count=None),
             network=DeploymentNetworkData(
                 open_to_public=False, access_token_ids=None, url=None, preferred_domain_name=None
             ),
@@ -550,7 +550,7 @@ def deployment_info(
                 scaling_state=ScalingState.STABLE,
                 retry_count=0,
             ),
-            replica=ReplicaCountData(replica_count=1, desired_replica_count=None),
+            replica=ReplicaData(replica_count=1, desired_replica_count=None),
             network=DeploymentNetworkData(
                 open_to_public=False, access_token_ids=None, url=None, preferred_domain_name=None
             ),
@@ -714,7 +714,7 @@ def test_current_revision_resolved_by_id_match_not_list_order(
                 scaling_state=ScalingState.STABLE,
                 retry_count=0,
             ),
-            replica=ReplicaCountData(replica_count=1, desired_replica_count=None),
+            replica=ReplicaData(replica_count=1, desired_replica_count=None),
             network=DeploymentNetworkData(
                 open_to_public=False, access_token_ids=None, url=None, preferred_domain_name=None
             ),

tests/unit/manager/sokovan/deployment/executor/conftest.py

@@ -21,7 +21,7 @@
     DeploymentOptions,
     DeploymentState,
     ModelRevisionData,
-    ReplicaCountData,
+    ReplicaData,
     RouteHealthStatus,
     RouteStatus,
 )
@@ -153,7 +153,7 @@ def _create_deployment_info(
             scaling_state=ScalingState.STABLE,
             retry_count=0,
         ),
-        replica=ReplicaCountData(
+        replica=ReplicaData(
             replica_count=replica_count,
             desired_replica_count=desired_replica_count,
         ),

tests/unit/manager/sokovan/deployment/handlers/test_deploying_handler.py

@@ -26,7 +26,7 @@
     DeploymentNetworkData,
     DeploymentOptions,
     DeploymentState,
-    ReplicaCountData,
+    ReplicaData,
 )
 from ai.backend.manager.data.resource.types import ScalingGroupProxyTarget
 from ai.backend.manager.sokovan.deployment.handlers.deploying import (
@@ -123,7 +123,7 @@ def deployment_created_without_revision(self) -> DeploymentWithHistory:
                     scaling_state=ScalingState.STABLE,
                     retry_count=0,
                 ),
-                replica=ReplicaCountData(
+                replica=ReplicaData(
                     replica_count=1,
                     desired_replica_count=1,
                 ),

tests/unit/manager/sokovan/deployment/route/executor/conftest.py

@@ -19,7 +19,7 @@
     DeploymentNetworkData,
     DeploymentOptions,
     DeploymentState,
-    ReplicaCountData,
+    ReplicaData,
     RouteHealthStatus,
     RouteStatus,
 )
@@ -161,7 +161,7 @@ def _create_deployment_info(
             scaling_state=ScalingState.STABLE,
             retry_count=0,
         ),
-        replica=ReplicaCountData(
+        replica=ReplicaData(
             replica_count=1,
             desired_replica_count=1,
         ),

tests/unit/manager/sokovan/deployment/strategy/test_rolling_update.py

@@ -32,7 +32,7 @@
     DeploymentNetworkData,
     DeploymentOptions,
     DeploymentState,
-    ReplicaCountData,
+    ReplicaData,
     RouteHealthStatus,
     RouteInfo,
     RouteStatus,
@@ -117,7 +117,7 @@ def make_deployment(
             scaling_state=ScalingState.STABLE,
             retry_count=0,
         ),
-        replica=ReplicaCountData(
+        replica=ReplicaData(
             replica_count=desired,
             desired_replica_count=None,
         ),
@@ -927,7 +927,7 @@ class TestDesiredReplicaCount:
     def test_desired_replica_count_overrides_replica_count(self) -> None:
         """When desired_replica_count is set, it takes precedence."""
         deployment = make_deployment(desired=3)
-        deployment.replica = ReplicaCountData(
+        deployment.replica = ReplicaData(
             replica_count=1,
             desired_replica_count=3,
         )
@@ -943,7 +943,7 @@ def test_desired_replica_count_overrides_replica_count(self) -> None:
     def test_replica_count_used_when_no_desired(self) -> None:
         """When desired_replica_count is None, uses replica_count."""
         deployment = make_deployment(desired=2)
-        deployment.replica = ReplicaCountData(
+        deployment.replica = ReplicaData(
             replica_count=2,
             desired_replica_count=None,
         )

tests/unit/manager/sokovan/deployment/test_coordinator_history.py

@@ -20,7 +20,7 @@
     DeploymentState,
     DeploymentStatusTransitions,
     DeploymentTargetStatuses,
-    ReplicaCountData,
+    ReplicaData,
 )
 from ai.backend.manager.repositories.deployment import DeploymentRepository
 from ai.backend.manager.sokovan.deployment.coordinator import DeploymentCoordinator
@@ -61,7 +61,7 @@ def sample_deployment_info() -> DeploymentInfo:
             scaling_state=ScalingState.STABLE,
             retry_count=0,
         ),
-        replica=ReplicaCountData(
+        replica=ReplicaData(
             replica_count=1,
             desired_replica_count=None,
         ),