ExaWorks / SDK

ExaWorks SDK
11 stars 12 forks source link

Docker image for Swift/T #7

Closed dongahn closed 3 years ago

dongahn commented 3 years ago

Creating a new subtask by breaking down from #2. Docker image with the make-check running.

dongahn commented 3 years ago

Tagging @jmjwozniak

dongahn commented 3 years ago

@jmjwozniak: now the base image has been merged, you can take a look.

j-woz commented 3 years ago

So inside SDK/ , I should create a new /docker/swift-t/Dockerfile ?

j-woz commented 3 years ago

That skeleton is almost complete, currently stuck on Tcl.

j-woz commented 3 years ago

Ok I think this works now - is someone interested in testing?

j-woz commented 3 years ago

Looking back at #2 it looks like I should have done OpenMPI first. I will do that now. How will we orchestrate testing both?

dongahn commented 3 years ago

@jmjwozniak: sorry for my late response. I was on vacation (and off the grid :-)

How will we orchestrate testing both?

I think testing orchestration needs to be done at the CI level, initially using GitHub Action. We probably don't need to run tests on applications that use both MPI implementations?

dongahn commented 3 years ago

FYI -- related to the need for multiple MPI coverage: https://github.com/flux-framework/flux-core/issues/3592#issuecomment-816346144

j-woz commented 3 years ago

For multiple MPI, I meant separately testing OpenMPI and MPICH in totally different images/builds

dongahn commented 3 years ago

Yes, that makes a lot of sense.

j-woz commented 3 years ago
SteVwonder commented 3 years ago

I think this can be closed now since we have a separate issue tracking the MPI ARG todo. Please re-open if not.