Re: [PATCH] Setup the disks before partitioning as the nodes are needed.

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

 



On Tue, 2009-04-07 at 16:58 +0200, Joel Granados Moreno wrote:
> ---
>  storage/__init__.py     |    6 ++++++
>  storage/partitioning.py |    6 ++++++
>  2 files changed, 12 insertions(+), 0 deletions(-)
> 
> diff --git a/storage/__init__.py b/storage/__init__.py
> index 7a9f93b..527ef48 100644
> --- a/storage/__init__.py
> +++ b/storage/__init__.py
> @@ -163,6 +163,12 @@ def storageComplete(anaconda):
>                                  custom_buttons=[_("Go _back"),
>                                                  _("_Write changes to disk")],
>                                  default = 0)
> +
> +    # Make sure that all is down, even the disks that we setup after popluate.
> +    for disk in anaconda.id.storage.disks:
> +        if hasattr(disk, "teardown"):
> +            disk.teardown()
> +
>      if rc == 0:
>          return DISPATCH_BACK
>  
> diff --git a/storage/partitioning.py b/storage/partitioning.py
> index 5068450..e480842 100644
> --- a/storage/partitioning.py
> +++ b/storage/partitioning.py
> @@ -539,6 +539,12 @@ def doPartitioning(storage, exclusiveDisks=None):
>      """
>      anaconda = storage.anaconda
>      disks = storage.disks
> +
> +    # Activate them before we choose the exclusive ones only.
> +    for disk in disks:
> +        if hasattr(disk, "setup"):
> +            disk.setup()
> +

Why not wait and only set up the disks we are going to use?

>      if exclusiveDisks:
>          disks = [d for d in disks if d.name in exclusiveDisks]
>  

_______________________________________________
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