llnl / llnl/benchpark

Test dynamic AWS system

Open
#470 0 comments 0 reactions 1 assignee View on GitHub

@wdhawkins is already working on this.

Since Dec 3, 2024.

Dominant language
Python
Stars
78
Forks
52
Avg merge
4d 19m
Merged PRs (30d)
10

Description

@wdhawkins The AWS system definition has been updated (systems/aws-pcluster), please test and let us know if you think it does what it should. Right now, this produces experiment scripts like:

```
#SBATCH --mpi=pmix --export=ALL,FI_EFA_USE_DEVICE_RDMA=1,FI_PROVIDER="efa",OMPI_MCA_mtl_base_verbose=100
#SBATCH -n 8
#SBATCH -N 1
...
srun --mpi=pmix --export=ALL,FI_EFA_USE_DEVICE_RDMA=1,FI_PROVIDER="efa",OMPI_MCA_mtl_base_verbose=100 -n 8 -N 1 saxpy -n 512 >> ...
```

(i.e. these are generated as sbatch opts and command opts) - this is decided by the allocation modifier (specifically for the SLURM scheduler). I think it's worth checking if these should be batch opts

* If so, this can be rewritten to generate a separate batch opt for `--mpi` and `--export`
* If not, I can change the allocation modifier

I don't think this decision need stop this PR, but the generated experiment scripts will have weird sbatch directives until that is decided.

Contributor guide

No contributing guide indexed for this repository

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.