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.

Sign up for GitHub

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

Jump to bottom

Docker image uses wrong path for default folder #6234

Closed
calmh opened this issue Dec 22, 2019 · 0 comments
Closed

Docker image uses wrong path for default folder #6234

calmh opened this issue Dec 22, 2019 · 0 comments
Labels
bug A problem with current functionality, as opposed to missing functionality (enhancement) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Milestone
v1.3.4

Comments

@calmh
Copy link
Member

calmh commented Dec 22, 2019 

jb@kvin:~ % docker run -p 8384:8384 -p 22000:22000 \
    -v /tmp/syncthing:/var/syncthing \
    syncthing/syncthing:latest
[monitor] 08:51:47 INFO: Default folder created and/or linked to new config
[monitor] 08:51:47 INFO: Starting syncthing
[start] 08:51:47 INFO: syncthing v1.3.2 "Fermium Flea" (go1.13.4 linux-amd64) docker@syncthing.net 2019-11-24 08:33:58 UTC
...
[CA2SF] 08:51:48 WARNING: Failed to create folder root directory mkdir /Sync: permission denied

It should use /var/syncthing/Sync.
@calmh calmh added the bug A problem with current functionality, as opposed to missing functionality (enhancement) label Dec 22, 2019
calmh added a commit to calmh/syncthing that referenced this issue Dec 22, 2019
@calmh
docker: Fix HOME setting (fixes syncthing#6234)
a136fb6 
su-exec sets $HOME, and we used to have this env call in there to fix
that up. It disappeared in the latest entrypoint.sh rewrite.
calmh added a commit to calmh/syncthing that referenced this issue Jan 4, 2020
@calmh
Merge branch 'master' into badger
54fe466 
* master:
  gui, lib/model: Prevent negative sync completion (fixes syncthing#4570) (syncthing#6248)
  build, pmp: Replace fork with upstream for go-nat-pmp and tidy go.mod (syncthing#6247)
  gui: Missing line break (fixes syncthing#6240) (syncthing#6241)
  docker: Fix HOME setting (fixes syncthing#6234) (syncthing#6235)
  golangci: Skip godox
  build(deps): bump github.com/mattn/go-isatty from 0.0.10 to 0.0.11 (syncthing#6231)
  lib/upgrade: Let Mac load .zip archives (syncthing#6230)
  lib/api: Prevent leaks in tests (syncthing#6227)
  lib/syncthing: Expose backend instead of lowlevel (syncthing#6224)
  gui: Prevent spurious api call (fixes syncthing#6222) (syncthing#6223)
  gui: Fix a typo in a class name (syncthing#6220)
  gui: Allow to degrade 'Automatic upgrades' option to 'No upgrades' (fixes syncthing#6044) (syncthing#6168)
  lib/db, lib/protocol: Compact FileInfo and BlockInfo alignment (syncthing#6215)
@calmh calmh added this to the v1.3.4 milestone Jan 13, 2020
@st-review st-review added the frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion label Dec 22, 2020
@syncthing syncthing locked and limited conversation to collaborators Dec 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
bug A problem with current functionality, as opposed to missing functionality (enhancement) frozen-due-to-age Issues closed and untouched for a long time, together with being locked for discussion
Projects
None yet
Milestone
v1.3.4
Development

No branches or pull requests
2 participants
@calmh @st-review