On Wed, 2009-03-11 at 14:34 +0100, Hans de Goede wrote:
>
> David Lehman wrote:
> > Looks good.
>
> Thanks, pushed.
>
> > Any chance you could throw something together to cover the
> > case where LVM2_VG_NAME is unset after the first lvm rule runs?
> >
>
> How about this (tested and works for me) ? :
Perfect.
>
> Subject: [PATCH] If a pv somehow does not contain a vg_name, do not try to get other vg info
>
> In the anacond udev rules do not execute the 2 import commands which depend
> up on the vg_name if we don't have a vg_name.
> ---
> 70-anaconda.rules | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/70-anaconda.rules b/70-anaconda.rules
> index 42ac1d4..e379a22 100644
> --- a/70-anaconda.rules
> +++ b/70-anaconda.rules
> @@ -22,6 +22,7 @@ ENV{ID_FS_TYPE}=="linux_raid_member", IMPORT{program}="/usr/sbin/mdadm --examine
>
> # probe metadata of LVM2 physical volumes
> ENV{ID_FS_TYPE}=="LVM2_member", IMPORT{program}="/usr/sbin/lvm pvs --units k --nosuffix --nameprefixes --rows --unquoted --noheadings -opv_name,pv_uuid,pv_size,vg_name,vg_uuid,pv_pe_count,pv_pe_alloc_count,pe_start $tempnode"
> +ENV{LVM2_VG_NAME}!="?*", GOTO="anaconda_end"
> ENV{ID_FS_TYPE}=="LVM2_member", IMPORT{program}="/usr/sbin/lvm vgs --units k --nosuffix --nameprefixes --rows --unquoted --noheadings -ouuid,size,free,extent_size,extent_count,free_count,pv_count $env{LVM2_VG_NAME}"
> ENV{ID_FS_TYPE}=="LVM2_member", IMPORT{program}="/usr/sbin/lvm lvs --units k --nosuffix --nameprefixes --rows --unquoted --noheadings -olv_name,lv_uuid,lv_size $env{LVM2_VG_NAME}"
>
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
