Skip to content

Commit

Permalink
Change suffix for external dbd unit test case
Browse files Browse the repository at this point in the history
The suffix `_external_slurmdbd` led to confusion with the
base name of the `slurmdbd.conf` include file generated
by ParallelCluster.

Switch to `_externaldbd` suffix to for more clarity.

Signed-off-by: Jacopo De Amicis <jdamicis@amazon.it>
  • Loading branch information
jdeamicis committed Jan 12, 2024
1 parent 1cbdced commit b47b934
Show file tree
Hide file tree
Showing 4 changed files with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions test/unit/slurm/test_slurm_config_generator.py
Original file line number Diff line number Diff line change
Expand Up @@ -129,9 +129,10 @@ def test_generate_slurm_config_files_memory_scheduling(
id="Case with Slurm Accounting passing DatabaseName",
),
pytest.param(
"sample_input_external_slurmdbd.yaml",
"sample_input_externaldbd.yaml",
[
"slurm_parallelcluster_external_slurmdbd.conf",
"slurm_parallelcluster_externaldbd.conf",
"slurm_parallelcluster_slurmdbd_externaldbd.conf",
],
id="Case with Slurmdbd daemon external to the cluster",
),
Expand Down

0 comments on commit b47b934

Please sign in to comment.