Fix failing unit tests and cookstyle linter

Add ExternalSlurmdbd subsection in unit tests for the generation of the Slurm configuration. Signed-off-by: Jacopo De Amicis <[email protected]>
jdeamicis · Feb 5, 2024 · 2b4af49 · 2b4af49
1 parent cdce680
commit 2b4af49
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 1 deletion.
diff --git a/cookbooks/aws-parallelcluster-slurm/recipes/config/config_slurm_accounting.rb b/cookbooks/aws-parallelcluster-slurm/recipes/config/config_slurm_accounting.rb
@@ -15,7 +15,6 @@
 # OR CONDITIONS OF ANY KIND, express or implied. See the License for the specific language governing permissions and
 # limitations under the License.
 
-
 template '/etc/systemd/system/slurmdbd.service' do
   source 'slurm/head_node/slurmdbd.service.erb'
   owner 'root'

diff --git a/...rm/test_slurm_config_generator/test_generate_partition_nodelist_mapping/sample_input.yaml b/...rm/test_slurm_config_generator/test_generate_partition_nodelist_mapping/sample_input.yaml
@@ -4,6 +4,7 @@ Scheduling:
     ScaledownIdletime: 10
     EnableMemoryBasedScheduling: false
     Database: null
+    ExternalSlurmdbd: null
   SlurmQueues:
     - Name: queue1
       ComputeResources:

diff --git a/..._slurm_config_generator/test_generate_slurm_config_with_custom_settings/sample_input.yaml b/..._slurm_config_generator/test_generate_slurm_config_with_custom_settings/sample_input.yaml
@@ -88,3 +88,4 @@ Scheduling:
     ScaledownIdletime: 10
     EnableMemoryBasedScheduling: false
     Database: null
+    ExternalSlurmdbd: null
diff --git a/..._slurm_config_generator/test_generate_slurm_config_with_job_exc_alloc/cluster_config.yaml b/..._slurm_config_generator/test_generate_slurm_config_with_job_exc_alloc/cluster_config.yaml
@@ -34,3 +34,4 @@ Scheduling:
     ScaledownIdletime: 10
     EnableMemoryBasedScheduling: false
     Database: null
+    ExternalSlurmdbd: null