Re: [PATCH rhel4] Fix clamping of VG size in case when it is not sufficient for lvm metadata (#489549)

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



looks ok.
On Wed, Mar 11, 2009 at 01:24:45PM +0100, Radek Vykydal wrote:
> This fixes patch 6c6a206bd8dfc6910fc4061274bd4f054400c838 which
> is essentialy kickstart part of a backport from rhel 5.3.
> The original patch didn't take in account that PE size is
> in KB, not MB. Also 1 PE per PV should be enough for lvm metadata.
> 
> I don't port gui part from 5.3 because the bug is for ks and
> we are so close to release.
> ---
>  partRequests.py |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/partRequests.py b/partRequests.py
> index f907525..fc89540 100644
> --- a/partRequests.py
> +++ b/partRequests.py
> @@ -763,7 +763,7 @@ class VolumeGroupRequestSpec(RequestSpec):
>                  #log("size for pv %s is %s" % (pvid, size))
>                  clamped = lvm.clampPVSize(size, self.pesize)
>                  if clamped == long(size):
> -                    clamped = clamped - (2*self.pesize)
> +                    clamped = clamped - (self.pesize / 1024)
>                  log("  got pv.size of %s, clamped to %s" % (size,clamped))
>                  #log("  clamped size is %s" % (size,))
>                  totalspace = totalspace + clamped
> -- 
> 1.5.4.3
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

-- 
Joel Andres Granados
Brno, Czech Republic, Red Hat.

_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list

[Index of Archives]     [Kickstart]     [Fedora Users]     [Fedora Legacy List]     [Fedora Maintainers]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [Yosemite Photos]     [KDE Users]     [Fedora Tools]
  Powered by Linux