Anaconda-devel December 2009

anaconda-devel@lists.stg.fedoraproject.org

30 participants
103 discussions

Re: Added *Anaconda Stage* to the How_to_debug_installation_problems page
by He Rui 22 Dec '09

22 Dec '09

On 12/22/2009 06:17 PM, Moray Henderson (ICT) wrote: > > He Rui wrote: > > Greetings, > > When testing boot methods of installation, '/anaconda is able to > transition to stage2 without error' /is really important to make sure > that the install actually starts the anaconda. But many people feel > confused of what the stages are during installation. Therefore, I > created a page[1] telling the differences and how to judge between > them. And this probably is something we should publicize better for > bug reporters as well. So I also add the *Anaconda Stage* content to > the existing How_to_debug_installation_problems wiki page[2]. > > Can someone please review it? Feel free for comments and improvements. > :-) > > > Cheers, > Rui He > > [1] http://fedoraproject.org/wiki/Anaconda/Stage > [2] > http://fedoraproject.org/wiki/How_to_debug_installation_problems#Anaconda_S… > > > Good articles – easy to understand. > Hi Moray, Yeah, I was afraid of making readers more confused. :-) Thanks, He Rui > Moray. > "To err is human. To purr, feline" > > > _______________________________________________ > Anaconda-devel-list mailing list > Anaconda-devel-list(a)redhat.com > https://www.redhat.com/mailman/listinfo/anaconda-devel-list -- Contacts FAS name: Rhe IRC nick: rhe #fedora-qa Email: rhe(a)redhat.com

1 0

Installing on pre-existing LVs over raid
by Clyde E. Kunkel 22 Dec '09

22 Dec '09

Does anaconda 13.12-1 support installing on pre-existing LVs? TIA

2 2

[PATCH] Remove a bunch of udev rules now provided by device-mapper.
by David Lehman 22 Dec '09

22 Dec '09

--- 70-anaconda.rules | 9 --------- 1 files changed, 0 insertions(+), 9 deletions(-) diff --git a/70-anaconda.rules b/70-anaconda.rules index 51a22f0..015248b 100644 --- a/70-anaconda.rules +++ b/70-anaconda.rules @@ -19,15 +19,6 @@ IMPORT{program}="$env{ANACBIN}/dmsetup info -c --nameprefixes --unquoted --rows ENV{DM_NAME}!="?*", GOTO="anaconda_end" ENV{DM_UUID}=="CRYPT-TEMP-?*", GOTO="anaconda_end" ENV{DM_UUID}!="?*", ENV{DM_NAME}=="temporary-cryptsetup-?*", GOTO="anaconda_end" -IMPORT{program}="$env{ANACBIN}/blkid -o udev -p $tempnode" - -SYMLINK+="disk/by-id/dm-name-$env{DM_NAME}" -ENV{DM_UUID}=="?*", SYMLINK+="disk/by-id/dm-uuid-$env{DM_UUID}" - -ENV{DM_STATE}=="SUSPENDED", GOTO="anaconda_end" - -OPTIONS="link_priority=-100" -ENV{ID_FS_USAGE}=="filesystem|other|crypto", ENV{ID_FS_UUID_ENC}=="?*", SYMLINK+="disk/by-uuid/$env{ID_FS_UUID_ENC}" LABEL="anaconda_mdraid" KERNEL!="md*", GOTO="anaconda_mdraid_member" -- 1.6.2.5

1 1

Storage state dumping, take two.
by David Lehman 22 Dec '09

22 Dec '09

PATCH 1/3 Sort by name for the sake of predictability. PATCH 2/3 No changes. PATCH 3/3 This version moves the dumping mechanism into Storage and adds a timestamp to the shelve keys, so instead of "devices.initial" you get something like "devices.23413534645.initial". This will cover crazy live-media people who do multiple installs, users adding iscsi/fcoe/zfcp/whatever, users going back and forth alot between screens. I hope all those people appreciate this.

2 4

Storage state dumping
by David Lehman 22 Dec '09

22 Dec '09

Here are some patches that dump storage state at the beginning and end of installation. Device and DeviceFormat classes define a "dict" attribute containing all of the important data describing that instance of the class. The dicts are then pickled using the shelve module into a file called "storage.state". The dicts can be retrieved from this file using the shelve module and, eventually, used to create Device and DeviceFormat instances to recreate installation scenarios.

2 8

[PATCH] repo UI patches
by Radek Vykydal 22 Dec '09

22 Dec '09

3 7

[PATCH] Put fsprofile support back in.
by David Lehman 22 Dec '09

22 Dec '09

--- kickstart.py | 15 ++++++--------- storage/formats/fs.py | 6 +++++- 2 files changed, 11 insertions(+), 10 deletions(-) diff --git a/kickstart.py b/kickstart.py index adab33a..5ba0773 100644 --- a/kickstart.py +++ b/kickstart.py @@ -484,9 +484,8 @@ class LogVolData(commands.logvol.F12_LogVolData): maxsize=self.maxSizeMB, percent=self.percent) - # FIXME: no way to specify an fsprofile right now - # if lvd.fsprofile: - # request.format.fsprofile = lvd.fsprofile + if self.fsprofile and hasattr(request.format, "fsprofile"): + request.format.fsprofile = self.fsprofile storage.createDevice(request) @@ -734,9 +733,8 @@ class PartitionData(commands.partition.F12_PartData): request = storage.newPartition(**kwargs) - # FIXME: no way to specify an fsprofile right now - # if self.fsprofile: - # request.format.fsprofile = self.fsprofile + if self.fsprofile and hasattr(request.format, "fsprofile"): + request.format.fsprofile = self.fsprofile storage.createDevice(request) @@ -874,9 +872,8 @@ class RaidData(commands.raid.F12_RaidData): except ValueError, e: raise KickstartValueError, formatErrorMsg(self.lineno, msg=str(e)) - # FIXME: no way to specify an fsprofile right now - # if pd.fsprofile: - # request.format.fsprofile = pd.fsprofile + if self.fsprofile and hasattr(request.format, "fsprofile"): + request.format.fsprofile = self.fsprofile storage.createDevice(request) diff --git a/storage/formats/fs.py b/storage/formats/fs.py index 239386d..09e764d 100644 --- a/storage/formats/fs.py +++ b/storage/formats/fs.py @@ -124,6 +124,7 @@ class FS(DeviceFormat): _defaultInfoOptions = [] _migrationTarget = None _existingSizeFields = [] + _fsProfileSpecifier = None # mkfs option specifying fsprofile def __init__(self, *args, **kwargs): """ Create a FS instance. @@ -143,10 +144,10 @@ class FS(DeviceFormat): raise TypeError("FS is an abstract class.") DeviceFormat.__init__(self, *args, **kwargs) - # TODO: fsprofiles and other ways to add format args self.mountpoint = kwargs.get("mountpoint") self.mountopts = kwargs.get("mountopts") self.label = kwargs.get("label") + self.fsprofile = kwargs.get("fsprofile") # filesystem size does not necessarily equal device size self._size = kwargs.get("size", 0) @@ -297,6 +298,8 @@ class FS(DeviceFormat): if options and isinstance(options, list): argv.extend(options) argv.extend(self.defaultFormatOptions) + if self._fsProfileSpecifier and self.fsprofile: + argv.extend([self._fsProfileSpecifier, self.fsprofile]) argv.append(self.device) return argv @@ -872,6 +875,7 @@ class Ext2FS(FS): _infofs = "dumpe2fs" _defaultInfoOptions = ["-h"] _existingSizeFields = ["Block count:", "Block size:"] + _fsProfileSpecifier = "-T" partedSystem = fileSystemType["ext2"] def _fsckFailed(self, rc): -- 1.6.5.2

2 1

[PATCH] Fix bootloader driveorder dialog.
by Radek Vykydal 22 Dec '09

22 Dec '09

_genStore expects disk objects, not disk names. --- iw/bootloader_main_gui.py | 8 +++----- 1 files changed, 3 insertions(+), 5 deletions(-) diff --git a/iw/bootloader_main_gui.py b/iw/bootloader_main_gui.py index 5f2fec5..fa5d554 100644 --- a/iw/bootloader_main_gui.py +++ b/iw/bootloader_main_gui.py @@ -98,11 +98,9 @@ class MainBootloaderWindow(InstallWindow): combo.set_attributes(cell, text = 0) for disk in disks: - size = disk.size - m = disk.description - i = model.append(None) - model[i] = ("%s %8.0f MB %s" %(disk.name, size, m), + model[i] = ("%s %8.0f MB %s" %(disk.name, disk.size, + disk.description), "%s" %(disk.name,)) if disk.name == active: combo.set_active_iter(i) @@ -138,7 +136,7 @@ class MainBootloaderWindow(InstallWindow): lbl.show() partitioned = anaconda.id.storage.partitioned disks = anaconda.id.storage.disks - bl_disks = [d.name for d in disks if d in partitioned] + bl_disks = [d for d in disks if d in partitioned] m = __genStore(combo, bl_disks, self.driveorder[i - 1]) dxml.get_widget("bd1Combo").connect("changed", __driveChange, dxml, choices) -- 1.6.0.6

2 1

[PATCH] Fix selection of default boot target in UI (#548695)
by Radek Vykydal 22 Dec '09

22 Dec '09

Corrects translation between device name and device path. Changes rendering style of toggle button in list from check to radio (which is how it behaves). --- iw/osbootwidget.py | 3 ++- 1 files changed, 2 insertions(+), 1 deletions(-) diff --git a/iw/osbootwidget.py b/iw/osbootwidget.py index 86f1718..fcde027 100644 --- a/iw/osbootwidget.py +++ b/iw/osbootwidget.py @@ -70,6 +70,7 @@ class OSBootWidget: active = 0) column.set_sizing(gtk.TREE_VIEW_COLUMN_AUTOSIZE) self.checkboxrenderer.connect("toggled", self.toggledDefault) + self.checkboxrenderer.set_radio(True) self.osTreeView.append_column(column) for columnTitle in theColumns[1:]: @@ -336,7 +337,7 @@ class OSBootWidget: def toggledDefault(self, data, row): iter = self.osStore.get_iter((int(row),)) dev = self.osStore.get_value(iter, 2) - self.defaultDev = dev[5:] + self.defaultDev = devicePathToName(dev) self.fillOSList() # fill in the os list tree view -- 1.6.0.6

2 1

[master] 'cleardiskssel' typos that made it impossible to run text install.
by Ales Kozumplik 21 Dec '09

21 Dec '09

--- kickstart.py | 4 ++-- text.py | 2 +- upgrade.py | 2 +- 3 files changed, 4 insertions(+), 4 deletions(-) diff --git a/kickstart.py b/kickstart.py index adab33a..3fbb3d5 100644 --- a/kickstart.py +++ b/kickstart.py @@ -299,7 +299,7 @@ class ClearPart(commands.clearpart.FC3_ClearPart): anaconda.id.storage.reinitializeDisks = self.initAll clearPartitions(anaconda.id.storage) - self.handler.skipSteps.append("cleardisksel") + self.handler.skipSteps.append("cleardiskssel") class Fcoe(commands.fcoe.F13_Fcoe): def parse(self, args): @@ -1439,7 +1439,7 @@ def setSteps(anaconda): if anaconda.id.displayMode == "t": missingSteps = [("bootloader", "Bootloader configuration"), ("filter", "Disks to use in installation"), - ("cleardisksel", "Disks to clear"), + ("cleardiskssel", "Disks to clear"), ("group-selection", "Package selection")] errors = [] diff --git a/text.py b/text.py index 3691e21..60eafb7 100644 --- a/text.py +++ b/text.py @@ -551,7 +551,7 @@ class InstallInterface: def setSteps(self, anaconda): anaconda.dispatch.skipStep("filtertype") anaconda.dispatch.skipStep("filter") - anaconda.dispatch.skipStep("cleardisksel") + anaconda.dispatch.skipStep("cleardiskssel") anaconda.dispatch.skipStep("basepkgsel") anaconda.dispatch.skipStep("group-selection") diff --git a/upgrade.py b/upgrade.py index f878812..58960ec 100644 --- a/upgrade.py +++ b/upgrade.py @@ -267,7 +267,7 @@ def setSteps(anaconda): "welcome", "filtertype", "filter", - "cleardisksel", + "cleardiskssel", "installtype", "storageinit", "findrootparts", -- 1.6.2.5

2 1

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Anaconda-devel December 2009