Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

hotfix: change default RUN_AS_DEVIN to true and change some network setting #895

Merged
merged 2 commits into from
Apr 8, 2024

Conversation

yufansong
Copy link
Collaborator

@yufansong yufansong commented Apr 8, 2024

Quick fix the problem #887 introduced in #847. Firstly change the RUN_AS_DEVIN: 'true'.
After changing also meet problem like

pexpect.pxssh.ExceptionPxssh: Could not establish connection to host

After trying locally with ssh -p 2222 opendevin@localhost still will meet.
So do little change to the docker network setting. (not sure this change from my local env setting, or a wrong docker network configruation)

@yufansong yufansong merged commit 6e566dd into All-Hands-AI:main Apr 8, 2024
@yufansong yufansong changed the title fix: change default RUN_AS_DEVIN to true and change some network setting hotfix: change default RUN_AS_DEVIN to true and change some network setting Apr 8, 2024
@Luckytiger666
Copy link

same problem, met

pexpect.pxssh.ExceptionPxssh: Could not establish connection to host

@yufansong
Copy link
Collaborator Author

yufansong commented Apr 8, 2024

same problem, met

pexpect.pxssh.ExceptionPxssh: Could not establish connection to host

I can run correctly after my change.
Do my new change about netwrok setting can fix it? Or you still meed the problem after my change?

@Luckytiger666
Copy link

After.
Should I merge xingyaoww's commits too?

@xingyaoww
Copy link
Contributor

@Luckytiger666

Can you try running on #891, too? I didn't encounter any issues with my local setup. Can you try first running docker ps to see if there's a container named sandbox-XXX created by opendevin, then try to docker stop these containers? After this, try rerun the whole thing?

@Luckytiger666
Copy link

restart the container, and it works fine now

lehcode pushed a commit to lehcode/oppendevin that referenced this pull request Apr 10, 2024
@marikan114
Copy link

It's better to first check with ssh -p 2222 opendevin@localhost. I found that there is an old host key.

Offending RSA key in ~/.ssh/known_hosts:13

Delete line 13 from the known_hosts file, save, and restart to fix the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants