search

wiserain / docker-flexget

Docker image for running Flexget
https://hub.docker.com/r/wiserain/flexget
68 stars 40 forks source link

Fix data path permission #6

Closed einverne closed 4 years ago

wiserain commented 4 years ago

Do you think this is really necessary? Because we don't even know what would be and how many files in /data. If there are millions of small files, it will keep changing their ownership every container restart.

einverne commented 4 years ago

Oh, sorry. I have to mention my problem here. When I start this container, I see log error show it cannot write to /data path, because it doesn't have the permission to write.

image

The id I log in is mi, which uid and gid are both 1000. I didn't create /home/mi/Documents/flexget1/data and /home/mi/Documents/flexget1/config first. The path is created by the docker command. Am I wrong with it?

➜ sudo docker run -d \
    --name=flexget1 \
    -p 3549:3539 \
    -v /home/mi/Documents/flexget1/data:/data \
    -v /home/mi/Documents/flexget1/config:/config \
    -e FG_WEBUI_PASSWD=password \
    -e FG_LOG_LEVEL=info \
    -e PUID=1000 \
    -e PGID=1000 \
    -e TZ=Asia/Shanghai \
    wiserain/flexget
wiserain commented 4 years ago

Oh, I see the problem. I will make further changes after merge this PR.

einverne commented 4 years ago

Thank you for following up. And thank you for this great image. Thanks. :)