Attachment 865164 Details for Bug 1065522 – proposed patch fix handling of partitions across multiple processActions calls

[patch] proposed patch fix handling of partitions across multiple processActions calls

0010-Fix-partition-handling-across-multiple-processAction.patch (text/plain), 1.57 KB, created by David Lehman on 2014-02-19 17:07:44 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Lehman

Created: 2014-02-19 17:07:44 UTC

Size: 1.57 KB

patch

obsolete

>From 2e8635d98c4b76a530ce25bce28e01d38e4917b6 Mon Sep 17 00:00:00 2001
>From: David Lehman <dlehman@redhat.com>
>Date: Tue, 18 Feb 2014 14:48:21 -0600
>Subject: [PATCH 10/10] Fix partition handling across multiple processActions
> calls. (#1065522)
>
>We replace the parted.Disk instances in all DiskLabel instances every
>time processActions is run, so we need update the parted.Partition
>instances throughout the devicetree accordingly.
>
>Also, we have to make sure that each DiskLabel's _origPartedDisk is
>updated before we update the DiskDevice's originalFormat with a new
>copy of the DiskLabel.
>---
> blivet/devicetree.py | 7 +++++++
> 1 file changed, 7 insertions(+)
>
>diff --git a/blivet/devicetree.py b/blivet/devicetree.py
>index b21654b..04db067 100644
>--- a/blivet/devicetree.py
>+++ b/blivet/devicetree.py
>@@ -282,8 +282,15 @@ class DeviceTree(object):
>         # removal of partitions makes use of originalFormat, so it has to stay
>         # up to date in case of multiple passes through this method
>         for disk in (d for d in self.devices if d.partitioned):
>+            disk.format.updateOrigPartedDisk()
>             disk.originalFormat = copy.deepcopy(disk.format)
> 
>+        # now we have to update the parted partitions of all devices so they
>+        # match the parted disks we just updated
>+        for partition in self.getDevicesByInstance(PartitionDevice):
>+            pdisk = partition.disk.format.partedDisk
>+            partition.partedPartition = pdisk.getPartitionByPath(partition.path)
>+
>     def _addDevice(self, newdev):
>         """ Add a device to the tree.
> 
>-- 
>1.8.5.3
>

Actions: View | Diff

Attachments on bug 1065522: 864063 | 864065 | 864815 | 865164