[blivet] Change the default /boot part on s390x to not be lvm. (#1035201)

Samantha N. Bueno sbueno+anaconda at redhat.com
Mon Sep 29 18:50:51 UTC 2014


On Mon, Sep 29, 2014 at 01:33:37PM -0500, David Lehman wrote:
> On 09/29/2014 12:35 PM, Samantha N. Bueno wrote:
> >On Mon, Sep 29, 2014 at 10:03:13AM -0500, David Lehman wrote:
> >>I think it's safe to remove all of the singlePV code from blivet since it
> >>was only ever there for the case of /boot on s390.
> >
> >Actually, I thought of one good reason to leave the singlePV code in--if
> >a user does not specifically create /boot, then / needs to be on a
> >singlePV. And while these changes account for /boot, there is no
> >accounting for /. So unless I am missing something, this should probably
> >stay in.
> 
> Is /boot on lvm supported or not? If it's not supported we should be
> removing "lvmlv" from pyanaconda.bootloader.ZIPL.stage2_device_types. If it
> is still supported, aren't you going to see the same problems when there's
> no /boot as you're seeing now with /boot on an LV?

I'm going to say no. "lvmlv" is removed from
pyanaconda.bootloader.ZIPL.stage2_device_types in the anaconda portion
of these patches. But yeah you do have a very good point. So I'll strip
all the singlePV references then. I'll post a follow-up patch in just a
bit.

Samantha


More information about the anaconda-patches mailing list