-
-
Notifications
You must be signed in to change notification settings - Fork 149
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
Fix the docker run command in README.md #278
Conversation
Include volume mounting, time zone, -d and better formatting
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM :)
--restart unless-stopped \ | ||
-p 3100:7745 \ | ||
-e TZ=Europe/Bucharest \ | ||
-v ${HOME}/.homebox:/data \ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you think should it be a bind mount instead of a volume?
--
I'm not sure I want to advocate people use their home directory over the named volume.
Given
Line 47 in 673db41
VOLUME [ "/data" ] |
the data should be persistent anyways
🤷
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you should mount the volume in a defined location for easy backups and access. home directory makes sense as there are the... user directories, configs and data. all backup solutions include the home directory by default.
if you really don't want to use the home directory as example, you can use -v /path/to/data/folder:/data \
in the docker run, like linuxserver guys do it.
* fix potential memory leak with time.After * add new background service to manage scheduled notifications * update docs * remove old js reference * closes #278 * tidy
* fix potential memory leak with time.After * add new background service to manage scheduled notifications * update docs * remove old js reference * closes hay-kot#278 * tidy
What type of PR is this?
What this PR does / why we need it:
Include volume mounting, time zone,
-d
and better formatting in the README.mdWhich issue(s) this PR fixes:
none
Release Notes