search

jgeusebroek / docker-spotweb

A docker image running ubuntu/20.04 Linux and Spotweb
47 stars 27 forks source link

Chowning of the spotweb folder #11

Closed riemers closed 5 years ago

riemers commented 5 years ago

Hi there, i am in the middle of jumping over to this image since i have a very old one that is pretty much broken.

So far its been running for 15 minutes and it has not finished the chown on the spotweb folder. Mostly because i have my cache folder mounted on the filesystem. That folder is 6.5gb and has a ton of folders too. At some point keeping the cache sounded like a good idea but now it leaves me with 2 questions.

I don't think there is an option inside docker to detect if a folder is mounted from outside, that way we could ignore it. What are your thoughts?

riemers commented 5 years ago

Just as a extra feedback.

The '' quotes on the cron examples can be left since if you copy those the crontab will fail. (easy mistake) If you have your spotweb in /spotweb, one needs to change the virtualhost to point to /var/www instead. You could add a environment option that does a inline sed to remove /spotweb. (could do a PR but have no time to test, kids screaming now 👶)

jgeusebroek commented 5 years ago

Thanks for your feedback. You do not need to mount your cache folder. I don't at least :)

If you have your spotweb in /spotweb, one needs to change the virtualhost to point to /var/www instead.

I don't really understand what you mean, the image has it in it's own specified folder and everything works fine?

What's the cron error you are receiving?

riemers commented 5 years ago

i have my spotweb in /spotweb, not as spotweb.domain.com hence the need for /spotweb.

The cron error if you add '' in the line is that it cannot parse it. So in the example if you copy paste it it won't work. Not a showstopper, but no harm done to tell right.

My cache folder was huge at some point though, perhaps its better to just clean it every now and then to keep the speed in it. Not like you cannot redownload an image unless it points to a expired link.

jgeusebroek commented 5 years ago

I clarified cron usage in the readme. Thanks!