If we're going to batch dry run, then let's actually use the results

Gemfile

@@ -4,4 +4,4 @@ source 'https://rubygems.org'
 # Specify your gem's dependencies in krane.gemspec
 gemspec
 
-gem "ruby-lsp", "~> 0.2.0", :group => :development
+gem "ruby-lsp", "~> 0.2.0", group: :development

lib/krane/deploy_task.rb

@@ -286,15 +286,16 @@ def validate_configuration(prune:)
 
     def validate_resources(resources)
       validate_globals(resources)
-      batch_dry_run_success = validate_dry_run(resources)
       resources.select! { |r| r.selected?(@selector) } if @selector_as_filter
+      applyables, individuals = resources.partition { |r| r.deploy_method == :apply }
+      begin
+        batch_dry_run_success = validate_dry_run(applyables)
+      rescue FatalDeploymentError => e
+        raise FatalDeploymentError, "Template validation failed"
+      end
       Krane::Concurrency.split_across_threads(resources) do |r|
         # No need to pass in kubectl (and do per-resource dry run apply) if batch dry run succeeded
-        if batch_dry_run_success
-          r.validate_definition(kubectl: nil, selector: @selector, dry_run: false)
-        else
-          r.validate_definition(kubectl: kubectl, selector: @selector, dry_run: true)
-        end
+        r.validate_definition(kubectl: nil, selector: @selector, dry_run: false)
       end
       failed_resources = resources.select(&:validation_failed?)
       if failed_resources.present?

lib/krane/resource_deployer.rb

@@ -169,7 +169,7 @@ def apply_all(resources, prune, dry_run: false)
         if st.success?
           log_pruning(out) if prune
         else
-          record_apply_failure(err, resources: resources) unless dry_run
+          record_apply_failure(err, resources: resources)
           raise FatalDeploymentError, "Command failed: #{Shellwords.join(command)}"
         end
       end