https://bugzilla.suse.com/show_bug.cgi?id=1203019
Bug ID: 1203019 Summary: yast2: replace mkinitrd wrapper with native dracut Classification: openSUSE Product: openSUSE Tumbleweed Version: Current Hardware: Other OS: Other Status: NEW Severity: Normal Priority: P5 - None Component: YaST2 Assignee: yast2-maintainers@suse.de Reporter: antonio.feijoo@suse.com QA Contact: jsrain@suse.com Blocks: 1202351 Found By: --- Blocker: ---
+++ This bug was initially created as a clone of Bug #1202351 +++
Upstream support removed in March 2021 (https://github.com/dracutdevs/dracut/commit/43df4ee2) and deprecation announcement on factory mailing list in May 2021 (https://lists.opensuse.org/archives/list/factory@lists.opensuse.org/thread/G...).
ALP will not ship the mkinitrd wrapper, so it's better to get rid of it in Tumbleweed first.
References:
- yast2
/usr/share/YaST2/modules/Initrd.rb:72: # parametr for mkinitrd because of splash screen /usr/share/YaST2/modules/Initrd.rb:75: # Additional parameters for mkinitrd /usr/share/YaST2/modules/Initrd.rb:391: "/lib/mkinitrd/scripts/setup-splash.sh" /usr/share/YaST2/modules/Initrd.rb:400: "/sbin/mkinitrd %1 %2 >> %3 2>&1", /usr/share/YaST2/modules/Initrd.rb:405: File.join(Directory.logdir, "y2logmkinitrd").shellescape /usr/share/YaST2/modules/Initrd.rb:411: Ops.add(Directory.logdir, "/y2logmkinitrd") /usr/share/YaST2/modules/Initrd.rb:434: # Set the -s parameter of mkinitrd /usr/share/YaST2/modules/Initrd.rb:463: # Get additional parameters for mkinitrd /usr/share/YaST2/modules/Initrd.rb:464: # @return [String] additional mkinitrd parameters /usr/share/YaST2/modules/Initrd.rb:469: # Set additional parameters for mkinitrd /usr/share/YaST2/modules/Initrd.rb:470: # @param [String] params string additional mkinitrd parameters
# rpm -qf /usr/share/YaST2/modules/Initrd.rb yast2-4.5.10-1.1.x86_64
- yast2-country
/usr/share/YaST2/modules/Timezone.rb:85: # if mkinitrd should be called at the end /usr/share/YaST2/modules/Timezone.rb:86: @call_mkinitrd = false /usr/share/YaST2/modules/Timezone.rb:467: Builtins.y2milestone("calling mkinitrd...") /usr/share/YaST2/modules/Timezone.rb:470: "/sbin/mkinitrd >> /var/log/YaST2/y2logmkinitrd 2>> /var/log/YaST2/y2logmkinitrd" /usr/share/YaST2/modules/Timezone.rb:860: CallMkinitrd() if @call_mkinitrd && !Stage.initial /usr/share/YaST2/modules/Timezone.rb:1081: publish :variable => :call_mkinitrd, :type => "boolean"
# rpm -qf /usr/share/YaST2/modules/Timezone.rb yast2-country-4.5.1-1.1.x86_64
- yast2-kdump
/usr/share/YaST2/modules/Kdump.rb:419: update_command = (using_fadump? ? "/usr/sbin/mkdumprd -f" : "/sbin/mkinitrd") /usr/share/YaST2/modules/Kdump.rb:426: update_logfile = File.join(Directory.logdir, "y2logmkinitrd")
# rpm -qf /usr/share/YaST2/modules/Kdump.rb yast2-kdump-4.5.3-1.1.x86_64
https://bugzilla.suse.com/show_bug.cgi?id=1203019
Michal Filka mfilka@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|NEW |CONFIRMED URL| |https://trello.com/c/6Nouwi | |Gv CC| |mfilka@suse.com Assignee|yast2-maintainers@suse.de |yast-internal@suse.de
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c1
--- Comment #1 from Antonio Feijoo antonio.feijoo@suse.com --- ping
ALP is moving forward and we need to get rid of mkinitrd. What's the status of this task?
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c2
Josef Reidinger jreidinger@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |jreidinger@suse.com
--- Comment #2 from Josef Reidinger jreidinger@suse.com --- yast2-kdump 4.5.7 should be already without calling mkinitrd
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c3
--- Comment #3 from Antonio Feijoo antonio.feijoo@suse.com --- (In reply to Josef Reidinger from comment #2)
yast2-kdump 4.5.7 should be already without calling mkinitrd
Thanks.
So the only missing pieces blocking the mkinitrd removal are yast2 and yast2-country. All the other identified packages (kernel, kdump, mdadm) are fixed.
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c4
--- Comment #4 from Lukas Ocilka locilka@suse.com --- (In reply to Antonio Feijoo from comment #3)
All the other identified packages (kernel, kdump, mdadm) are fixed.
I rather say "adapted to the new state" ;) It's not a bug per se.
We'll work on this after the ALP March Prototype is out, because that's our current priority.
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c5
Lukas Ocilka locilka@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |rtsvetkov@suse.com Flags| |needinfo?(rtsvetkov@suse.co | |m)
--- Comment #5 from Lukas Ocilka locilka@suse.com --- Rado, do you want this also for SP5?
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c6
Josef Reidinger jreidinger@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|CONFIRMED |IN_PROGRESS
--- Comment #6 from Josef Reidinger jreidinger@suse.com --- fix is under review https://github.com/yast/yast-country/pull/306 and https://github.com/yast/yast-yast2/pull/1289
Question about SP5 is still valid as I do change only for TW
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c7
Lukas Ocilka locilka@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- CC| |antonio.feijoo@suse.com Flags| |needinfo?(antonio.feijoo@su | |se.com)
--- Comment #7 from Lukas Ocilka locilka@suse.com --- Considering bug #1202351 I personally feel that TW-only would be OK?
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c8
Antonio Feijoo antonio.feijoo@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(antonio.feijoo@su | |se.com) |
--- Comment #8 from Antonio Feijoo antonio.feijoo@suse.com --- (In reply to Lukas Ocilka from comment #7)
Considering bug #1202351 I personally feel that TW-only would be OK?
Yes, we are not touching SLE.
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c9
Lukas Ocilka locilka@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Flags|needinfo?(rtsvetkov@suse.co | |m) |
--- Comment #9 from Lukas Ocilka locilka@suse.com --- Josef, TW-only (+ALP) please
https://bugzilla.suse.com/show_bug.cgi?id=1203019 https://bugzilla.suse.com/show_bug.cgi?id=1203019#c10
Josef Reidinger jreidinger@suse.com changed:
What |Removed |Added ---------------------------------------------------------------------------- Status|IN_PROGRESS |RESOLVED Resolution|--- |FIXED Flags| |needinfo?
--- Comment #10 from Josef Reidinger jreidinger@suse.com --- prs merged. So lets close it.
-
bugzilla_noreply＠suse.com