[PATCH 2/3] Apply changes in disk selection even if not reclaiming space

David Lehman dlehman at redhat.com
Mon Sep 15 18:05:59 UTC 2014


On 09/11/2014 09:05 AM, Vratislav Podzimek wrote:
> If we don't run any dialog and user didn't requested reclaiming space we should
> still apply disk selection in the Storage spoke.
>
> Related: rhbz#1085201
> Signed-off-by: Vratislav Podzimek <vpodzime at redhat.com>
> ---
>   pyanaconda/ui/gui/spokes/storage.py | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/pyanaconda/ui/gui/spokes/storage.py b/pyanaconda/ui/gui/spokes/storage.py
> index 220bb14..2a48fca 100644
> --- a/pyanaconda/ui/gui/spokes/storage.py
> +++ b/pyanaconda/ui/gui/spokes/storage.py
> @@ -823,9 +823,10 @@ class StorageSpoke(NormalSpoke, StorageChecker):
>                   if not self._check_encrypted():
>                       return
>
> +                self.apply()
> +
>                   # Oh and then we might also want to go to the reclaim dialog.
>                   if self._reclaim.get_active():
> -                    self.apply()
>                       if not self._show_resize_dialog(disks):
>                           # User pressed cancel on the reclaim dialog, so don't leave
>                           # the storage spoke.
>

This is likely to expose a pile of bugs related to how we handle custom 
storage and kickstart, some of which I started fixing last week. Since 
this set overlaps with what I'm working on I'll see how these fit with 
what I'm doing by applying them and seeing how much of my progress it 
undoes. I'll report back once I have more information. I like patches 2 
and 3, but I'm unsure about patch 1 as far as handling multiple trips 
into the storage spoke when autopart is selected.

David


More information about the anaconda-patches mailing list