[blivet][master/rhel7-branch] Make sure bootDevice is not LVM LV if we're on s390x (#873135)

Anne Mulhern amulhern at redhat.com
Tue Dec 9 15:31:03 UTC 2014





----- Original Message -----
> From: "Samantha N. Bueno" <sbueno+anaconda at redhat.com>
> To: "anaconda patch review" <anaconda-patches at lists.fedorahosted.org>
> Sent: Tuesday, December 9, 2014 9:44:35 AM
> Subject: Re: [blivet][master/rhel7-branch] Make sure bootDevice is not LVM LV	if we're on s390x (#873135)
> 
> On Tue, Dec 09, 2014 at 08:13:14AM -0500, Anne Mulhern wrote:
> > I think that this code change should probably be in
> > pyanaconda.kickstart.Bootloader.execute(), not in Blivet.setUpBootloader().
> > 
> > Also, since it's not just a kickstart but also a GUI bug, it might be good
> > to
> > also edit custom.CustomPartitioningSpoke._validateMountpoint()...or
> > wherever
> > that code is in RHEL7.
> 
> Err, wait, how is this also a GUI bug? The problem is that sanityCheck
> is not ever called if a user explicitly defines partition info in
> kickstart, so the stage2 device check is never performed. If you try and
> put /boot on LVM performing a manual installation, it is not allowed
> (which is correct behavior).
> 

It was reported as a GUI bug (in 2012). Guess it's come a long
way since then.

<-- SNIP -->

- mulhern


More information about the anaconda-patches mailing list