[freeipa PR#1433][opened] ipa-run-tests: replace chdir with plugin
by tiran
URL: https://github.com/freeipa/freeipa/pull/1433
Author: tiran
Title: #1433: ipa-run-tests: replace chdir with plugin
Action: opened
PR body:
"""
The ipa-run-tests command used os.chdir() to change into the ipatests/
directory. The approach works for simple cases but breaks some pytest
features. For example it makes it impossible to selects tests by their
fully qualified test name.
Further more, coverage statistics break because path and module names
get messed up by chdir.
A name plugin takes care of adjusting paths relative to ipatests and to
add ipatests as base. It's now possible to run tests with qualified test
names, e.g.
ipa-run-tests ipatests/test_ipalib/test_base.py::test_ReadOnly::test_lock
Signed-off-by: Christian Heimes <cheimes(a)redhat.com>
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1433/head:pr1433
git checkout pr1433
6 years, 3 months
[freeipa PR#1282][opened] Log contents of config files when updating/writing them
by rcritten
URL: https://github.com/freeipa/freeipa/pull/1282
Author: rcritten
Title: #1282: Log contents of config files when updating/writing them
Action: opened
PR body:
"""
This was originally going to be a PR to log the contents of default.conf for debugging purposes, mostly for replicas where in DL1 it gets written several times. Given that other config files like nsswitch.conf also use the same functions we get even more output as an extra benefit. The file sizes are currently not that big so it doesn't bloat the logs too much IMHO.
One patch adds the logging and the other patch changes the server installer to use the configuration writing too rather than manually creating the file.
"""
To pull the PR as Git branch:
git remote add ghfreeipa https://github.com/freeipa/freeipa
git fetch ghfreeipa pull/1282/head:pr1282
git checkout pr1282
6 years, 3 months