Fixes #36691 - use 'connectefi scsi' by default

Co-Authored-by: Shimon Shtein <[email protected]>
Co-Authored-by: Ewoud Kohl van Wijngaarden <[email protected]>

Author: 3 people

app/views/unattended/provisioning_templates/snippet/pxegrub2_chainload.erb

@@ -32,9 +32,13 @@ insmod fat
 insmod chain
 
 echo "VMWare hosts with QuickBoot feature enabled may not find the local ESP"
-echo "partition due to not initializing all the EFI devices. To workaround, upgrade"
-echo "to the latest grub2 (*) and uncomment "connectefi scsi" statement in the"
-echo "grub2_chainload template."
+echo "partition due to not initializing all the EFI devices. To address this,"
+echo "use an up-to-date grub2 (*) version and include the "connectefi scsi" statement"
+echo "as provided below. If you're using an older grub2 version or"
+echo "the "connectefi" option isn't recognized by your grub2, grub2 will print a error"
+echo "like 'can't find command connectefi' but the boot process will continue."
+echo "You can omit this by adding a (global) parameter to 'grub2-connectefi=false'."
+echo "Valid values of 'grub2-connectefi' parameter: false, scsi, pciroot"
 echo
 echo "Virtual or physical hosts using Software RAID for the ESP partition may try"
 echo "booting on the Software RAID, which will fail. To workaround, upgrade to the"
@@ -43,7 +47,10 @@ echo "the grub2_chainload template."
 echo
 echo "(*) grub2-efi-x64-2.02-122.el8 (upstream doesn't have the patches yet)"
 echo
-#connectefi scsi
+<%=
+  connectefi_option = host_param('grub2-connectefi', 'scsi')
+  "connectefi #{connectefi_option}" if connectefi_option.present? && connectefi_option != 'false'
+-%>
 
 menuentry 'Chainload Grub2 EFI from ESP' --id local_chain_hd0 {
   echo "Chainloading Grub2 EFI from ESP, enabled devices for booting:"

test/unit/foreman/renderer/snapshots/ProvisioningTemplate/PXEGrub2/PXEGrub2_default_local_boot.host4dhcp.snap.txt

@@ -7,9 +7,13 @@ insmod fat
 insmod chain
 
 echo "VMWare hosts with QuickBoot feature enabled may not find the local ESP"
-echo "partition due to not initializing all the EFI devices. To workaround, upgrade"
-echo "to the latest grub2 (*) and uncomment "connectefi scsi" statement in the"
-echo "grub2_chainload template."
+echo "partition due to not initializing all the EFI devices. To address this,"
+echo "use an up-to-date grub2 (*) version and include the "connectefi scsi" statement"
+echo "as provided below. If you're using an older grub2 version or"
+echo "the "connectefi" option isn't recognized by your grub2, grub2 will print a error"
+echo "like 'can't find command connectefi' but the boot process will continue."
+echo "You can omit this by adding a (global) parameter to 'grub2-connectefi=false'."
+echo "Valid values of 'grub2-connectefi' parameter: false, scsi, pciroot"
 echo
 echo "Virtual or physical hosts using Software RAID for the ESP partition may try"
 echo "booting on the Software RAID, which will fail. To workaround, upgrade to the"
@@ -18,8 +22,7 @@ echo "the grub2_chainload template."
 echo
 echo "(*) grub2-efi-x64-2.02-122.el8 (upstream doesn't have the patches yet)"
 echo
-#connectefi scsi
-
+connectefi scsi
 menuentry 'Chainload Grub2 EFI from ESP' --id local_chain_hd0 {
   echo "Chainloading Grub2 EFI from ESP, enabled devices for booting:"
   ls

test/unit/foreman/renderer/snapshots/ProvisioningTemplate/PXEGrub2/PXEGrub2_global_default.host4dhcp.snap.txt

@@ -23,9 +23,13 @@ insmod fat
 insmod chain
 
 echo "VMWare hosts with QuickBoot feature enabled may not find the local ESP"
-echo "partition due to not initializing all the EFI devices. To workaround, upgrade"
-echo "to the latest grub2 (*) and uncomment "connectefi scsi" statement in the"
-echo "grub2_chainload template."
+echo "partition due to not initializing all the EFI devices. To address this,"
+echo "use an up-to-date grub2 (*) version and include the "connectefi scsi" statement"
+echo "as provided below. If you're using an older grub2 version or"
+echo "the "connectefi" option isn't recognized by your grub2, grub2 will print a error"
+echo "like 'can't find command connectefi' but the boot process will continue."
+echo "You can omit this by adding a (global) parameter to 'grub2-connectefi=false'."
+echo "Valid values of 'grub2-connectefi' parameter: false, scsi, pciroot"
 echo
 echo "Virtual or physical hosts using Software RAID for the ESP partition may try"
 echo "booting on the Software RAID, which will fail. To workaround, upgrade to the"
@@ -34,8 +38,7 @@ echo "the grub2_chainload template."
 echo
 echo "(*) grub2-efi-x64-2.02-122.el8 (upstream doesn't have the patches yet)"
 echo
-#connectefi scsi
-
+connectefi scsi
 menuentry 'Chainload Grub2 EFI from ESP' --id local_chain_hd0 {
   echo "Chainloading Grub2 EFI from ESP, enabled devices for booting:"
   ls

test/unit/foreman/renderer/snapshots/ProvisioningTemplate/snippet/pxegrub2_chainload.host4dhcp.snap.txt

@@ -3,9 +3,13 @@ insmod fat
 insmod chain
 
 echo "VMWare hosts with QuickBoot feature enabled may not find the local ESP"
-echo "partition due to not initializing all the EFI devices. To workaround, upgrade"
-echo "to the latest grub2 (*) and uncomment "connectefi scsi" statement in the"
-echo "grub2_chainload template."
+echo "partition due to not initializing all the EFI devices. To address this,"
+echo "use an up-to-date grub2 (*) version and include the "connectefi scsi" statement"
+echo "as provided below. If you're using an older grub2 version or"
+echo "the "connectefi" option isn't recognized by your grub2, grub2 will print a error"
+echo "like 'can't find command connectefi' but the boot process will continue."
+echo "You can omit this by adding a (global) parameter to 'grub2-connectefi=false'."
+echo "Valid values of 'grub2-connectefi' parameter: false, scsi, pciroot"
 echo
 echo "Virtual or physical hosts using Software RAID for the ESP partition may try"
 echo "booting on the Software RAID, which will fail. To workaround, upgrade to the"
@@ -14,8 +18,7 @@ echo "the grub2_chainload template."
 echo
 echo "(*) grub2-efi-x64-2.02-122.el8 (upstream doesn't have the patches yet)"
 echo
-#connectefi scsi
-
+connectefi scsi
 menuentry 'Chainload Grub2 EFI from ESP' --id local_chain_hd0 {
   echo "Chainloading Grub2 EFI from ESP, enabled devices for booting:"
   ls

test/unit/foreman/templates/snippets/pxegrub2_chainload_test.rb

@@ -0,0 +1,52 @@
+require 'test_helper'
+
+class PxeGrub2ChainloadTest < ActiveSupport::TestCase
+  def renderer
+    @renderer ||= Foreman::Renderer::SafeModeRenderer
+  end
+
+  def render_template(host)
+    @snippet ||= File.read(Rails.root.join('app', 'views', 'unattended', 'provisioning_templates', 'snippet', 'pxegrub2_chainload.erb'))
+
+    source = OpenStruct.new(
+      name: 'Test',
+      content: @snippet
+    )
+
+    scope = Class.new(Foreman::Renderer::Scope::Provisioning).send(
+      :new,
+      host: host,
+      source: source,
+      variables: {
+        host: host,
+      })
+
+    renderer.render(source, scope)
+  end
+
+  setup do
+    @host = FactoryBot.create(:host, :managed, :build => true)
+  end
+
+  test 'should render connectefi scsi option by default' do
+    actual = render_template(@host)
+
+    assert_match(/^ *connectefi scsi/, actual)
+  end
+
+  test 'should not render connectefi option if parameter false' do
+    FactoryBot.create(:host_parameter, host: @host, name: 'grub2-connectefi', value: 'false')
+
+    actual = render_template(@host)
+
+    assert_no_match(/^ *connectefi/, actual)
+  end
+
+  test 'should render connectefi option if parameter present' do
+    FactoryBot.create(:host_parameter, host: @host, name: 'grub2-connectefi', value: 'TESTOPT')
+
+    actual = render_template(@host)
+
+    assert_match(/^ *connectefi TESTOPT/, actual)
+  end
+end