Hi,
On 12/23/2009 06:45 PM, David Lehman wrote:
On Tue, 2009-12-22 at 21:43 +0100, Hans de Goede wrote:
When we don't specify any MD / LVM / DM / LUKS to activate dracut will
activate all found by default, so when we don't specify any write
RD_NO_foo to the kernel cmdline in grub.
The idea is fine, but...
---
booty/bootloaderInfo.py | 13 ++++++++++++-
1 files changed, 12 insertions(+), 1 deletions(-)
diff --git a/booty/bootloaderInfo.py b/booty/bootloaderInfo.py
index 6e6d403..3d77254 100644
--- a/booty/bootloaderInfo.py
+++ b/booty/bootloaderInfo.py
@@ -88,16 +88,27 @@ class KernelArguments:
def getDracutStorageArgs(self):
args = []
+ types = {}
root = self.id.storage.rootDevice
for d in self.id.storage.devices:
if not root.dependsOn(d):
continue
- args.append(d.dracutSetupString())
+ s = d.dracutSetupString()
+ types[s.split("=")[0]] = True
+ args.append(s)
+
import storage
if isinstance(d, storage.devices.NetworkStorageDevice):
args.append(self.id.network.dracutSetupString(d))
+ for i in [ [ "rd_LUKS_UUID", "rd_NO_LUKS" ],
+ [ "rd_LVM_VG", "rd_NO_LVM" ],
+ [ "rd_MD_UUID", "rd_NO_MD" ],
+ [ "rd_DM_UUID", "rd_NO_DM" ] ]:
+ if not types.has_key(i[0]):
+ args.append(i[1])
+
return args
def get(self):
It would be a bit more straightforward for types to be a list. Then you
can just do:
types.append(s.split("=")[0])
and, later,
if i[0] not in types:
Good point, I'll respin this patch with that change (may take
some time before I get around to doing that).
Regards,
Hans
_______________________________________________
Anaconda-devel-list mailing list
Anaconda-devel-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/anaconda-devel-list
