Bug 1223344

Summary: fedup plymouth theme is not applied with RC1 upgrade.img
Product: [Fedora] Fedora Reporter: Kamil Páral <kparal>
Component: plymouthAssignee: Will Woods <wwoods>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 22CC: dennis, elad, fedora, kevin, mruckman, notting, rstrode, satellitgo, tflink, wwoods
Target Milestone: ---   
Target Release: ---   
Hardware: Unspecified   
OS: Unspecified   
Whiteboard: AcceptedFreezeException
Fixed In Version: plymouth-0.8.9-9.2013.08.14.fc22 Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2015-05-22 19:53:51 UTC Type: Bug
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:
Embargoed:
Bug Depends On:    
Bug Blocks: 1043132    
Attachments:
Description Flags
fedup with theme
none
fedup without theme
none
upgrade.log from RC1 upgrade
none
0001-plymouth-populate-initrd-fix-THEME_OVERRIDE-with-emp.patch none

Description Kamil Páral 2015-05-20 11:30:52 UTC
Description of problem:
When upgrading using fedup and using the RC1 upgrade.img, there's no plymouth splash displayed, but just console output.

But when I've built my own upgrade.img and tried to use it, I see nice fedup splash showing progress.

There has to be a problem somewhere either in fedup-dracut or in the upgrade.img build process. The build process seems plausible, because when I build the upgrade.img locally and don't have fedup-dracut-plymouth package installed, I see this:

modifying plymouthd.conf: Theme=fedup
grep: /usr/share/plymouth/themes/fedup/fedup.plymouth: No such file or directory
The default plymouth plugin () doesn't exist

So maybe fedup-dracut-plymouth is not installed on the machine (in the chroot) where upgrade.img is being built?

Version-Release number of selected component (if applicable):
fedup-dracut-0.9.2-1.fc22

How reproducible:
always

Steps to Reproduce:
1. run `fedup --network 22 --instrepo http://dl.fedoraproject.org/pub/alt/stage/22_RC1/Workstation/x86_64/os/` , see no splash
2. build your own upgrade.img on F22 using `/usr/share/doc/fedup-dracut/makefeduprepo myrepo`
3. share myrepo over http
4. run `fedup --network 22 --instrepo http://localhost/myrepo`, see splash

Comment 1 Kamil Páral 2015-05-20 11:31:33 UTC
Created attachment 1027643 [details]
fedup with theme

This is how upgrade process looks like with a fedup theme.

Comment 2 Kamil Páral 2015-05-20 11:31:58 UTC
Created attachment 1027644 [details]
fedup without theme

This is how upgrade process looks like without fedup theme.

Comment 3 Kamil Páral 2015-05-20 11:35:55 UTC
I think we should consider this as a final blocker, because we're not able to fix this with an update. After the upgrade.img is built, it's set in stone. Unless rel-eng is OK with building a fixed version separate from the RC, and then overwrite it/host it elsewhere and update Mirror Manager (which I don't consider likely).

Comment 4 Kamil Páral 2015-05-20 11:39:13 UTC
Changing to distribution, we likely need to see fedup-dracut build logs from RC1. Dennis, or someone else with access, can you take a look?

Comment 5 Kamil Páral 2015-05-20 11:41:39 UTC
Created attachment 1027646 [details]
upgrade.log from RC1 upgrade

This is the upgrade.log from RC1 upgrade which doesn't show the splash. In case it helps debugging why it wasn't shown.

Comment 6 Kamil Páral 2015-05-20 11:49:50 UTC
FWIW, I tried using this branched daily compose, which is the last one having images/ directory correctly created:
http://koji.fedoraproject.org/mash/branched-20150517/22/x86_64/os/
And the fedup splash is not shown either, even though the build log seems to indicate that fedup-dracut-plymouth was correctly installed:
http://koji.fedoraproject.org/mash/branched-20150517/logs/x86_64/x86_64.log

But I don't see makefeduprepo output there, so I can't really say if that's the right place where the upgrade.img process happens or not.

Comment 7 Kamil Páral 2015-05-20 11:55:16 UTC
Please note that if we don't have this splash, but use console, we're affected by bug 1173135, which might confuse some users and potentially cause data loss - they might get the idea that the upgrade process got broken/corrupted and reset the system in the middle of the operation.

Comment 8 Kamil Páral 2015-05-20 12:01:43 UTC
My custom upgrade.img is here, if you want to try out the fedup splash easily, or compare the changes:
https://kparal.fedorapeople.org/bugs/rhbz_1223344/repo/

Comment 9 Stephen Gallagher 2015-05-20 12:05:06 UTC
This sounds to me more like a Freeze Exception request than a Blocker bug, to me. It's not any worse than it was in Fedora 21 and the upgrade criteria only states: "For each one of the release-blocking package sets, it must be possible to successfully complete an upgrade from a fully updated installation of the previous stable Fedora release with that package set installed."

I'm +1 FE for this if we have a solution in time for the RC2 request today. I'm definitely -1 blocker. There's no reason to delay specifically for this.

Comment 10 Kamil Páral 2015-05-20 12:14:12 UTC
Adding FE proposal.

Comment 11 Kamil Páral 2015-05-20 12:44:46 UTC
After thinking more about this, I'm retracting the blocker proposal. It is not serious enough to block the release on, it just doesn't paint us a nice picture.

However, since we need RC2 anyway, I'd be really glad if this got resolved in it, even if we had to delay the RC2 compose for a few hours. It'd be worth it.

Comment 12 Kamil Páral 2015-05-20 13:56:38 UTC
RC1 compose logs are here:
https://ausil.fedorapeople.org/22_RC1/

fedup-dracut-plymouth seems to be installed.

Comment 13 Will Woods 2015-05-20 16:18:09 UTC
plymouth-populate-initrd is supposed to modify etc/plymouth/plymouthd.conf to set Theme=fedup, but.. it isn't. You can check that like so:

  lsinitrd upgrade.img etc/plymouth/plymouthd.conf

and you'll get:

  # Administrator customizations go in this file
  #[Daemon]
  #Theme=fade-in

I'm going to attach a patch against plymouth git master that should fix the problem.

Comment 14 Will Woods 2015-05-20 16:20:00 UTC
Created attachment 1027789 [details]
0001-plymouth-populate-initrd-fix-THEME_OVERRIDE-with-emp.patch

Patch against plymouth master; should fix plymouth-populate-initrd so it correctly sets Theme=fedup.

Comment 15 Will Woods 2015-05-20 16:35:19 UTC
I've tested the patch locally. Before patch:

  [wwoods@metroid ~]$ sudo PLYMOUTH_THEME_NAME=fedup dracut -f initrd.img &&
                      lsinitrd initrd.img etc/plymouth/plymouthd.conf
  modifying plymouthd.conf: Theme=fedup
  # Administrator customizations go in this file
  #[Daemon]
  #Theme=fade-in

After patch:

  [wwoods@metroid ~]$ sudo PLYMOUTH_THEME_NAME=fedup dracut -f initrd.img && 
                      lsinitrd initrd.img etc/plymouth/plymouthd.conf
  modifying plymouthd.conf: Theme=fedup
  # Administrator customizations go in this file
  #[Daemon]
  #Theme=fade-in
  [Daemon]
  # theme modified by plymouth-populate-initrd
  Theme=fedup

halfline has given me the green light to apply and build, so.. imma do that.

Comment 16 Kevin Fenzi 2015-05-20 16:37:05 UTC
+1 FE here.

Comment 17 Mike Ruckman 2015-05-20 16:43:04 UTC
+1 here as well.

Comment 18 Fedora Update System 2015-05-20 17:10:57 UTC
plymouth-0.8.9-9.2013.08.14.fc22 has been submitted as an update for Fedora 22.
https://admin.fedoraproject.org/updates/plymouth-0.8.9-9.2013.08.14.fc22

Comment 19 Fedora Update System 2015-05-21 00:16:46 UTC
Package plymouth-0.8.9-9.2013.08.14.fc22:
* should fix your issue,
* was pushed to the Fedora 22 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing plymouth-0.8.9-9.2013.08.14.fc22'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2015-8625/plymouth-0.8.9-9.2013.08.14.fc22
then log in and leave karma (feedback).

Comment 20 Kamil Páral 2015-05-21 07:38:37 UTC
I see a graphical splash when upgrading using RC2 upgrade.img. Yay!

Comment 21 Fedora Update System 2015-05-22 19:53:51 UTC
plymouth-0.8.9-9.2013.08.14.fc22 has been pushed to the Fedora 22 stable repository.  If problems still persist, please make note of it in this bug report.