-
Notifications
You must be signed in to change notification settings - Fork 338
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
add Dockerfile #30
base: main
Are you sure you want to change the base?
add Dockerfile #30
Conversation
3f7fc6c
to
fd81a7f
Compare
Nothing to do with my day job, no conflict of interest or IP claims - all my own sweat and tears @microsoft-github-policy-service agree |
caae392
to
ea2c291
Compare
Add a post-build script that runs the first time you execute it on a machine with a GPU, since some stuff won't build in the builder Changed app.py to serve on 0.0.0.0, 'cause otherwise we can't expose the ports
Dockerfile
Outdated
# git and build-essential are needed for post_install.sh script. | ||
# vim and strace are useful for debugging, remove those if you want to. | ||
RUN apt update && \ | ||
apt upgrade && \ |
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.
Maybe missing a -y
option here in apt upgrade
? Otherwise docker will fail:
Need to get xxxMB of archives.
After this operation, xxxMB of additional disk space will be used.
Do you want to continue? [Y/n] Abort.
ERROR: failed to solve: process "/bin/sh -c apt upgrade" did not complete successfully: exit code: 1
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.
oof! Yes I added this without testing it. You wouldn't believe how many times I've rebuilt this image! Thanks :)
Add a Dockerfile, a script to start it on the machine, and a post-build script that runs the first time you execute it on a machine with a GPU (some things won't build without CUDA in the builder, which is currently unsupported by buildx)