Re: [PATCH] allow kickstart to preexisting LV w/ encrypted PV (448129)

[Hans de Goede <hdegoede redhat com>]

Dave Lehman wrote:
> The referenced bug is a failure to kickstart when the user specifies
> that we install to a pre-existing logical volume which contains one or
> more encrypted physical volumes.
>
> Joel's recent patch to Partitions.setFromDisk gets us part of the way
> there by prompting for a passphrase so we can look at the pv.
>
> The second problem comes when lvm, in all it's brilliance, returns
> '/dev/dm-0' from pvdisplay instead of '/dev/mapper/foo'. I have a little
> patch to resolve /dev/dm-X to a useful (ie: persistent) device mapper
> name.
>
> Comments, suggestions?

Looks ok to me.

> I'm also curious whether people think we should stuff this into F10. I
> think we should, assuming we will be building for it again anyhow.

I vote for putting it in F-10.

Regards,

Hans