Since the parted ebusy errors due to unnecessary udev event generation on the
partition nodes bug has been nailed, we no longer need the retry loop around
committing to disk.
---
storage/formats/disklabel.py | 23 ++++++-----------------
1 files changed, 6 insertions(+), 17 deletions(-)
diff --git a/storage/formats/disklabel.py b/storage/formats/disklabel.py
index a387c56..192f1da 100644
--- a/storage/formats/disklabel.py
+++ b/storage/formats/disklabel.py
@@ -197,23 +197,12 @@ class DiskLabel(DeviceFormat):
def commit(self):
""" Commit the current partition table to disk and notify the OS. """
- # give committing 5 tries, failing that, raise an exception
- attempt = 1
- maxTries = 5
- keepTrying = True
-
- while keepTrying and (attempt <= maxTries):
- try:
- self.partedDisk.commit()
- keepTrying = False
- except parted.DiskException as msg:
- log.warning(msg)
- attempt += 1
- else:
- udev_settle()
-
- if keepTrying:
- raise DeviceFormatError("cannot commit to disk after %d attempts" % (maxTries,), )
+ try:
+ self.partedDisk.commit()
+ except parted.DiskException as msg:
+ raise DeviceFormatError(msg)
+ else:
+ udev_settle()
def commitToDisk(self):
""" Commit the current partition table to disk. """
--
1.6.4.2
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
