fix(BA-3938): Remove PROVISIONING from Endpoint resolved status and consolidate to DEGRADED (#8115)

Author: jopemachine

changes/8115.fix.md

@@ -0,0 +1 @@
+Remove `PROVISIONING` from `Endpoint` resolved status and consolidate to `DEGRADED`

src/ai/backend/manager/api/gql_legacy/endpoint.py

@@ -923,18 +923,7 @@ async def resolve_status(self, info: graphene.ResolveInfo) -> str:
                         return EndpointStatus.HEALTHY
                     if healthy_service_count == 0:
                         return EndpointStatus.UNHEALTHY
-                    problematic_service_count = len([
-                        r
-                        for r in self.routings
-                        if r.status
-                        in (
-                            RouteStatus.UNHEALTHY.name,
-                            RouteStatus.DEGRADED.name,
-                        )
-                    ])
-                    if problematic_service_count > 0:
-                        return EndpointStatus.DEGRADED
-                return EndpointStatus.PROVISIONING
+                return EndpointStatus.DEGRADED
 
     async def resolve_model_vfolder(self, info: graphene.ResolveInfo) -> VirtualFolderNode:
         if not self.model:

tests/unit/manager/api/endpoint/test_types.py

@@ -45,3 +45,25 @@ async def test_status_degraded_when_healthy_and_degraded_routes_mixed(self) -> N
 
         result = await Endpoint.resolve_status(mock_endpoint, info=Mock())
         assert result == EndpointStatus.DEGRADED
+
+    async def test_status_degraded_when_healthy_and_provisioning_routes_mixed(self) -> None:
+        """
+        When some routes are healthy and others are still provisioning,
+        the endpoint status should be DEGRADED (not PROVISIONING).
+
+        Previously returned PROVISIONING, but this was changed to DEGRADED
+        for clearer status semantics.
+        """
+        mock_endpoint = Mock(spec=Endpoint)
+        mock_endpoint.lifecycle_stage = EndpointLifecycle.READY.name
+        mock_endpoint.retries = 0
+
+        healthy_route = Mock()
+        healthy_route.status = RouteStatus.HEALTHY.name
+        provisioning_route = Mock()
+        provisioning_route.status = RouteStatus.PROVISIONING.name
+
+        mock_endpoint.routings = [healthy_route, provisioning_route]
+
+        result = await Endpoint.resolve_status(mock_endpoint, info=Mock())
+        assert result == EndpointStatus.DEGRADED