-
Notifications
You must be signed in to change notification settings - Fork 30
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 setup #826
Docker setup #826
Conversation
launch jenkins |
launch jenkins |
launch jenkins |
launch jenkins |
Dockerfile
Outdated
|
||
FROM dawn-gcc9-env AS dawn-gcc9 | ||
# IMAGE needs be be set to one of the docker/dawn-env.dockerfile images | ||
ARG IMAGE=gtclang/dawn-env-cuda10.1-ubuntu18.04 |
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.
if we now have a docker folder, should we put this file also in there (and adapt the path on dockerhub?)
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'm not sure... Sometimes it's nice to have a top-level Dockerfile
that people can immediately build. Something like a Makefile
used to be.
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.
But I'm open to either way! Let me know.
launch jenkins |
launch jenkins |
Everything passes so I'm going to go ahead and merge this now. |
Technical Description
Use Sarus for containerized tests of Dawn.
To do before merging:
scripts/update-images
)sarus pull $image
is the correct usage -- sarus run --pull eth-cscs/sarus#6scripts/jenkins/srun_dawn_PR.sh
toscripts/daint-dawn-PR
Resolves / Enhances
Resolves #623.
Testing
http://localhost:8080/view/PASCHA/job/dawn_gtclang_clang-gridtools_PR/490/