Closed marmistrz closed 5 years ago
The PR is basically ready, we're only waiting for the GU blockers.
We'll probably need to have --cap-add=SYS_PTRACE
for MPI to work under Docker efficiently. https://github.com/open-mpi/ompi/issues/4948
Because this PR was pending for a very long time, I'll merge this PR without a review as an exception. @kubkon, please take a look when you have a while, I'll address the issues (if any) in a subsequent PR.
This change takes advantage of the Docker capabilities present in golem-unlimited. Closes #12. Closes #2. Closes #29.
Changes:
/app/
,/output/
,/input/
) and a default location for the output directory (/output
) has been addedTODO:
output
[needs-gu-changes]mpirun
[needs-gu-changes]mv_cmd
)mca btl
hack (nope)SYS_PTRACE
capabilityTODO-GU:
GU blockers:
Command::Exec
doesn't return proper errors in case the program execution fails under dockerman: https://github.com/golemfactory/golem-unlimited/issues/250Deferred issues: (follow-up issues will be created)
/dev/shm
, Docker limits it to 64M. Probably using--tmpfs /dev/shm
should be enough to guarantee unlimited/dev/shm