[SlurmTopo] Update the not_if guard for pcluster_topology_generator.py

* We do not update the file if queues are not updated and p6egb200_block_sizes does not exist

Author: Himani Anil Deshpande

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

@@ -51,7 +51,7 @@
               " --input-file #{node['cluster']['cluster_config_path']}"\
               "#{topology_generator_command_args}"\
               "#{topology_generator_extra_args}"
-    not_if { ::File.exist?(node['cluster']['previous_cluster_config_path']) && topology_generator_command_args.nil? }
+    not_if { topology_generator_command_args.nil? }
   end
 end
 

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

@@ -129,10 +129,18 @@ def self.update(chef_run)
                 .with(mode: '0644')
             end
 
-            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)
+            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)
+              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)
+              end
             end
+
           end
         end
       end