Re: [PATCH] Keep track of pvcount for non existing vgs (#593871)

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

 



ack

--

  Martin Gracik

----- "Hans de Goede" <hdegoede@xxxxxxxxxx> wrote:

> ---
>  storage/devices.py |    9 +++++++++
>  1 files changed, 9 insertions(+), 0 deletions(-)
> 
> diff --git a/storage/devices.py b/storage/devices.py
> index 5d26b26..be5a9b1 100644
> --- a/storage/devices.py
> +++ b/storage/devices.py
> @@ -1773,6 +1773,9 @@ class LVMVolumeGroupDevice(DMDevice):
>          if not self.peSize:
>              self.peSize = 32.0   # MB
>  
> +        if not self.exists:
> +            self.pvCount = len(self.parents)
> +
>          #self.probe()
>  
>      def __str__(self):
> @@ -2037,6 +2040,9 @@ class LVMVolumeGroupDevice(DMDevice):
>          self.parents.append(pv)
>          pv.addChild()
>  
> +        # and update our pv count
> +        self.pvCount = len(self.parents)
> +
>      def _removePV(self, pv):
>          """ Remove an PV from this VG. """
>          if not pv in self.pvs:
> @@ -2049,6 +2055,9 @@ class LVMVolumeGroupDevice(DMDevice):
>          self.parents.remove(pv)
>          pv.removeChild()
>  
> +        # and update our pv count
> +        self.pvCount = len(self.parents)
> +
>      # We can't rely on lvm to tell us about our size, free space, &c
>      # since we could have modifications queued, unless the VG and all
> of
>      # its PVs already exist.
> -- 
> 1.7.0.1
> 
> _______________________________________________
> Anaconda-devel-list mailing list
> Anaconda-devel-list@xxxxxxxxxx
> https://www.redhat.com/mailman/listinfo/anaconda-devel-list

_______________________________________________
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