Skip to content

Commit 77cb25d

Browse files
authored
fix(archive): add RayOperator.get_remote_status override so archiving state advances (#1218)
RayOperator never overrode AbstractOperator.get_remote_status (which returns empty ServiceStatus), causing _try_advance_archiving to never see completion. Add the override that delegates to the module-level rocklet_probe.get_remote_status, and keep SandboxManager calling through self._operator for consistency.
1 parent b81e3f9 commit 77cb25d

1 file changed

Lines changed: 3 additions & 0 deletions

File tree

rock/sandbox/operator/ray.py

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -92,6 +92,9 @@ async def submit(self, config: DockerDeploymentConfig, user_info: dict = {}) ->
9292
logger.info(f"sandbox {sandbox_id} is submitted")
9393
return sandbox_info
9494

95+
async def get_remote_status(self, sandbox_id: str, host_ip: str):
96+
return await get_remote_status(sandbox_id, host_ip)
97+
9598
async def get_status(self, sandbox_id: str) -> SandboxInfo | None:
9699
sandbox_info: SandboxInfo = await build_sandbox_from_redis(self._redis_provider, sandbox_id)
97100
if sandbox_info is None:

0 commit comments

Comments
 (0)