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

Restart troubleshooting documentation. #5317

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Conversation

mamoodi
Copy link
Collaborator

@mamoodi mamoodi commented Nov 28, 2024

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

  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

Restart troubleshooting documentation.


Give a summary of what the PR does, explaining any non-trivial design decisions

Restarting this documentation with up to date information. Will try to fill this up appropriately. Ideally it should not be filled with bugs that can be resolved through code.


Link of any specific issues this addresses


To run this PR locally, use the following command:

docker run -it --rm   -p 3000:3000   -v /var/run/docker.sock:/var/run/docker.sock   --add-host host.docker.internal:host-gateway   -e SANDBOX_RUNTIME_CONTAINER_IMAGE=docker.all-hands.dev/all-hands-ai/runtime:0be1496-nikolaik   --name openhands-app-0be1496   docker.all-hands.dev/all-hands-ai/openhands:0be1496

But clicking "Resume" still starts a fresh new chat.

**Details**
**Description**
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kept this Sessions are not restored here. But I feel like troubleshooting is not a good place for it.
Is there a better place to move it to?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

IIRC Robert was suggesting an Advanced Configuration section, and linked from the main README. That could work?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Alright. Yeah just remembered the thread. I'll take a stab at it.


## How to Persist Session Data

### Development Workflow
Copy link
Collaborator Author

@mamoodi mamoodi Nov 30, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I got this to work with the below instructions but could not get it to work using the docker command. I ran the docker command with:
-e FILE_STORE="local" -e FILE_STORE_DATA="<path>" -e JWT_SECRET="secretpass"

What am I missing?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

-e FILE_STORE="local" -e FILE_STORE_DATA="" -e JWT_SECRET="secretpass"

Isn't it FILE_STORE_DATA -> FILE_STORE_PATH instead?

@@ -63,6 +63,9 @@ run OpenHands in a scriptable [headless mode](https://docs.all-hands.dev/modules
interact with it via a [friendly CLI](https://docs.all-hands.dev/modules/usage/how-to/cli-mode),
or run it on tagged issues with [a github action](https://github.com/All-Hands-AI/OpenHands/blob/main/openhands/resolver/README.md).

Follow the [Persisting Session Data](https://docs.all-hands.dev/modules/usage/how-to/persist-session-data) guide if you
would like your session data to persist.
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@enyst and @rbren I am confused about something. If this is something that's important enough to mention in the README, does this mean this is a bug that we currently don't persist the session data?
Also if it's this important, why would we not add this to the docker command above?

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.

2 participants