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

Update example docker docs to work out of the box #1844

Closed
wants to merge 1 commit into from

Conversation

stankud
Copy link

@stankud stankud commented Apr 16, 2023

Background

This change attempts to provide a more streamlined user experience by updating the documentation in the docker section to work out of the box

Changes

The "Bind mount a volume" (-v) param value in the example docker command changes the target from /app/auto_gpt_workspace to the actual directory used by autogpt by default, which is /home/appuser/auto_gpt_workspace

Documentation

This change is to the actual documentation

Test Plan

I tested this by starting a docker container with the updated -v param value and insured that the files autogpt wrote to auto_gpt_workspace folder were also persisted on my local machine in auto_gpt_workspace folder.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes

No tests need because it's a documentation change.

@stankud stankud changed the title update example docker command to mount volume at path used by autogpt Update example docker docs to work out of the box Apr 16, 2023
@Pwuts
Copy link
Member

Pwuts commented Apr 16, 2023

#1843 would also fix it, by changing back the workdir 😅

@stankud
Copy link
Author

stankud commented Apr 16, 2023

Ah yes - that's probably a better way to solve this. I'll close.

@stankud
Copy link
Author

stankud commented Apr 16, 2023

#1843 fixes this

@stankud stankud closed this Apr 16, 2023
@stankud stankud deleted the fix-docker-docs branch April 16, 2023 12:01
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