OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Update Daytona Runtime

Open SmartManoj opened this issue 6 months ago • 14 comments

  • [ ] This change is worth documenting at https://docs.all-hands.dev/
  • [ ] Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality this introduces.


Summarize what the PR does, explaining any non-trivial design decisions. updated daytona_target from eu to default value us - to avoid "No available nodes" Removed deprecated Workspace class with Sandbox CreateSandboxParams doesn't accept id argument - updated logic Disabled VS code and jupyter plugins for 1GB memory limit.


Link of any specific issues this addresses: Fixes #8825 #8824 #8523

SmartManoj avatar Jun 02 '25 07:06 SmartManoj

@idagelic how would you like us to treat Daytona related issues? I saw a few github issues for the Daytona runtime. Should we tag you on the issues, refer them to Daytona, something else?

mamoodi avatar Jun 03 '25 14:06 mamoodi

@mamoodi Hey, you can assign me to any issue or request a review and I will take a look asap. I will review this one as well, just checking some stuff on our backend.

idagelic avatar Jun 03 '25 15:06 idagelic

Disabled VS code and jupyter plugins for 1GB memory limit.

@idagelic

SmartManoj avatar Jun 04 '25 13:06 SmartManoj

Disabled VS code and jupyter plugins for 1GB memory limit.

@idagelic

I understand, but I think users might want to be able to edit the codebase in VSCode once the agent is done generating

idagelic avatar Jun 04 '25 14:06 idagelic

resources=SandboxResources(cpu=2, memory=4)

@idagelic this would solve #8825 too? I tried it on my side but the error persists :thinking: .

arun-mani-j avatar Jun 14 '25 15:06 arun-mani-j

Would you check the logs using this script?

SmartManoj avatar Jun 15 '25 07:06 SmartManoj

Would you check the logs using this script?

Ah unfortunately I'm out of my Daytona credits, so all my sandboxes are deleted. :/

arun-mani-j avatar Jun 16 '25 15:06 arun-mani-j

Hi @SmartManoj , I encountered the following error when running the script you mentioned: Command output: mkdir: cannot create directory '/workspace': Permission denied

This came from home/daytona/sessions/{id}/output.log. Have you seen this issue before or know what might be causing it?

fulldiver-ilya avatar Jun 17 '25 06:06 fulldiver-ilya

Which sandbox image did you use?

SmartManoj avatar Jun 17 '25 06:06 SmartManoj

# Runtime container image to use (if not provided, will be built from base_container_image)
#runtime_container_image = ""

looks like it's not configured in my case, which one you suggest?

fulldiver-ilya avatar Jun 17 '25 07:06 fulldiver-ilya

docker.all-hands.dev/all-hands-ai/runtime:0.31-nikolaik

https://github.com/All-Hands-AI/OpenHands/blob/main/docs/usage/runtimes/daytona.mdx#maclinux-1

SmartManoj avatar Jun 17 '25 07:06 SmartManoj

yea found that version in bash scripts, and now it has another issue

INFO:     Application startup complete.
INFO:     Uvicorn running on http://0.0.0.0:4444 (Press CTRL+C to quit)
INFO:     172.20.0.2:58614 - "GET /alive HTTP/1.1" 200 OK
INFO:     172.20.0.2:58620 - "POST /execute_action HTTP/1.1" 200 OK
INFO:     172.20.0.2:58622 - "POST /execute_action HTTP/1.1" 200 OK
INFO:     172.20.0.2:58634 - "POST /execute_action HTTP/1.1" 200 OK
INFO:     172.20.0.2:58650 - "POST /execute_action HTTP/1.1" 200 OK
INFO:     172.20.0.2:58658 - "POST /list_files HTTP/1.1" 200 OK
08:06:57 - openhands:ERROR: action_execution_server.py:578 - HTTP exception occurred: Not Found
INFO:     172.20.0.2:58674 - "POST /update_mcp_server HTTP/1.1" 404 Not Found

...

openhands.runtime.utils.request.RequestHTTPError: Client error '404 Not Found' for url 'https://***.daytona.work/update_mcp_server'

i'll investigate what's happened, and thanks for mentioning image name 👌

@SmartManoj do you have any insights about most breaking changes between 0.31 vs 0.40+ versions? What i should looks first?

fulldiver-ilya avatar Jun 17 '25 08:06 fulldiver-ilya

docker.all-hands.dev/all-hands-ai/runtime:0.31-nikolaik

Sorry, should use the latest version. I thought the script would dynamically update.

docker.all-hands.dev/all-hands-ai/runtime:0.43-nikolaik

SmartManoj avatar Jun 17 '25 08:06 SmartManoj

#9187 solved this using snapshot.

SmartManoj avatar Jun 18 '25 02:06 SmartManoj