Update /etc/hosts on all nodes when hostname changes (#228)

Fixes #227

Author: silug

Diff for: CHANGELOG.md

@@ -1,3 +1,7 @@
+### 1.34.3 / 2024-12-20
+* Fixed:
+  * Update /etc/hosts on all nodes when hostname changes (#227)
+
 ### 1.34.2 / 2024-12-19
 * Fixed:
   * Update /etc/hosts when hostname changes (#224)

Diff for: lib/simp/beaker_helpers.rb

@@ -729,7 +729,7 @@ def linux_errata( suts )
       if current_domain.nil? || current_domain.empty?
         new_fqdn = hostname + '.beaker.test'
 
-        safe_sed(sut, "s/#{hostname}.*/#{new_fqdn} #{hostname}/", '/etc/hosts')
+        safe_sed(suts, "s/#{hostname}.*/#{new_fqdn} #{hostname}/", '/etc/hosts')
 
         if !sut.which('hostnamectl').empty?
           on(sut, "hostnamectl set-hostname #{new_fqdn}")
@@ -939,8 +939,13 @@ def rhel_rhsm_unsubscribe(suts)
 
   # Apply known OS fixes we need to run Beaker on each SUT
   def fix_errata_on( suts = hosts )
-    block_on(suts, :run_in_parallel => @run_in_parallel) do |sut|
-      if is_windows?(sut)
+    windows_suts = suts.select { |sut| is_windows?(sut) }
+    linux_suts = suts - windows_suts
+
+    linux_errata(linux_suts) unless linux_suts.empty?
+
+    unless windows_suts.empty?
+      block_on(windows_suts, :run_in_parallel => @run_in_parallel) do |sut|
         # Load the Windows requirements
         require 'simp/beaker_helpers/windows'
 
@@ -971,8 +976,6 @@ def fix_errata_on( suts = hosts )
         EOM
 
         install_cert_on_windows(sut, 'geotrustglobal', geotrust_global_ca)
-      else
-        linux_errata(sut)
       end
     end
 

Diff for: lib/simp/beaker_helpers/version.rb

@@ -3,5 +3,5 @@
 module Simp; end
 
 module Simp::BeakerHelpers
-  VERSION = '1.34.2'
+  VERSION = '1.34.3'
 end