search

ExaWorks / SDK

ExaWorks SDK
11 stars 12 forks source link

Add flux dockerfile #17

Closed SteVwonder closed 3 years ago

SteVwonder commented 3 years ago

Builds on top of #16. It was only after trying to compile Flux that I realized the python36-cffi package is only in EPEL. So this PR adds EPEL before installing all of the dependency packages. Otherwise, the previous RUN commands from #16 should be the same.

This PR ensures that flux is built correctly by running make check as part of the image build. It also adds a small test.sh that ensures flux starts and can run a simple single-core job.

While the Dockerfile is technically parameterized to support different versions of flux-core, I only tested that the latest release (0.24.0) builds and runs successfully.

Closes #4

dongahn commented 3 years ago

From our today's discussion, @SteVwonder will add a few modification and force a push.

dongahn commented 3 years ago

FYI -- need to be rebased before going in.

SteVwonder commented 3 years ago

Thanks @dongahn . Added a newline to the docker file, added the flux image to the CI, and rebase onto master. Let's hope the CI tests still pass after the new addition. 🤞

dongahn commented 3 years ago

Great. Merging.