anaconda-patches March 2015

anaconda-patches@lists.fedorahosted.org

13 participants
166 discussions

[PATCH rhel6-branch] Increment MD container child counter even if its volumes are ignored (#1120640)
by Artur Paszkiewicz 11 Jan '16

11 Jan '16

Volumes which are not selected in Specialized Storage Devices are ignored in addUdevDevice. This causes that their parent container won't have its child counter incremented, which can later cause an incorrect unusedRaidMembersWarning. Resolves: rhbz#1120640 Signed-off-by: Artur Paszkiewicz <artur.paszkiewicz(a)intel.com> --- storage/devicetree.py | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/storage/devicetree.py b/storage/devicetree.py index 4045cdf..b9ea399 100644 --- a/storage/devicetree.py +++ b/storage/devicetree.py @@ -1328,6 +1328,13 @@ class DeviceTree(object): # slice off the "/dev/" part, lvm filter cares only about the rest partitions_paths = [p[5:] for p in partitions_paths] map(lvm.lvm_cc_addFilterRejectRegexp, partitions_paths) + + if udev_device_get_md_container(info): + parentName = devicePathToName(udev_device_get_md_container(info)) + container = self.getDeviceByName(parentName) + if container: + container.addChild() + return log.debug("scanning %s (%s)..." % (name, sysfs_path)) -- 1.8.4.5

4 14

Updated: [rhinstaller/anaconda/pulls/6] network: Overwrite resolv.conf generated by systemd
by dashea 22 Oct '15

22 Oct '15

[Note: against F22-branch, but installing rawhide content, that's how I hit the bug] systemd changed to generate a symlink resolv.conf -> /run/networkd/resolv.conf which doesn't exist at installation time. (What we should do at some point is tell NetworkManager to serialize its state into the installation root and get out of the business of writing these files directly) https://bugzilla.redhat.com/show_bug.cgi?id=1116651 -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/6

1 5

Updated: [rhinstaller/anaconda/pulls/49 master] Fix issues with fetching pull requests.
by dashea 22 Oct '15

22 Oct '15

The "base" SHA is not actually where the pull request branched from, though that would be pretty handy. Instead of trying to find a point to branch from, just fetch the PR branch into a new local branch and rebase that onto the destination branch. -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/49

1 6

Updated: [rhinstaller/anaconda/pulls/33 f22-branch] master/f22-branch - Set the default fs label when reformatting (#1141981)
by dashea 21 Oct '15

21 Oct '15

If the new format has a default label, and a label hasn't been set by the user, use the default. This allows the ESP to be reformatted and used without removing it and recreating it. -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/33

1 18

New: [rhinstaller/blivet/pulls/59 master] Implement the __dir__ method of the LazyImportObject class
by dashea 10 Jul '15

10 Jul '15

This makes the 'TAB-TAB' completion in ipython and dir() calls work as if the lazy-imported object was imported in a normal way. -- To view this pull request on github, visit https://github.com/rhinstaller/blivet/pull/59

1 4

Updated: [rhinstaller/anaconda/pulls/32 master] Check if gnome-icon-theme is installed
by dashea 25 Jun '15

25 Jun '15

Just minimize the false negatives when this is installed. -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/32

1 5

New: [rhinstaller/blivet/pulls/44 master] Set format options for mkfs
by dashea 18 Jun '15

18 Jun '15

Currently format options can be specified only for doFormat function. This patch provides options to set the format options. Usage ex:- format = blivet.formats.getFormat("xfs", device=lv.path) format.formatOptions = ["-f", "-K", "-i", "size=256", "-d", ...] blivetEnv.formatDevice(lv, format) blivetEnv.doIt() bug-url: https://bugzilla.redhat.com/show_bug.cgi?id=1180221 Signed-off-by: Timothy Asir Jeyasingh <tjeyasin(a)redhat.com> -- To view this pull request on github, visit https://github.com/rhinstaller/blivet/pull/44

1 7

New: [rhinstaller/anaconda/pulls/17] Backport pyanaconda_tests/keyboard_test.py updates from master
by dashea 03 Jun '15

03 Jun '15

Seems OK (nosetests passes) but there's one thing on master that worries me: commit 45a05d0a16014e9a46a625bc2f418888d3bdeb7d Author: Vratislav Podzimek Date: Thu Jul 17 16:12:14 2014 +0200 Prevent crashes due to accessing X server from multiple threads I don't know how that applies to rhel7-branch. Please comment. -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/17

1 9

New: [rhinstaller/anaconda/pulls/19] Backport grub_raid_test.py from master
by dashea 03 Jun '15

03 Jun '15

A few notes here: As is this test fails in grub_mbr_raid1_test(). See the lines: ``` expected_targets = set([(self.sda, self.boot_md), (self.sdb, self.boot_md)]) # expected_targets = set([(self.sda, self.sda2), (self.sdb, self.sdb2)]) ``` If the second one is used instead of the first one then the result is PASS. This is due to different support for GRUB stage2 between master and rhel7-branch. On RHEL7: ``` for stage2dev in self.stage2_device.parents: # if target disk contains any of /boot array's member # partitions, set up stage1 on each member's disk # and stage2 on each member partition stage1dev = stage2dev.disk targets.append((stage1dev, stage2dev)) ``` While on master: ``` for stage2dev in stage2_parents: # if target disk contains any of /boot array's member # partitions, set up stage1 on each member's disk stage1dev = stage2dev.disk targets.append((stage1dev, self.stage2_device)) ``` Notice the self.stage2_device vs. stage2dev. The first one is the md1 partition while the second one is either sda2 or sdb2 - the mdraid member partitions. I don't know which one is the expected behavior for RHEL 7. -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/19

1 7

New: [rhinstaller/anaconda/pulls/12] Remove W0123 which is not defined in older pylint and causes traceback
by dashea 03 Jun '15

03 Jun '15

Running the test suite on RHEL7 yields: ``` Traceback (most recent call last): File "/usr/bin/pylint", line 9, in <module> load_entry_point('pylint==1.0.0', 'console_scripts', 'pylint')() File "/usr/lib/python2.7/site-packages/pylint/__init__.py", line 21, in run_pylint Run(sys.argv[1:]) File "/usr/lib/python2.7/site-packages/pylint/lint.py", line 990, in __init__ args = linter.load_command_line_configuration(args) File "/usr/lib/python2.7/site-packages/logilab/common/configuration.py", line 685, in load_command_line_configuration (options, args) = self.cmdline_parser.parse_args(args=args) File "/usr/lib64/python2.7/optparse.py", line 1399, in parse_args stop = self._process_args(largs, rargs, values) File "/usr/lib64/python2.7/optparse.py", line 1439, in _process_args self._process_long_opt(rargs, values) File "/usr/lib64/python2.7/optparse.py", line 1514, in _process_long_opt option.process(opt, value, values, self) File "/usr/lib/python2.7/site-packages/logilab/common/optik_ext.py", line 237, in process self.action, self.dest, opt, value, values, parser) File "/usr/lib64/python2.7/optparse.py", line 808, in take_action self.callback(self, opt, value, parser, *args, **kwargs) File "/usr/lib/python2.7/site-packages/logilab/common/configuration.py", line 540, in cb_set_provider_option self.global_set_option(opt, value) File "/usr/lib/python2.7/site-packages/logilab/common/configuration.py", line 544, in global_set_option self._all_options[opt].set_option(opt, value) File "/usr/lib/python2.7/site-packages/pylint/lint.py", line 383, in set_option meth(_id) File "/usr/lib/python2.7/site-packages/pylint/utils.py", line 251, in disable msg = self.check_message_id(msgid) File "/usr/lib/python2.7/site-packages/pylint/utils.py", line 315, in check_message_id raise UnknownMessage('No such message id %s' % msgid) pylint.utils.UnknownMessage: No such message id W0123 ``` With this small fix it displays correct results from pylint: ``` FAIL: pylint/runpylint ====================== pylint reports the following issues: ************* Module ../pyanaconda/ui/gui/spokes/lib/resize.py E0611: 25,0: : No name 'Gdk' in module 'gi.repository' E0611: 25,0: : No name 'Gtk' in module 'gi.repository' ************* Module ../pyanaconda/anaconda.py F0401:161,8: Anaconda.dumpState: Unable to import 'meh' F0401:162,8: Anaconda.dumpState: Unable to import 'meh.dump' ``` -- To view this pull request on github, visit https://github.com/rhinstaller/anaconda/pull/12

1 9

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

anaconda-patches March 2015