-
Notifications
You must be signed in to change notification settings - Fork 4.8k
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
Docker build failure - Add Tini and set version range for numpy #1514
Conversation
Thank you I had the same issue, I hope this is integrated soon |
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.
Had one request and one question, otherwise looks reasonable to me.
@@ -49,13 +54,14 @@ RUN mkdir ${PROJECT_DIR} \ | |||
|
|||
WORKDIR /ta-lib | |||
|
|||
RUN pip install numpy==1.9.2 \ | |||
RUN pip install 'numpy>=1.11.1,<2.0.0' \ |
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.
Latest zipline is also on pandas 0.18.1 and scipy 0.17.1 if you want to bump those as well.
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.
Good point, latest commit sets the version range for pandas and scipy.
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.
Please note I added && pip install matplotlib \
to the Dockerfile in order to fix an error when executing zipline/examples/buyapple.ipynb .
ENV TINI_VERSION v0.10.0 | ||
ADD https://github.com/krallin/tini/releases/download/${TINI_VERSION}/tini /tini | ||
RUN chmod +x /tini | ||
ENTRYPOINT ["/tini", "--"] |
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.
Is there a good executive summary somewhere of the benefits of using tini
as the container pid 1? I'm aware vaguely that it solves issues with zombie processes not being correctly reaped, but I don't have a clear sense of concrete cases where it fixes issues.
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.
I didn't dig much into it, but this is the best explanation I found krallin/tini#8 .
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.
For what it's worth I am running it with docker-compose .
👍, thanks @fzagarzazu! |
First time user, I spent some time setting it up and I ran into a few issues, the purpose of this pull request is to bring it to your attention, maybe it will help somebody else. Please close it if it doesn't apply.
Background:
Docker build failure:
Possible solution / workaround:
Jupyter notebooks: Dead kernels / automatic restart failures.
Thank you for your work.