Skip to content

Commit

Permalink
tests(slurm_ops): set StateSaveLocation in integration tests
Browse files Browse the repository at this point in the history
Slurm 23.11 requires permissions to be properly set on the `StateSaveLocation`.
The default is `/var/spool` which isn't set up the way Slurm wants. Instead
provide `/var/lib/slurm/checkpoint` which is set up the way Slurm wants
by the Debian package.

Signed-off-by: Jason C. Nucciarone <[email protected]>
  • Loading branch information
NucciTheBoss committed Dec 6, 2024
1 parent d27aa61 commit 8ed2d81
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions tests/integration/slurm_ops/test_manager.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ def test_slurm_config(slurmctld: SlurmctldManager) -> None:
"""Test that the slurm config can be changed."""
with slurmctld.config.edit() as config:
config.slurmctld_host = [slurmctld.hostname]
config.state_save_location = "/var/lib/slurm/checkpoint"
config.cluster_name = "test-cluster"

for line in str(slurmctld.config.load()).splitlines():
Expand Down

0 comments on commit 8ed2d81

Please sign in to comment.