May 2012 - Anaconda-devel - Fedora mailing-lists

Ability to install to loopmounted filesystem - mentor wanted
by Tirtha Chatterjee 02 Dec '12

02 Dec '12

Hello. I am trying to implement something like Wubi on Fedora. For this, I first need to make it possible to install to loopmounted filesystems through anaconda. I don't think a lot of modification to the existing source code is required. I tried going through the source code, but it seemed quite big to manage without some guidance. I would be extremely happy if someone guided me on this. Hoping for an affirmative reply :-) -- Regards Tirtha Chatterjee KDE developer http://wyuka.co.cc/

2 3

[newui] Setting up multiple layouts in the installation and target
by Vratislav Podzimek 05 Jun '12

05 Jun '12

This is the second version of patches that enable two things: 1) Setting up multiple layouts defined in the kickstart ASAP, so that they are already active when the GUI appears. 2) Setting up multiple layouts in the installed system. -- Vratislav Podzimek

2 8

[lorax 1/2] Install rpcbind (#824835)
by Martin Gracik 01 Jun '12

01 Jun '12

Dracut nfs module needs it. --- share/runtime-install.tmpl | 3 +++ 1 file changed, 3 insertions(+) diff --git a/share/runtime-install.tmpl b/share/runtime-install.tmpl index 4916b4e..d15aaf2 100644 --- a/share/runtime-install.tmpl +++ b/share/runtime-install.tmpl @@ -38,6 +38,9 @@ installpkg kernel ## extra dracut modules installpkg dracut-network anaconda-dracut +## rpcbind or portmap needed by dracut nfs module +installpkg rpcbind + ## required for dracut installpkg kbd kbd-misc ## required for anaconda-dracut (img-lib etc.) -- 1.7.9.5

2 4

[PATCH rhel6-branch] Handle offload qla4xxx iscsi devices properly (#823810)
by Radek Vykydal 01 Jun '12

01 Jun '12

These kind of devices, when qla4xxx.ql4xdisablesysfsboot=1, is discovered by anaconda/udev as iscsi device yet it can not be handled by iscsiadm so we need to treat it specially. Resolves: rhbz#823810 --- storage/devices.py | 33 +++++++++++++++++++++++++-------- storage/devicetree.py | 30 ++++++++++++++++++++---------- storage/udev.py | 28 +++++++++++++++++++++++++--- 3 files changed, 70 insertions(+), 21 deletions(-) diff --git a/storage/devices.py b/storage/devices.py index 96c6238..674ef5a 100644 --- a/storage/devices.py +++ b/storage/devices.py @@ -3634,19 +3634,36 @@ class iScsiDiskDevice(DiskDevice, NetworkStorageDevice): self.ibft = kwargs.pop("ibft") self.nic = kwargs.pop("nic") self.initiator = kwargs.pop("initiator") - DiskDevice.__init__(self, device, **kwargs) - NetworkStorageDevice.__init__(self, host_address=self.node.address, - nic=self.nic) - log.debug("created new iscsi disk %s %s:%d via %s:%s" % (self.node.name, - self.node.address, - self.node.port, - self.node.iface, - self.nic)) + + if self.node is None: + # qla4xxx partial offload + name = kwargs.pop("fw_name") + address = kwargs.pop("fw_address") + port = kwargs.pop("fw_port") + DiskDevice.__init__(self, device, **kwargs) + NetworkStorageDevice.__init__(self, + host_address=address, + nic=self.nic) + log.debug("created new iscsi disk %s %s:%s using fw initiator %s" + % (name, address, port, self.initiator)) + else: + DiskDevice.__init__(self, device, **kwargs) + NetworkStorageDevice.__init__(self, host_address=self.node.address, + nic=self.nic) + log.debug("created new iscsi disk %s %s:%d via %s:%s" % (self.node.name, + self.node.address, + self.node.port, + self.node.iface, + self.nic)) def dracutSetupArgs(self): if self.ibft: return set(["iscsi_firmware"]) + # qla4xxx partial offload + if self.node is None: + return set() + address = self.node.address # surround ipv6 addresses with [] if ":" in address: diff --git a/storage/devicetree.py b/storage/devicetree.py index e6ae639..30d0a2a 100644 --- a/storage/devicetree.py +++ b/storage/devicetree.py @@ -1180,16 +1180,26 @@ class DeviceTree(object): kwargs = { "serial": serial, "vendor": vendor, "bus": bus } if udev_device_is_iscsi(info): diskType = iScsiDiskDevice - node = self.iscsi.getNode( - udev_device_get_iscsi_name(info), - udev_device_get_iscsi_address(info), - udev_device_get_iscsi_port(info), - udev_device_get_iscsi_nic(info)) - kwargs["node"] = node - kwargs["nic"] = self.iscsi.ifaces.get(node.iface, node.iface) - kwargs["ibft"] = node in self.iscsi.ibftNodes - kwargs["initiator"] = self.iscsi.initiator - log.debug("%s is an iscsi disk" % name) + initiator = udev_device_get_iscsi_initiator(info) + target = udev_device_get_iscsi_name(info) + address = udev_device_get_iscsi_address(info) + port = udev_device_get_iscsi_port(info) + nic = udev_device_get_iscsi_nic(info) + kwargs["initiator"] = initiator + if initiator == self.iscsi.initiator: + node = self.iscsi.getNode(target, address, port, nic) + kwargs["node"] = node + kwargs["ibft"] = node in self.iscsi.ibftNodes + kwargs["nic"] = self.iscsi.ifaces.get(node.iface, node.iface) + log.debug("%s is an iscsi disk" % name) + else: + # qla4xxx partial offload + kwargs["node"] = None + kwargs["ibft"] = False + kwargs["nic"] = "offload:not_accessible_via_iscsiadm" + kwargs["fw_address"] = address + kwargs["fw_port"] = port + kwargs["fw_name"] = name elif udev_device_is_fcoe(info): diskType = FcoeDiskDevice kwargs["nic"] = udev_device_get_fcoe_nic(info) diff --git a/storage/udev.py b/storage/udev.py index f00e316..15ec238 100644 --- a/storage/udev.py +++ b/storage/udev.py @@ -553,19 +553,41 @@ def udev_device_get_iscsi_port(info): # IPV6 contains : within the address, the part after the last : is the port return path_components[address_field].split(":")[-1] -def udev_device_get_iscsi_nic(info): +def udev_device_get_iscsi_session(info): # '/devices/pci0000:00/0000:00:02.0/0000:09:00.0/0000:0a:01.0/0000:0e:00.2/host3/session1/target3:0:0/3:0:0:0/block/sda' # The position of sessionX part depends on device # (e.g. offload vs. sw; also varies for different offload devs) + session = None match = re.match('/.*/(session\d+)', info["sysfs_path"]) if match: session = match.groups()[0] + else: + log.error("udev_device_get_iscsi_session: session not found in %s" % info) + return session + + +def udev_device_get_iscsi_nic(info): + iface = None + session = udev_device_get_iscsi_session(info) + if session: iface = open("/sys/class/iscsi_session/%s/ifacename" % session).read().strip() - else: - iface = None return iface +def udev_device_get_iscsi_initiator(info): + initiator = None + if udev_device_is_partoff_iscsi(info): + host = re.match('.*/(host\d+)', info["sysfs_path"]).groups()[0] + initiator = open("/sys/class/iscsi_host/%s/initiatorname" % + host).read().strip() + else: + session = udev_device_get_iscsi_session(info) + if session: + initiator = open("/sys/class/iscsi_session/%s/initiatorname" % + session).read().strip() + return initiator + + # fcoe disks have ID_PATH in the form of: # For FCoE directly over the NIC (so no VLAN and thus no DCB): # pci-eth#-fc-${id} -- 1.7.4

2 3

[lorax 1/3] livemedia-creator: handle failed mount for ami
by Brian C. Lane 31 May '12

31 May '12

From: "Brian C. Lane" <bcl(a)redhat.com> --- src/sbin/livemedia-creator | 3 +++ 1 files changed, 3 insertions(+), 0 deletions(-) diff --git a/src/sbin/livemedia-creator b/src/sbin/livemedia-creator index 6b22315..3c37cb5 100755 --- a/src/sbin/livemedia-creator +++ b/src/sbin/livemedia-creator @@ -364,6 +364,9 @@ def make_ami( disk_img, ami_img="ami-root.img", ami_label="AMI" ): All other AMI setup is handled by the kickstart's %post """ with PartitionMount( disk_img ) as img_mount: + if not img_mount or not img_mount.mount_dir: + return None + work_dir = tempfile.mkdtemp() log.info("working dir is {0}".format(work_dir)) log.info("creating {0}".format(ami_img)) -- 1.7.7.6

1 2

[PATCH rhel7-alpha2-branch] Don't depend on plymouth for shell (#826984)
by Jesse Keating 31 May '12

31 May '12

--- data/systemd/anaconda-shell@.service | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/data/systemd/anaconda-shell@.service b/data/systemd/anaconda-shell@.service index 52bed74..34d5848 100644 --- a/data/systemd/anaconda-shell@.service +++ b/data/systemd/anaconda-shell@.service @@ -3,7 +3,7 @@ [Unit] Description=Shell on %I BindTo=dev-%i.device -After=dev-%i.device systemd-user-sessions.service plymouth-quit-wait.service +After=dev-%i.device systemd-user-sessions.service [Service] WorkingDirectory=/ -- 1.7.10.2

2 2

Re: Allow sharing of the VNC server (Jesse Keating)
by Mark Hamzy 31 May '12

31 May '12

jkeating(a)redhat.com wrote on 05/31/2012 12:17:39 -0700: > Also, are you really sure we only allow one install ssh session? 'cause > I've sshed in as root@ multiple times in my testing, no restriction. Of > course, can't do multiple install@ because the shell for that user is > /sbin/anaconda :) Yes, I've verified that as well. You can have multiple ssh root@ip open at once. > Are there any other opinions here? I agree that if we change the default to open a shared vnc session and allow the user to specify a vncpassword (which we already do), then that should address any security concerns. Mark

1 0

[newui] network spoke patches
by Radek Vykydal 31 May '12

31 May '12

Apply method and two UI refresh fixes.

2 7

[PATCH rhel7-alpha2-branch] dracut: fix parse-kickstart traceback with 'network' command (#824842)
by Will Woods 31 May '12

31 May '12

str.join() takes a single list argument. It won't take a list of arguments. Grr. Also, fix some goofiness with empty items (sometimes they're None and sometimes they're "") --- dracut/parse-kickstart | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/dracut/parse-kickstart b/dracut/parse-kickstart index 9baece0..12c9f8b 100755 --- a/dracut/parse-kickstart +++ b/dracut/parse-kickstart @@ -180,11 +180,17 @@ def ksnet_to_dracut(args, lineno, net, bootdev=False): if ":" in net.gateway: net.gateway = "[%s]" % net.gateway + if net.ipv6 and not net.ip: + ip = "[%s]" % net.ipv6 + else: + ip = net.ip - ip=":".join(net.ip or "[%s]" % net.ipv6, "", net.gateway, net.netmask, - net.hostname, net.device, net.autoconf, net.mtu) + iplist = [ip, "", net.gateway, net.netmask, + net.hostname, net.device, net.autoconf] + if net.mtu: + iplist.append(net.mtu) - line.append("ip=%s" % ip) + line.append("ip=" + ":".join(i or "" for i in iplist)) for ns in net.nameserver.split(","): if ns: -- 1.7.10.2

2 1

[PATCH] Set the python path within anaconda
by Jesse Keating 31 May '12

31 May '12

Instead of manipulating the path outside of anaconda, which can get tricky when starting anaconda from ssh, instead update the path inside anaconda itself. This means we can't get things like os, sys, etc.. from /tmp/updates/, but an updates image can replace these modules by using the full path to them (e.g. /usr/lib/python...) --- anaconda | 2 ++ data/systemd/anaconda@.service | 2 +- 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/anaconda b/anaconda index b76af4d..4959a87 100755 --- a/anaconda +++ b/anaconda @@ -277,6 +277,8 @@ def parseOptions(argv=None, cmdline=None): return (opts, args, op.deprecated_bootargs) def setupPythonPath(): + # First add our updates path + sys.path.insert(0, '/tmp/updates/') sys.path.append('/usr/share/system-config-date') def setupEnvironment(): diff --git a/data/systemd/anaconda@.service b/data/systemd/anaconda@.service index 903df7a..03ecabb 100644 --- a/data/systemd/anaconda@.service +++ b/data/systemd/anaconda@.service @@ -3,7 +3,7 @@ Description=Anaconda After=anaconda.target [Service] -Environment=HOME=/root MALLOC_CHECK_=2 MALLOC_PERTURB_=204 PATH=/usr/bin:/bin:/sbin:/usr/sbin:/mnt/sysimage/bin:/mnt/sysimage/usr/bin:/mnt/sysimage/usr/sbin:/mnt/sysimage/sbin PYTHONPATH=/tmp/updates +Environment=HOME=/root MALLOC_CHECK_=2 MALLOC_PERTURB_=204 PATH=/usr/bin:/bin:/sbin:/usr/sbin:/mnt/sysimage/bin:/mnt/sysimage/usr/bin:/mnt/sysimage/usr/sbin:/mnt/sysimage/sbin Type=oneshot WorkingDirectory=/root ExecStart=/usr/sbin/anaconda -- 1.7.10.2

2 1

[PATCH rhel7-alpha2-branch] Start ssh with systemd, not in anaconda (#824580)
by Radek Vykydal 31 May '12

31 May '12

In this take: 1) sshd is started always, regardless of sshd boot option, (as in s390x) with empty root password. 2) sshpw kickstart command works, so root/users passwords can be set, but this happens in anaconda so between sshd and anaconda is run there is root ssh access without password. Is this ok for alpha2? To achieve original behaviour we'll need to parse sshd option in dracut and change sshd (and/or perhaps anaconda/sshd systemd) configuration files in dracut. Setting passwords (at least for root) already in dracut may be needed. I'd need to consult this with Will, or leave it to him. Resolves: rhbz#824580 --- data/systemd/anaconda.target | 1 + pyanaconda/sshd.py | 46 ++++------------------------------------- 2 files changed, 6 insertions(+), 41 deletions(-) diff --git a/data/systemd/anaconda.target b/data/systemd/anaconda.target index 983ff3b..1af847d 100644 --- a/data/systemd/anaconda.target +++ b/data/systemd/anaconda.target @@ -9,3 +9,4 @@ Wants=rsyslog.service Wants=udev-settle.service Wants=NetworkManager.service Wants=plymouth-quit.service plymouth-quit-wait.service +Wants=sshd.service diff --git a/pyanaconda/sshd.py b/pyanaconda/sshd.py index b6c9372..300d943 100644 --- a/pyanaconda/sshd.py +++ b/pyanaconda/sshd.py @@ -23,34 +23,16 @@ import logging import os, sys log = logging.getLogger("anaconda") -import iutil import users from flags import flags from constants import ROOT_PATH -def createSshKey(algorithm, keyfile): - path = '/etc/ssh/%s' % (keyfile,) - argv = ['-q','-t',algorithm,'-f',path,'-C','','-N',''] - if os.access(path, os.R_OK): - return - log.debug("running \"%s\"" % (" ".join(['ssh-keygen']+argv),)) - - so = "/tmp/ssh-keygen-%s-stdout.log" % (algorithm,) - se = "/tmp/ssh-keygen-%s-stderr.log" % (algorithm,) - iutil.execWithRedirect('ssh-keygen', argv, stdout=so, stderr=se) - def doSshd(anaconda): - if flags.sshd: - # we need to have a libuser.conf that points to the installer root for - # sshpw, but after that we start sshd, we need one that points to the - # install target. - luserConf = users.createLuserConf(instPath="") - handleSshPw(anaconda) - startSsh() - del(os.environ["LIBUSER_CONF"]) - else: - log.info("sshd: not enabled, skipping.") - + # we need to have a libuser.conf that points to the installer root for + # sshpw, but after that we need one that points to the install target. + luserConf = users.createLuserConf(instPath="") + handleSshPw(anaconda) + del(os.environ["LIBUSER_CONF"]) users.createLuserConf(ROOT_PATH) def handleSshPw(anaconda): @@ -70,21 +52,3 @@ def handleSshPw(anaconda): u.createUser(ud.username, **kwargs) del u - -def startSsh(): - if iutil.isS390(): - return - - if not iutil.fork_orphan(): - os.open("/var/log/lastlog", os.O_RDWR | os.O_CREAT, 0644) - ssh_keys = { - 'rsa1':'ssh_host_key', - 'rsa':'ssh_host_rsa_key', - 'dsa':'ssh_host_dsa_key', - } - for (algorithm, keyfile) in ssh_keys.items(): - createSshKey(algorithm, keyfile) - sshd = iutil.find_program_in_path("sshd") - args = [sshd, "-f", "/etc/ssh/sshd_config.anaconda"] - os.execv(sshd, args) - sys.exit(1) -- 1.7.4

4 5

[newui] NTP servers configuration
by Vratislav Podzimek 31 May '12

31 May '12

These three patches introduce the NTP servers configuration button and dialog on the Datetime spoke. Video preview can be found at: http://vpodzime.fedorapeople.org/ntp_config_dialog.ogg There is a question when to refresh the working states of the servers. For now, the are refreshed in the refresh method of the spoke and when "Cancel" is hit on the configuration dialog (that results in repopulation of the store with servers from the config file. I think this should be both enough and not too much. -- Vratislav Podzimek

2 12

[lorax] Don't remove sha256sum from the install image
by Martin Gracik 31 May '12

31 May '12

--- share/runtime-cleanup.tmpl | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/share/runtime-cleanup.tmpl b/share/runtime-cleanup.tmpl index 5a08ab1..b454e67 100644 --- a/share/runtime-cleanup.tmpl +++ b/share/runtime-cleanup.tmpl @@ -154,7 +154,7 @@ removefrom coreutils /usr/bin/mkfifo /usr/bin/nl /usr/bin/nohup /usr/bin/nproc removefrom coreutils /usr/bin/od /usr/bin/paste /usr/bin/pathchk removefrom coreutils /usr/bin/pinky /usr/bin/pr /usr/bin/printenv removefrom coreutils /usr/bin/printf /usr/bin/ptx /usr/bin/runcon /usr/bin/seq -removefrom coreutils /usr/bin/sha224sum /usr/bin/sha256sum /usr/bin/sha384sum +removefrom coreutils /usr/bin/sha224sum /usr/bin/sha384sum removefrom coreutils /usr/bin/sha512sum /usr/bin/shuf /usr/bin/stat removefrom coreutils /usr/bin/stdbuf /usr/bin/sum /usr/bin/test removefrom coreutils /usr/bin/timeout /usr/bin/truncate /usr/bin/tsort -- 1.7.9.5

1 0

[grubby] Pick the last / device (#826537)
by Brian C. Lane 30 May '12

30 May '12

From: "Brian C. Lane" <bcl(a)redhat.com> When there are multiple devices mounted on /, as there can be during a preupgrade, the last device is the one that should match the grub.cfg entry. This was preventing preupgrade from installing the new f17 kernel. Related: rhbz#820340 Related: rhbz#820351 --- grubby.c | 18 +++++++++++------- 1 files changed, 11 insertions(+), 7 deletions(-) diff --git a/grubby.c b/grubby.c index b7d989d..fe8317d 100644 --- a/grubby.c +++ b/grubby.c @@ -1460,6 +1460,8 @@ static char *findDiskForRoot() buf[rc] = '\0'; chptr = buf; + char *foundanswer = NULL; + while (chptr && chptr != buf+rc) { devname = chptr; @@ -1487,13 +1489,8 @@ static char *findDiskForRoot() * for '/' obviously. */ if (*(++chptr) == '/' && *(++chptr) == ' ') { - /* - * Move back 2, which is the first space after the device name, set - * it to \0 so strdup will just get the devicename. - */ - chptr -= 2; - *chptr = '\0'; - return strdup(devname); + /* remember the last / entry in mtab */ + foundanswer = devname; } /* Next line */ @@ -1502,6 +1499,13 @@ static char *findDiskForRoot() chptr++; } + /* Return the last / entry found */ + if (foundanswer) { + chptr = strchr(foundanswer, ' '); + *chptr = '\0'; + return strdup(foundanswer); + } + return NULL; } -- 1.7.7.6

1 0

Allow sharing of the VNC server
by Jesse Keating 30 May '12

30 May '12

I'm not really sure what might break here, we've always used -nevershared since the vnc code was first added to Anaconda waaaay back when. But it's worth testing in rawhide to see what falls over I suppose. -- jlk

3 4

Anaconda storage module does not see LVM
by Jan Safranek 29 May '12

29 May '12

Hi, I have one volume group /dev/vgMine with one logical volume /dev/vgMine/lvMine and I have troubles seeing these devices in Anaconda storage module. E.g. this simple program does not list them: <imports omitted> platform = pyanaconda.platform.getPlatform(None) storage = pyanaconda.storage.Storage(platform=platform) storage.devicetree.populate() print storage.vgs, storage.lvs --> [] [] print storage.devices -> neither vgMine nor lvMine is there I am pretty sure I am doing something wrong, I'd bet you test such basic cases. I'm using anaconda-18.2-1.fc18. You can see storage log attached. It seems /dev/sda1 is correctly detected as lvmpv, but its vgName = None and vgUUID = None. vgMine is not added to the device tree and later lvMine is thrown away, because it misses vgMine. Jan

2 1

GUI TODO
by Martin Sivak 29 May '12

29 May '12

Hi Chris, some of the outstanding entries in the TODO list in newui branch lack enough context. We mostly understand the summaries, but would like to know more about your ideas and directions you want to pursue. Can you please elaborate on some of the following items? (in the TODO itself or on the list). We can think about solutions ourselves, but as it is your design, you probably have ideas about how to make it properly and consistently. We would like to hear those. - Custom partitioning needs to be revisited, then written. - Disk tug-o-war needs to be written. - Test handling of inconsistent LVM/RAID, and uninitialized disks. If I followed correctly, this is what you are doing ATM, right? - Install hub needs to be skipped if all spokes are completed on a kickstart install. How to handle the fact that spokes may only be ready once they're finished downloading? This would be handled in Hub's refresh method I assume. Can we detect that a spoke is still downloading? I mean, do we have a "spoke ready" message? We might just show the Hub screen with information about the ongoing operation and then continue once everything is ready && completed. I am not sure about whether we should allow the user to make any changes while in ks mode, we used to ask for stuff we didn't get, so we probably should.. - Update exception handling. This requires updating python-meh to use gtk3, which also requires updating firstboot and s-c-ks to gtk3. I want to turn s-c-ks into an anaconda wrapper first, though. Anything we can help with (the gtk port is obvious)? How do you envision the wrapper to work and interact with newui anaconda? Do we still want s-c-ks? I thought we are about to kill it and use the newui in standalone mode. And what about firstboot and anaconda widgets? Do we still plan to implement interoperability there? (I am asking because there is another project seeking to replace firstboot - https://fedoraproject.org/wiki/Features/InitialExperience ) - Wall of anaconda needs to be taken into account. I have no idea what this is supposed to mean... -- Martin Sivák msivak(a)redhat.com Red Hat Czech Anaconda team / Brno, CZ

2 1

[PATCH] Implementation of normal Language spoke
by Martin Sivak 29 May '12

29 May '12

Shares most of the code with WelcomeLanguageSpoke and transfers the selection ListView and label between the two display variants as needed. --- pyanaconda/ui/gui/spokes/welcome.py | 110 +++++++++++++++++++++++++++++------ pyanaconda/ui/gui/spokes/welcome.ui | 63 +++++++++++++++++++- 2 files changed, 153 insertions(+), 20 deletions(-) diff --git a/pyanaconda/ui/gui/spokes/welcome.py b/pyanaconda/ui/gui/spokes/welcome.py index b80ccb9..741d291 100644 --- a/pyanaconda/ui/gui/spokes/welcome.py +++ b/pyanaconda/ui/gui/spokes/welcome.py @@ -23,33 +23,34 @@ import sys import gettext _ = lambda x: gettext.ldgettext("anaconda", x) +N_ = lambda x: x from gi.repository import AnacondaWidgets, Gtk from pyanaconda.ui.gui.hubs.summary import SummaryHub -from pyanaconda.ui.gui.spokes import StandaloneSpoke +from pyanaconda.ui.gui.spokes import StandaloneSpoke, NormalSpoke from pyanaconda.ui.gui.utils import enlightbox +from pyanaconda.ui.gui.categories.localization import LocalizationCategory from pyanaconda.localization import Language, LOCALE_PREFERENCES from pyanaconda.product import productName, productVersion from pyanaconda import xklavier from pyanaconda import localization -__all__ = ["WelcomeLanguageSpoke"] +__all__ = ["WelcomeLanguageSpoke", "LanguageSpoke"] -class WelcomeLanguageSpoke(StandaloneSpoke): - mainWidgetName = "welcomeWindow" - uiFile = "spokes/welcome.ui" - - preForHub = SummaryHub - priority = 0 +class LanguageMixIn(object): + builderObjects = ["languageStore"] - def __init__(self, *args): - StandaloneSpoke.__init__(self, *args) + def __init__(self, labelName = "welcomeLabel", + viewName = "languageView", selectionName = "languageViewSelection"): self._xklwrapper = xklavier.XklWrapper.get_instance() self._origStrings = {} + self._labelName = labelName + self._viewName = viewName + self._selectionName = selectionName def apply(self): - selected = self.builder.get_object("languageViewSelection") + selected = self.builder.get_object(self._selectionName) (store, itr) = selected.get_selected() lang = store[itr][2] @@ -87,8 +88,6 @@ class WelcomeLanguageSpoke(StandaloneSpoke): return self.data.lang.lang and self.data.lang.lang != "" def initialize(self): - StandaloneSpoke.initialize(self) - store = self.builder.get_object("languageStore") # TODO We can use the territory from geoip here @@ -106,8 +105,7 @@ class WelcomeLanguageSpoke(StandaloneSpoke): self._selectLanguage(store, self.language.preferred_translation.short_name) def retranslate(self): - StandaloneSpoke.retranslate(self) - welcomeLabel = self.builder.get_object("welcomeLabel") + welcomeLabel = self.builder.get_object(self._labelName) if not welcomeLabel in self._origStrings: self._origStrings[welcomeLabel] = welcomeLabel.get_label() @@ -116,11 +114,25 @@ class WelcomeLanguageSpoke(StandaloneSpoke): xlated = _(before) % (productName.upper(), productVersion) welcomeLabel.set_label(xlated) - def refresh(self): - StandaloneSpoke.refresh(self) + def refresh(self, displayArea): store = self.builder.get_object("languageStore") self._selectLanguage(store, self.data.lang.lang) + # Rip the label and language selection window + # from where it is right now and add it to this + # spoke. + # This way we can share the dialog and code + # between Welcome and Language spokes + langList = self.builder.get_object("languageWindow") + langList.get_parent().remove(langList) + langLabel = self.builder.get_object("pickLanguageLabel") + langLabel.get_parent().remove(langLabel) + + content = self.builder.get_object(displayArea) + content.pack_start(child = langLabel, fill = True, expand = False, padding = 0) + content.pack_start(child = langList, fill = True, expand = True, padding = 0) + + def _addLanguage(self, store, native, english, setting): store.append([native, english, setting]) @@ -133,7 +145,7 @@ class WelcomeLanguageSpoke(StandaloneSpoke): if not itr: return - treeview = self.builder.get_object("languageView") + treeview = self.builder.get_object(self._viewName) selection = treeview.get_selection() selection.select_iter(itr) path = store.get_path(itr) @@ -142,7 +154,8 @@ class WelcomeLanguageSpoke(StandaloneSpoke): # Signal handlers. def on_selection_changed(self, selection): (store, selected) = selection.get_selected_rows() - self.window.set_may_continue(len(selected) > 0) + if hasattr(self.window, "set_may_continue"): + self.window.set_may_continue(len(selected) > 0) if selected: lang = store[selected[0]][2] @@ -150,6 +163,27 @@ class WelcomeLanguageSpoke(StandaloneSpoke): self.language.set_system_lang(lang) self.retranslate() + +class WelcomeLanguageSpoke(LanguageMixIn, StandaloneSpoke): + mainWidgetName = "welcomeWindow" + uiFile = "spokes/welcome.ui" + builderObjects = LanguageMixIn.builderObjects + [mainWidgetName, "betaWarnDialog"] + + preForHub = SummaryHub + priority = 0 + + def __init__(self, *args): + StandaloneSpoke.__init__(self, *args) + LanguageMixIn.__init__(self) + + def retranslate(self): + StandaloneSpoke.retranslate(self) + LanguageMixIn.retranslate(self) + + def refresh(self): + StandaloneSpoke.refresh(self) + LanguageMixIn.refresh(self, "welcomeWindowContentBox") + # Override the default in StandaloneSpoke so we can display the beta # warning dialog first. def _on_continue_clicked(self, cb): @@ -163,3 +197,41 @@ class WelcomeLanguageSpoke(StandaloneSpoke): sys.exit(0) else: StandaloneSpoke._on_continue_clicked(self, cb) + + +class LanguageSpoke(LanguageMixIn, NormalSpoke): + mainWidgetName = "languageSpokeWindow" + uiFile = "spokes/welcome.ui" + builderObjects = LanguageMixIn.builderObjects + [mainWidgetName, WelcomeLanguageSpoke.mainWidgetName] + + category = LocalizationCategory + + icon = "accessories-character-map-symbolic" + title = N_("LANGUAGE") + + + def __init__(self, *args): + NormalSpoke.__init__(self, *args) + LanguageMixIn.__init__(self) + + def retranslate(self): + print "retranslate" + NormalSpoke.retranslate(self) + LanguageMixIn.retranslate(self) + + def refresh(self): + NormalSpoke.refresh(self) + LanguageMixIn.refresh(self, "languageSpokeWindowContentBox") + + @property + def completed(self): + # The language spoke is always completed, as it does not require you do + # anything. There's always a default selected. + return True + + @property + def status(self): + selected = self.builder.get_object(self._selectionName) + (store, itr) = selected.get_selected() + + return store[itr][0] diff --git a/pyanaconda/ui/gui/spokes/welcome.ui b/pyanaconda/ui/gui/spokes/welcome.ui index 0d64ffc..655f9f7 100644 --- a/pyanaconda/ui/gui/spokes/welcome.ui +++ b/pyanaconda/ui/gui/spokes/welcome.ui @@ -119,6 +119,15 @@ OS you can rely on. It's for testing purposes only.</property> <child> <placeholder/> </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> + <child> + <placeholder/> + </child> </object> <packing> <property name="expand">True</property> @@ -133,6 +142,58 @@ OS you can rely on. It's for testing purposes only.</property> <action-widget response="1">continueButton</action-widget> </action-widgets> </object> + <object class="AnacondaSpokeWindow" id="languageSpokeWindow"> + <property name="startup_id">filler</property> + <property name="can_focus">False</property> + <property name="startup_id">filler</property> + <signal name="back-clicked" handler="on_back_clicked" swapped="no"/> + <child internal-child="main_box"> + <object class="GtkBox" id="AnacondaSpokeWindow-main_box1"> + <property name="can_focus">False</property> + <property name="orientation">vertical</property> + <property name="spacing">6</property> + <child internal-child="nav_area"> + <object class="GtkGrid" id="AnacondaSpokeWindow-nav_area1"> + <property name="can_focus">False</property> + <property name="margin_left">6</property> + <property name="margin_right">6</property> + <property name="margin_top">6</property> + </object> + <packing> + <property name="expand">False</property> + <property name="fill">False</property> + <property name="position">0</property> + </packing> + </child> + <child internal-child="alignment"> + <object class="GtkAlignment" id="AnacondaSpokeWindow-alignment1"> + <property name="can_focus">False</property> + <property name="yalign">0</property> + <property name="xscale">0</property> + <property name="yscale">0.5</property> + <child internal-child="action_area"> + <object class="GtkBox" id="languageSpokeWindowContentBox"> + <property name="can_focus">False</property> + <property name="orientation">vertical</property> + <property name="spacing">6</property> + <child> + <placeholder/> + </child> + </object> + </child> + </object> + <packing> + <property name="expand">True</property> + <property name="fill">True</property> + <property name="position">1</property> + </packing> + </child> + <child> + <placeholder/> + </child> + </object> + </child> + </object> <object class="GtkListStore" id="languageStore"> <columns>  @@ -174,7 +235,7 @@ OS you can rely on. It's for testing purposes only.</property> <property name="xscale">0</property> <property name="yscale">0.5</property> <child internal-child="action_area"> - <object class="GtkBox" id="welcomeWindow-actionArea"> + <object class="GtkBox" id="welcomeWindowContentBox"> <property name="can_focus">False</property> <property name="orientation">vertical</property> <property name="spacing">6</property> -- 1.7.10.1

2 1

[PATCH] Change cursor shape when the pointer is over SpokeSelector icon
by Martin Sivak 29 May '12

29 May '12

I have no idea where should we free private SpokeSelector variables.. I haven't found any structure or call for that.. any idea? --- widgets/src/SpokeSelector.c | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/widgets/src/SpokeSelector.c b/widgets/src/SpokeSelector.c index f4247a6..bb2162e 100644 --- a/widgets/src/SpokeSelector.c +++ b/widgets/src/SpokeSelector.c @@ -58,12 +58,14 @@ struct _AnacondaSpokeSelectorPrivate { GtkWidget *icon, *incomplete_icon; GtkWidget *title_label; GtkWidget *status_label; + GdkCursor *cursor; }; G_DEFINE_TYPE(AnacondaSpokeSelector, anaconda_spoke_selector, GTK_TYPE_EVENT_BOX) static void anaconda_spoke_selector_get_property(GObject *object, guint prop_id, GValue *value, GParamSpec *pspec); static void anaconda_spoke_selector_set_property(GObject *object, guint prop_id, const GValue *value, GParamSpec *pspec); +static void anaconda_spoke_selector_realize(GtkWidget *widget, gpointer user_data); static gboolean anaconda_spoke_selector_focus_changed(GtkWidget *widget, GdkEventFocus *event, gpointer user_data); @@ -156,6 +158,11 @@ static void anaconda_spoke_selector_init(AnacondaSpokeSelector *spoke) { g_signal_connect(spoke, "focus-in-event", G_CALLBACK(anaconda_spoke_selector_focus_changed), NULL); g_signal_connect(spoke, "focus-out-event", G_CALLBACK(anaconda_spoke_selector_focus_changed), NULL); + /* Set "hand" cursor shape when over the selector */ + spoke->priv->cursor = gdk_cursor_new(GDK_HAND2); + g_signal_connect(spoke, "realize", G_CALLBACK(anaconda_spoke_selector_realize), NULL); + /* TODO free the cursor when not needed */ + /* Set property defaults. */ spoke->priv->is_incomplete = FALSE; @@ -284,3 +291,9 @@ static gboolean anaconda_spoke_selector_focus_changed(GtkWidget *widget, GdkEven gtk_widget_set_state_flags(widget, new_state, TRUE); return FALSE; } + +static void anaconda_spoke_selector_realize(GtkWidget *widget, gpointer user_data) { + AnacondaSpokeSelector *spoke_selector = ANACONDA_SPOKE_SELECTOR(widget); + + gdk_window_set_cursor(gtk_widget_get_window(widget), spoke_selector->priv->cursor); +} -- 1.7.10.1

2 1

f17-branch is in the can
by Brian C. Lane 29 May '12

29 May '12

They declared RC4 as gold today, so the f17-branch is now finished. Onward to F18 with newui! Thanks to everyone who helped with this release, especially Will Woods for killing off loader and being so responsive with bugfixes as we sorted things out. -- Brian C. Lane | Anaconda Team | IRC: bcl #anaconda | Port Orchard, WA (PST8PDT)

2 1

[lorax] Remove hfsplus-tools dependency (#818913)
by Martin Gracik 29 May '12

29 May '12

Mac boot images are optional. Don't require hfsplus-tools by default, but warn the user that he needs to install them if he wants to create mac boot images. --- lorax.spec | 1 - src/pylorax/__init__.py | 7 +++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/lorax.spec b/lorax.spec index e136430..fd39bec 100644 --- a/lorax.spec +++ b/lorax.spec @@ -23,7 +23,6 @@ Requires: isomd5sum Requires: glibc Requires: util-linux Requires: dosfstools -Requires: hfsplus-tools Requires: genisoimage Requires: parted Requires: gzip diff --git a/src/pylorax/__init__.py b/src/pylorax/__init__.py index 44598d5..147fdbd 100644 --- a/src/pylorax/__init__.py +++ b/src/pylorax/__init__.py @@ -135,6 +135,13 @@ class Lorax(BaseLoraxClass): assert self._configured + if domacboot: + try: + subprocess.check_call("rpm -q hfsplus-tools") + except subprocess.CalledProcessError: + logger.critical("you need to install hfsplus-tools to create mac images") + sys.exit(1) + # set up work directory self.workdir = workdir or tempfile.mkdtemp(prefix="pylorax.work.") if not os.path.isdir(self.workdir): -- 1.7.9.5

1 0

[PATCH rhel7-alpha2-branch] Do not --force systemctl reboot (#818581)
by Radek Vykydal 29 May '12

29 May '12

Forced reboot just kills processes without shutting units/services down. In case of NetworkManager.service terminating of NM is racing with that of dhclient. If dhclient dies first NM brings down respective connection which - if used by NFS mounted in dracut (e.g. for repo=nfs: repo=nfsiso: installs) - causes hang when systemd umounts filesystems. Resolves: rhbz#818581 This patch obsoletes my first take on the bz: https://www.redhat.com/archives/anaconda-devel-list/2012-May/msg00271.html that didn't fix the issue completely. The hang kept occuring in some cases (repo=nfsiso: with kvm direct kernel boot on f17). --- anaconda | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/anaconda b/anaconda index 939262d..8693588 100755 --- a/anaconda +++ b/anaconda @@ -71,13 +71,13 @@ def exitHandler(anaconda, exitCode=None): dracut_eject(drive.path) if anaconda.ksdata.reboot.action == KS_SHUTDOWN: - os.system("systemctl --force --no-wall poweroff") + os.system("systemctl --no-wall poweroff") elif anaconda.ksdata.reboot.action == KS_WAIT: - os.system("systemctl --force --no-wall halt") + os.system("systemctl --no-wall halt") else: # reboot action is KS_REBOOT or None - os.system("systemctl --force --no-wall reboot") + os.system("systemctl --no-wall reboot") elif not flags.imageInstall: - os.system("systemctl --force --no-wall reboot") + os.system("systemctl --no-wall reboot") def startMetacityWM(): childpid = os.fork() -- 1.7.4

2 1

volunteer to anaconda
by flash yan 27 May '12

27 May '12

Hi all, I am a fedora linux system developer.I have done more about fedora and anaconda. SO ,I want to contribute to anaconda.It can improve my python skills. Thanks. Brad Yan

1 0

Kickstart packages --default parsing query
by Amit Saha 26 May '12

26 May '12

Hello: I posted this to Kickstart and Devel lists. No response yet. Would anyone here have any idea? I am trying to add a repository to an existing KS file. This is the script [2]. The script is doing its job for KS files which do not have a '--default' switch added to the %packages section. However, for a KS file which has a %packages --default section added, the parser does not read beyond it and hence creates a KS file which do not have any of the following packages specified in the KS file. To quickly verify the problem, I have added two KS files (basically the fedora-install-fedora.ks used to create Fedora installation DVDs) with one having the --default option, and the other not having it. I have commented out the actual repo addition code so that the problem can be reproduced easily. For example, with the default.ks file: $python ks_addrepo.py #version=DEVEL repo --name="fedora" --mirrorlist=http://mirrors.fedoraproject.org/mirrorlist?repo=fedora-$releasever&arch=$basearch --excludepkgs="kernel*debug*,kernel-kdump*,syslog-ng*,java-1.5.0-gcj-devel,astronomy-bookmarks,generic*,java-1.5.0-gcj-javadoc,btanks*,GConf2-dbus*,bluez-gnome" repo --name="fedora-source" --mirrorlist=http://mirrors.fedoraproject.org/mirrorlist?repo=fedora-source-$releasever&arch=$basearch --excludepkgs="kernel*debug*,kernel-kdump*,syslog-ng*,java-1.5.0-gcj-devel,astronomy-bookmarks,generic*,java-1.5.0-gcj-javadoc,btanks*,GConf2-dbus*,bluez-gnome" # Installation logging level logging --level=info %packages --default %end And with the nodefault.ks file: $python ks_addrepo.py #version=DEVEL repo --name="fedora" --mirrorlist=http://mirrors.fedoraproject.org/mirrorlist?repo=fedora-$releasever&arch=$basearch --excludepkgs="kernel*debug*,kernel-kdump*,syslog-ng*,java-1.5.0-gcj-devel,astronomy-bookmarks,generic*,java-1.5.0-gcj-javadoc,btanks*,GConf2-dbus*,bluez-gnome" repo --name="fedora-source" --mirrorlist=http://mirrors.fedoraproject.org/mirrorlist?repo=fedora-source-$releasever&arch=$basearch --excludepkgs="kernel*debug*,kernel-kdump*,syslog-ng*,java-1.5.0-gcj-devel,astronomy-bookmarks,generic*,java-1.5.0-gcj-javadoc,btanks*,GConf2-dbus*,bluez-gnome" # Installation logging level logging --level=info %packages @afrikaans-support @albanian-support @arabic-support @armenian-support @assamese-support @basque-support @belarusian-support @bengali-support @bhutanese-support @bosnian-support @brazilian-support @breton-support @british-support @bulgarian-support @burmese-support .. .. [1] https://www.redhat.com/archives/kickstart-list/2012-May/msg00002.html [2] https://github.com/amitsaha/gsoc2012_fbs/blob/master/scripts/ks_addrepo.py Have I hit a bug or is it something which is known can be worked around? Thanks a lot in advance. Best, Amit -- http://echorand.me

1 1

livemedia-creator help
by Amit Saha 26 May '12

26 May '12

Hello: I am trying to use 'livemedia-creator' to build live images using the command line: # livemedia-creator --make-iso --ks fedora-livemedia.ks --iso /tmp/netinst.iso Which ends with: WARNING: gnome-keyring:: couldn't connect to: /run/user/gene/keyring-60f8kj/pkcs11: No such file or directory WARNING KVM acceleration not available, using 'qemu' Starting install... Retrieving file vmlinuz... | 8.8 MB 00:00 ... Retrieving file initrd.img... | 46 MB 00:02 ... Creating storage file diskdyhj81.img | 5.0 GB 00:00 ERROR internal error process exited while connecting to monitor: char device redirected to /dev/pts/4 Could not allocate dynamic translator buffer Domain installation does not appear to have been successful. If it was, you can restart your domain by running: virsh --connect qemu:///system start LiveOS-27e73b13-3bfd-4e22-bb30-a67f272f063c otherwise, please restart your installation. Traceback (most recent call last): File "/usr/sbin/livemedia-creator", line 708, in <module> virtio_port = log_monitor.port ) File "/usr/sbin/livemedia-creator", line 355, in __init__ raise Exception("Problem starting virtual install") I am trying to do this in a VM, and i have tried to increase the RAM by --ram 1200 (total RAM given to the VM is 1.7G), but it still doesn't proceed. Has anyone else succeeded using it in a VM or, should i go for bare metal? Many thanks. -Amit -- http://echorand.me

3 4

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Anaconda-devel May 2012