AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

Fix docker usage readme

Open pkqs90 opened this issue 2 years ago • 2 comments

Background

Since Dockerfile uses /home/appuser as workdir, the mounted volume should also be /home/appuser, or else we can't persist the written files.

https://github.com/Significant-Gravitas/Auto-GPT/blob/master/Dockerfile#L25

Changes

Documentation

Test Plan

Tested locally.

PR Quality Checklist

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

pkqs90 avatar Apr 18 '23 16:04 pkqs90

The docker image in latest master in deed uses that workdir.

=> CACHED [ 7/11] WORKDIR /home/appuser                                                                                                                                                        ```

hdkiller avatar Apr 20 '23 04:04 hdkiller

LGTM. There are two more occurrences of /app in docker-compose.yml.

np avatar Apr 20 '23 15:04 np

LGTM. There are two more occurrences of /app in docker-compose.yml.

Updated

pkqs90 avatar Apr 21 '23 06:04 pkqs90