[blivet] small md cleanups (v2)

Anne Mulhern amulhern at redhat.com
Thu Aug 7 15:23:30 UTC 2014





----- Original Message -----
> From: "David Lehman" <dlehman at redhat.com>
> To: anaconda-patches at lists.fedorahosted.org
> Sent: Monday, August 4, 2014 5:14:56 PM
> Subject: [blivet] small md cleanups (v2)
> 
> New patch 1 teaches udev.device_get_md_device_uuid to ask udev first
> instead of relying solely on having run mdexamine.
> 
> New patch 2 takes advantage of md member and array UUIDs being in udev
> to delay running mdexamine until handleUdevMDMemberFormat. This also
> allows us to drop the md caching patch from v2 of the pyudev set.
> 
> Patch 3 is unchanged.
> 
> Patch 4 is generalized so the name is updated regardless of how we found
> the device and also to bypass the name setter, which disallows setting a
> name for existing devices.
> 
> Patch 5 has updated comment text.
> 
> New patch 6 removes obsolete code to avoid returning an md array UUID as
> the UUID of an md member.
> 
> 
> At some point it might be worthwhile to change things so that the udev
> functions handle the ID_FS_UUID_SUB situation transparently, but that will
> have to be a separate effort.
> 
> David Lehman (6):
>   Look in udev data for md member UUID.
>   Get md member and array UUIDs for format ctor from udev.
>   Remove an obsolete block related to unpredictable md device names.
>   Update md name when lookup relied on UUID.
>   Don't pass md array UUID as member format UUID.
>   Simplify udev.device_get_uuid.
> 
>  blivet/devicetree.py | 50 ++++++++++----------------------------------------
>  blivet/udev.py       | 11 ++++-------
>  2 files changed, 14 insertions(+), 47 deletions(-)
> 
> --
> 1.9.3
> 
> _______________________________________________
> anaconda-patches mailing list
> anaconda-patches at lists.fedorahosted.org
> https://lists.fedorahosted.org/mailman/listinfo/anaconda-patches
> 

Aside from the comment on patch 4, which looks like it might have been
a previous version sent by mistake, this looks to me like it ought to work
correctly.

- mulhern

 


More information about the anaconda-patches mailing list