[SlurmTopo] cookstyle

Author: Himani Anil Deshpande

cookbooks/aws-parallelcluster-slurm/attributes/slurm_attributes.rb

@@ -27,4 +27,4 @@
 default['cluster']['pyxis']['runtime_path'] = '/run/pyxis'
 
 # Block Topology Plugin
-default['cluster']['slurm']['block_topology']['force_configuration'] = false
+default['cluster']['slurm']['block_topology']['force_configuration'] = false

cookbooks/aws-parallelcluster-slurm/resources/block_topology/partial/_block_topology_common.rb

@@ -74,7 +74,5 @@ def topology_generator_command_args
 def topology_generator_extra_args
   if ['true', 'yes', true].include?(node['cluster']['slurm']['block_topology']['force_configuration'])
     " --force-configuration"
-  else
-    nil
   end
-end
+end

cookbooks/aws-parallelcluster-slurm/spec/unit/resources/block_topology_spec.rb

@@ -61,15 +61,15 @@ def self.update(chef_run)
               .with(group: 'root')
               .with(mode: '0644')
           end
-          command =  "#{cookbook_env}/bin/python #{script_dir}/slurm/pcluster_topology_generator.py" \
+          command = "#{cookbook_env}/bin/python #{script_dir}/slurm/pcluster_topology_generator.py" \
             " --output-file #{slurm_install_dir}/etc/topology.conf" \
             " --block-sizes #{block_sizes}" \
             " --input-file #{cluster_config}"
-          if ['true', 'yes', true].include?(force_configuration)
-            command_to_exe =  "#{command}#{force_configuration_extra_args}"
-          else
-            command_to_exe = "#{command}"
-          end
+          command_to_exe = if ['true', 'yes', true].include?(force_configuration)
+                             "#{command}#{force_configuration_extra_args}"
+                           else
+                             "#{command}"
+                           end
           it 'generates topology config when block sizes are present' do
             expect(chef_run).to run_execute('generate_topology_config')
               .with(command: command_to_exe)
@@ -115,11 +115,11 @@ def self.update(chef_run)
               " --output-file #{slurm_install_dir}/etc/topology.conf" \
               " --input-file #{cluster_config}"\
               "#{topo_command_args}"
-            if ['true', 'yes', true].include?(force_configuration)
-              command_to_exe =  "#{command}#{force_configuration_extra_args}"
-            else
-              command_to_exe = "#{command}"
-            end
+            command_to_exe = if ['true', 'yes', true].include?(force_configuration)
+                               "#{command}#{force_configuration_extra_args}"
+                             else
+                               "#{command}"
+                             end
 
             it 'creates the topology configuration template' do
               expect(chef_run).to create_template("#{slurm_install_dir}/etc/slurm_parallelcluster_topology.conf")
@@ -132,12 +132,12 @@ def self.update(chef_run)
             if topo_command_args.nil?
               it 'update or cleanup topology.conf when block sizes are present' do
                 expect(chef_run).not_to run_execute('update or cleanup topology.conf')
-                                      .with(command: command_to_exe)
+                  .with(command: command_to_exe)
               end
             else
               it 'update or cleanup topology.conf when block sizes are present' do
                 expect(chef_run).to run_execute('update or cleanup topology.conf')
-                                      .with(command: command_to_exe)
+                  .with(command: command_to_exe)
               end
             end