search

lanl / BEE

Other
13 stars 3 forks source link

Add test (ci) when slurmrestd is not available #823

Open pagrubel opened 2 months ago

pagrubel commented 2 months ago

We have a path for systems that do not have slurmrestd. We need to test it and make sure it doesn't break.

pagrubel commented 2 months ago

Do we just want to set use_commands to True in the bee.conf file to test this.

jtronge commented 2 months ago

We might be able to test this with the integration CI. We would need to update .github/workflows/integration.yml to also test a slurm+commands instance. Right now that workflow runs two instances of the integration tests, one for slurm+slurmrestd and one for flux.