Bug 150630
Summary: | traceback on boot loader install grubbyDiskName | ||
---|---|---|---|
Product: | [Fedora] Fedora | Reporter: | Charles R. Anderson <cra> |
Component: | booty | Assignee: | Anaconda Maintenance Team <anaconda-maint-list> |
Status: | CLOSED RAWHIDE | QA Contact: | Mike McLean <mikem> |
Severity: | medium | Docs Contact: | |
Priority: | medium | ||
Version: | rawhide | CC: | clumens |
Target Milestone: | --- | ||
Target Release: | --- | ||
Hardware: | i386 | ||
OS: | Linux | ||
Whiteboard: | |||
Fixed In Version: | Doc Type: | Bug Fix | |
Doc Text: | Story Points: | --- | |
Clone Of: | Environment: | ||
Last Closed: | 2005-03-09 20:20:37 UTC | Type: | --- |
Regression: | --- | Mount Type: | --- |
Documentation: | --- | CRM: | |
Verified Versions: | Category: | --- | |
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | |
Cloudforms Team: | --- | Target Upstream Version: | |
Embargoed: |
Description
Charles R. Anderson
2005-03-08 23:48:39 UTC
Problem appears to be this code in grubby. Looks like the hack isn't going to cut it anymore. for drive in devs: # XXX hack city. If they're not the sort of thing that'll # be in the device map, they shouldn't still be in the list. if not drive.startswith('md') and not drive.startswith('Vol'): f.write("(%s) /dev/%s\n" % (self.grubbyDiskName(drive), drive)) f.close() Moving this bug to booty component, version 0.49-1. Some other problems are apparent in the code w.r.t LVM device names. E.g.: # return (disk, partition number) eg ('hda', 1) def getDiskPart(dev): getDiskPart() is handed LVM partition names from right before the "XXX hack city" code: for dev in devs: drive = getDiskPart(dev)[0] if usedDevs.has_key(drive): continue usedDevs[drive] = 1 devs = usedDevs.keys() devs.sort() Those come from back further when rootDev is LVM: for dev in self.getPhysicalDevices(rootDev): usedDevs[dev] = 1 Should this function be handling LVM devices? def getPhysicalDevices(self, device): # This finds a list of devices on which the given device name resides. # Accepted values for "device" are raid1 md devices (i.e. "md0"), # physical disks ("hda"), and real partitions on physical disks # ("hda1"). The comment "# XXX hack city. If they're not the sort of thing that'll # be in the device map, they shouldn't still be in the list." seems to suggest that it doesn't want LVM devices in the usedDevs list. |