Re: Cannot add a partition after devicetree.processActions()

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

 



On Mon, 2012-03-26 at 17:30 +0200, Jan Safranek wrote:
> Consider following code snippet ('disk' is /dev/sda without any partitions):
>     part1 = storage.newPartition(disks=[disk], size=10)
>     storage.createDevice(part1)
>     pyanaconda.storage.partitioning.doPartitioning(storage=storage)
>     storage.devicetree.processActions(dryRun=False)
> 
>     part2 = storage.newPartition(disks=[disk])
>     storage.createDevice(part2)
>     pyanaconda.storage.partitioning.doPartitioning(storage=storage)
>     storage.devicetree.processActions(dryRun=False)
> 
> The second processActions() fails with DeviceError('device has already
> been created', 'sda1'). sda1 has been indeed create during the first
> processActions(). Why the devicetree tries to create it again? How can I
> clean it up?
> 
> Is the above code valid or really only one processActions() is allowed?

It is not designed specifically to allow multiple calls to
processActions. However, it should require little more than a single
line of code to clear the action list before creating the second set of
devices. For now we can't clear the action list automatically because it
is used later to aid in writing anaconda-ks.cfg. This will probably
change in F18.

Try adding the following right after each processActions call:

storage.devicetree._actions = []

We can add a cleaner way to accomplish this once we see that this
approach works.

Dave

> 
> Jan
> 
> _______________________________________________
> 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