[rhel6-branch] [PATCH] Use udev_resolve_devspec to recognize the partition used in --onpart

[Vratislav Podzimek]

We allow partition specification using /dev/disk/by-id/... paths,
but we need to translate this to a device name (e.g. /dev/sda1),
because that's what storage.devicetree works with.

Resolves: rhbz#809640
---
 kickstart.py |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kickstart.py b/kickstart.py
index f867a24..167440a 100644
--- a/kickstart.py
+++ b/kickstart.py
@@ -1106,7 +1106,7 @@ class VolGroupData(commands.volgroup.F16_VolGroupData):
         for pv in self.physvols:
             # if pv is using --onpart, use original device
             pv = anaconda.id.ksdata.onPart.get(pv, pv)
-            dev = devicetree.getDeviceByName(pv)
+            dev = devicetree.getDeviceByName(udev_resolve_devspec(pv))
             if dev and dev.format.type == "luks":
                 try:
                     dev = devicetree.getChildren(dev)[0]

[Jesse Keating]

On 07/27/2012 04:16 AM, Vratislav Podzimek wrote:
> We allow partition specification using /dev/disk/by-id/... paths,
> but we need to translate this to a device name (e.g. /dev/sda1),
> because that's what storage.devicetree works with.
> Resolves: rhbz#809640
>
> kickstart.py |    2 +-
>   1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kickstart.py b/kickstart.py
> index f867a24..167440a 100644
> --- a/kickstart.py
> +++ b/kickstart.py
> @@ -1106,7 +1106,7 @@ class VolGroupData(commands.volgroup.F16_VolGroupData):
>           for pv in self.physvols:
>               # if pv is using --onpart, use original device
>               pv = anaconda.id.ksdata.onPart.get(pv, pv)
>        dev = devicetree.getDeviceByName(pv)
>        dev = devicetree.getDeviceByName(udev_resolve_devspec(pv))
>         if dev and dev.format.type == "luks":
>             try:
>                 dev = devicetree.getChildren(dev)[0]

ACK