Merge pull request #143 from yast/huha-fadump-master

Don't Add Empty fadump="" Kernel Parameter [master]
yast · Sep 18, 2024 · 5cdaa7c · 5cdaa7c
2 parents 45a3a74 + ab2e1e7
commit 5cdaa7c
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 1 deletion.
diff --git a/package/yast2-kdump.changes b/package/yast2-kdump.changes
@@ -1,3 +1,9 @@
+-------------------------------------------------------------------
+Tue Sep 17 14:24:44 UTC 2024 - Stefan Hundhammer <[email protected]>
+
+- Don't write empty  fadump=""  kernel parameter (bsc#1230359)
+- 5.0.2
+
 -------------------------------------------------------------------
 Wed Jun 19 15:36:03 UTC 2024 - Stefan Schubert <[email protected]>
 

diff --git a/package/yast2-kdump.spec b/package/yast2-kdump.spec
@@ -17,7 +17,7 @@
 
 
 Name:           yast2-kdump
-Version:        5.0.1
+Version:        5.0.2
 Release:        0
 Summary:        Configuration of kdump
 License:        GPL-2.0-only

diff --git a/src/modules/Kdump.rb b/src/modules/Kdump.rb
@@ -1230,6 +1230,7 @@ def write_fadump_boot_param
 
       # If fdump is selected and we want to enable kdump
       value = "on" if using_fadump? && @add_crashkernel_param
+      value ||= :missing
       Bootloader.modify_kernel_params(:common, :recovery, "fadump" => value)
       Bootloader.Write unless Yast::Stage.initial # do mass write in installation to speed up
     end