Fix logic to reset Slurm accounting password after queue update

Signed-off-by: Jacopo De Amicis <[email protected]>
aws · Apr 18, 2023 · 8daa943 · 8daa943
1 parent e70cff8
commit 8daa943
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/cookbooks/aws-parallelcluster-slurm/recipes/update_head_node.rb b/cookbooks/aws-parallelcluster-slurm/recipes/update_head_node.rb
@@ -167,7 +167,8 @@ def update_nodes_in_queue(strategy, queues)
   user 'root'
   group 'root'
   command "#{node['cluster']['scripts_dir']}/slurm/update_slurm_database_password.sh"
-  not_if { ::File.exist?(node['cluster']['previous_cluster_config_path']) && !are_queues_updated? && node['cluster']['config'].dig(:Scheduling, :SlurmSettings, :Database).nil? }
+  # This horrible only_if guard is needed to cover all cases that trigger "generate_pcluster_slurm_settings", in the case Slurm accounting is being used
+  only_if { !(::File.exist?(node['cluster']['previous_cluster_config_path']) && !are_queues_updated?) && !node['cluster']['config'].dig(:Scheduling, :SlurmSettings, :Database).nil? }
 end
 
 # Generate custom Slurm settings include files