Comments (25)
Yes, this solved the problem nicely. Thanks!
from sickgear.docker.
So I've spoken with @JackDandy and we're working on testing a config directive for specifying the default thread stack size. That way we can revert to an Alpine based docker image in master, and just update the config template to specify the stack size to something large enough to keep it working. I haven't yet managed to test the PR over on kulta regarding this new config setting though.
from sickgear.docker.
Which platform are you running the image on? Crashing without any messages makes me think of a recent Docker bug that could be causing the crash.
from sickgear.docker.
I'm running it on CentOS x64.
Docker version 1.12.1, build 23cf638.
from sickgear.docker.
I've tried running it on Docker version 1.12.3, build 6b644ec and the problem remains.
When I run the developer image instead, the webserver crashes but the docker container keeps running. The webserver crashes with the message "The connection was reset" and no further output in the container log. I'm running the container interactively and after the webserver crashes it just hangs silently without writing anything else to stdout or stderr.
from sickgear.docker.
OK, I'll try and reproduce tonight.
from sickgear.docker.
I'm able to reproduce the issue. There doesn't appear to be any messages in the log or console. I'll try and dig deeper and try and find out what is causing the exit.
from sickgear.docker.
@ressu Any progress on this?
from sickgear.docker.
I'm having a hard time getting this resolved, it feels a lot like an musl compatibility issue. I'm currently trying to narrow down what is actually causing the crash and working backwards from there
from sickgear.docker.
The fixed builds should now be up. Took me a bit longer to get around to fixing this than I expected.. The only solution I could find was to switch away from the Alpine based image to Debian based image.
from sickgear.docker.
@ressu this can be fixed by setting the threading stack size in SickGear. Thus we can continue to use Alpine as the base.
These are the refs to the root cause of the issue:
python-pillow/Pillow#1935
python-pillow/Pillow#1935 (comment)
docker-library/python#211 (comment)
https://github.com/voidlinux/void-packages/issues/4147
https://github.com/voidlinux/void-packages/pull/6889/files
from sickgear.docker.
Cool, I personally prefer Alpine over Debian for the Docker image. Pull requests are welcome, I'm not sure when I'm able to revert PR #8 and test with the new threading stack sizes.
from sickgear.docker.
Thanks for looking into the fix, however, shouldn't this one still be open ? Took me a little to dig it up is all
from sickgear.docker.
@timm088 I basically investigated and resolved the issue entirely independently before realising it was fixed on master, because there wasn't an open issue :(
@ressu I do think you should make it clearer to people that this is a problem on dev with a change to the dev branches README. And look at my PR/harass @JackDandy for input :-)
from sickgear.docker.
Ok thanks. Would be great if that pr could be linked here, so I can just pulled it myself?
from sickgear.docker.
from sickgear.docker.
Last commit on master was
9c3a158a5df3aea9f438e3038074282e2fbe5f44
Confirmed my alpine docker container has this same commit in Sickgear / Settings / About
Problem still exists for Alpine docker images on current master branch
No problems if it isn't merged yet, I manually configured notifications via config.ini (which for me is mounted into the container each time)
from sickgear.docker.
I can't find the commit you're referring to... This is the latest commit on the master branch:
from sickgear.docker.
Sorry, i'm referring to the master branch of Sickgear itself, not the dockerfile
i.e., you said its fixed on master? You mean changed to debian, not actually fixed (i.e, threading stack size fixed)
from sickgear.docker.
If you just want to change the SickGear version to fix this issue, you can use this commit from the current hotfix branch: 70edd5462fca9ebdfc4ffa6caef7708219e79704
Then set the stack_size = 327680 under General in config.ini
from sickgear.docker.
Hotfix branch?
There are only two visible branches...
from sickgear.docker.
Did you actually find the commit I referred to?
from sickgear.docker.
You do realise there is no hotfix branch right?
https://github.com/SickGear/SickGear/branches/all
Guessing hotfix branch is private?
You mean this commit? SickGear/SickGear@70edd54
from sickgear.docker.
https://git.kulta.us/SickGear/SickGear/src/hotfix/0.12.26
That's the right commit yes.
from sickgear.docker.
Merged into master on github now for anyone else that might come across this.
SickGear/SickGear@2c2a5b1
from sickgear.docker.
Related Issues (5)
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from sickgear.docker.