OpenHands icon indicating copy to clipboard operation
OpenHands copied to clipboard

Feat add agent manager

Open iFurySt opened this issue 1 year ago • 1 comments

modified includes:

  • add agent manager to manage all agents;
  • extract the host ssh port to prevent conflict.
  • clean all containers with the prefix sandbox-

iFurySt avatar Apr 08 '24 14:04 iFurySt

image image

do we decide to merge this? it can't create a new container if previous container is still exist, the hardcode host port 2222 will cause the problem.

iFurySt avatar Apr 10 '24 05:04 iFurySt

Yes, I think this fixes a lot of minor problems with the SSHBox (hard coded port, lingering containers)

rbren avatar Apr 12 '24 08:04 rbren

i've resolved the conflicts, appreciate it if you have time to merge it.

iFurySt avatar Apr 12 '24 10:04 iFurySt

A few quick notes:

  • I changed the prefix to opendevin-sandbox-, which should be a little safer
  • If we instantiate multiple sandboxes in the future, the cleanups might fight each other...may need to rethink that eventually

rbren avatar Apr 12 '24 11:04 rbren

Will merge once tests pass

rbren avatar Apr 12 '24 11:04 rbren

i'm getting an error when starting. noted it here: https://github.com/OpenDevin/OpenDevin/issues/1050

katmai avatar Apr 12 '24 12:04 katmai