Re: [PATCH 5/5] Add a questionInitializeDisk method to the rescue interface (#582304)

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

 



Hi,

On 04/14/2010 06:51 PM, Steffen Maier wrote:
On 04/14/2010 05:47 PM, Hans de Goede wrote:
And for good measure also add a questionReinitInconsistentLVM method.
---
  rescue.py |    8 ++++++++
  1 files changed, 8 insertions(+), 0 deletions(-)

diff --git a/rescue.py b/rescue.py
index a19985a..269b9b3 100644
--- a/rescue.py
+++ b/rescue.py
@@ -110,6 +110,14 @@ class RescueInterface(InstallInterfaceBase):
      def resetReinitInconsistentLVMQuestion(self):
          self._inconsistentLVMAnswers = {}

+    def questionInitializeDisk(self, path, description, size, details=""):
+        # Never initialize disks in rescue mode!
+        return False
+
+    def questionReinitInconsistentLVM(self, pv_names=None, lv_name=None, vg_name=None):
+        # Never reinit VG's in rescue mode!
+        return False
+
      def shutdown (self):
          pass


That's cool and reminds me of low-level formatting of DASDs which we
also do not want here for rescue mode by any means.
I'm not familiar enough with the code paths, would DASD.startup() not be
called anyway

Oh, good catch, please file a bug against RHEL-6 for this, then I or David
will take care of it.

Regards,

Hans

_______________________________________________
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