Fixes #36694 - Show failed resources on installation

Kafo 7.1.0 has started to capture Puppet reports and in 7.2.0 it was
extended for better messages. This allows showing the failed resources
which makes it easier to troubleshoot failures.  Ideally speaking the
user doesn't have to open the installer log anymore and the regular
installer command is sufficient.

Gemfile

@@ -4,7 +4,7 @@ source 'https://rubygems.org'
 # We don't want to list psych since that updates the bundled version
 gem 'rdoc', '< 6.4'
 
-gem 'kafo', '>= 6.4', '< 8'
+gem 'kafo', '>= 7.2', '< 8'
 gem 'librarian-puppet', '>= 3.0'
 gem 'puppet', ENV.key?('PUPPET_VERSION') ? "~> #{ENV['PUPPET_VERSION']}" : '~> 7.0'
 gem 'facter', '>= 3.0', '!= 4.0.52'

hooks/boot/02-message-helpers.rb

@@ -38,13 +38,35 @@ def dev_new_install_message(kafo)
   end
 
   def failure_message
-    say "\n  <%= color('There were errors detected during install.', :bad) %>"
-    say "  Please address the errors and re-run the installer to ensure the system is properly configured."
-    say "  Failing to do so is likely to result in broken functionality."
+    message = 'There were errors detected during installation.'
+
+    if puppet_report&.failed_resources&.any?
+      say "\n"
+
+      puppet_report.failed_resources.each_with_index do |failed_resource, index|
+        say "Error #{index + 1}: #{failed_resource} failed. Logs:"
+        failed_resource.log_messages_by_source.each do |source, log_messages|
+          say "  #{source}"
+          log_messages.each do |log_message|
+            say "    #{log_message}"
+          end
+        end
+      end
+
+      message = if puppet_report.failed_resources.length == 1
+                  '1 error was detected during installation.'
+                else
+                  "#{puppet_report.failed_resources.length} errors were detected."
+                end
+    end
+
+    say "\n<%= color('#{message}', :bad) %>"
+    say "Please address the errors and re-run the installer to ensure the system is properly configured."
+    say "Failing to do so is likely to result in broken functionality."
   end
 
   def log_message(kafo)
-    say "\n  The full log is at <%= color('#{kafo.config.log_file}', :info) %>"
+    say "\nThe full log is at <%= color('#{kafo.config.log_file}', :info) %>"
   end
 end