Re: [PATCH] LVM VG size is not same as the device where PV is. (#480793)

[Hans de Goede <hdegoede redhat com>]

Joel Granados Moreno wrote:
> We need to account for PV metadata in the LV grow functions.  This patch
> avoids the missing one PE error message.
> ---
>  autopart.py |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)

Looks good,

Regards,

Hans

> diff --git a/autopart.py b/autopart.py
> index 2c6f5e8..2938d9b 100644
> --- a/autopart.py
> +++ b/autopart.py
> @@ -534,7 +534,7 @@ def growLogicalVolumes(diskset, requests):
>  		    log("considering %s, start size = %s",req.logicalVolumeName, req.getStartSize())
>  		    
>  		# get remaining free space
> -		vgfree = lvm.getVGFreeSpace(vgreq, requests, diskset)
> +		vgfree = lvm.getVGFreeSpace(vgreq, requests, diskset) - (2*vgreq.pesize)
>  
>  #		print "vgfree = ", vgfree
>                  if DEBUG_LVM_GROW: