June 2021 - FreeIPA-devel - Fedora mailing-lists

[freeipa PR#5827][opened] ipatests: disable test_nfs.py::TestNFS in nightly runs on Fedora 33
by wladich 10 Jun '21

10 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5827 Author: wladich Title: #5827: ipatests: disable test_nfs.py::TestNFS in nightly runs on Fedora 33 Action: opened PR body: """ Also disable in Fedora 33 as it also has the faulty version of sssd which produces multi-gigabyte log file Related to https://pagure.io/freeipa/issue/8877 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5827/head:pr5827 git checkout pr5827

2 1

[freeipa PR#5828][opened] [Backport][ipa-4-9] ipatests: disable test_nfs.py::TestNFS in nightly runs on Fedora 33
by wladich 10 Jun '21

10 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5828 Author: wladich Title: #5828: [Backport][ipa-4-9] ipatests: disable test_nfs.py::TestNFS in nightly runs on Fedora 33 Action: opened PR body: """ This is a manual backport of #5827 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5828/head:pr5828 git checkout pr5828

2 1

Framework execute callback feedback
by Rob Crittenden 10 Jun '21

10 Jun '21

Currently the framework has four types of methods: - pre_callback: called before any real work is done. Usually used to tune up incoming options - execute: does the brute work. For most CRUD plugins this is all done in baseldap.py in the LDAP* classes - post_callback: supposed to be data cleanup for presentation to the caller but in some cases actual work is being done. user-add calls ldap.add_entry_to_group for example. - exc_callback: special case handling for exceptions If you have "work" to be done that would normally be done in execute there is no way to do it other than to abuse post_callback to get it done. Overriding execute is only an option if you want to completely replace the call, otherwise order execution is going to be strange. Say for example in user_add you want to do work after the execute() is done so you'd do something like: class user_add() def execute(stuff): result = super(user_add, self).execute(stuff) # some extra work return result The order of execution is: user_add:pre_callback user_add:execute() LDAPCreate:execute() user_add:post_callback() user_add:execute() This is because the LDAP* baseldap methods handle the pre/post/exc callbacks directly. So in order to be able to insert more "work" in execute itself we'd need another callback type. Why do I want this? One feature of post_callback is that for add/mod methods ldap.get_entry() is called so we can prepare the final results to be returned to the user for display. There are cases where the user doesn't need or case to see that result. We could save an least one LDAP query by returning only errors in this case. The use case is mostly around bulk operations where the user doesn't care about seeing the UID of all users added, for example. To do this I think we can skip post_callback altogether, assuming it does no real "work". In the case of user_add/mod skipping post_callback would save three queries: fetch the user, has_keytab and has_password. But before re-architecting this I'd like to get some opinions on it. I can do a full-blown design if desired. rob

2 4

[freeipa PR#5824][opened] [Backport][ipa-4-9] freeipa.spec: bump 389-ds version
by rcritten 10 Jun '21

10 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5824 Author: rcritten Title: #5824: [Backport][ipa-4-9] freeipa.spec: bump 389-ds version Action: opened PR body: """ This PR was opened automatically because PR #5819 was pushed to master and backport to ipa-4-9 is required. """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5824/head:pr5824 git checkout pr5824

2 1

[freeipa PR#5819][opened] freeipa.spec: bump 389-ds version
by flo-renaud 09 Jun '21

09 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5819 Author: flo-renaud Title: #5819: freeipa.spec: bump 389-ds version Action: opened PR body: """ IPA depends on the 389-ds version with the fix for https://github.com/389ds/389-ds-base/issues/4700 Regression in winsync replication agreement The same 389-ds version also fixes https://github.com/389ds/389-ds-base/issues/4670 389ds coredump in IPA nightly test test_caless.py::TestReplicaInstall::test_wildcard_http Fixes: https://pagure.io/freeipa/issue/8691 Fixes: https://pagure.io/freeipa/issue/8756 Signed-off-by: Florence Blanc-Renaud <flo(a)redhat.com> """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5819/head:pr5819 git checkout pr5819

2 1

[freeipa PR#5820][opened] [Backport][ipa-4-9] ipatests: temporary disable execution of test_nfs.py::TestNFS in nightly runs
by wladich 09 Jun '21

09 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5820 Author: wladich Title: #5820: [Backport][ipa-4-9] ipatests: temporary disable execution of test_nfs.py::TestNFS in nightly runs Action: opened PR body: """ This is a manual backport of #5818 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5820/head:pr5820 git checkout pr5820

1 1

[freeipa PR#5818][opened] ipatests: temporary disable execution of test_nfs.py::TestNFS in nightly runs
by wladich 08 Jun '21

08 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5818 Author: wladich Title: #5818: ipatests: temporary disable execution of test_nfs.py::TestNFS in nightly runs Action: opened PR body: """ During test run on Fedora 34 and 35 sssd produces multi-gigabyte log file which causes test runners to run out of disk space. Related to https://pagure.io/freeipa/issue/8877 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5818/head:pr5818 git checkout pr5818

2 1

[freeipa PR#5817][opened] [Backport][ipa-4-9] ipatests: delete the replica before uninstallation
by flo-renaud 08 Jun '21

08 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5817 Author: flo-renaud Title: #5817: [Backport][ipa-4-9] ipatests: delete the replica before uninstallation Action: opened PR body: """ This PR was opened automatically because PR #5814 was pushed to master and backport to ipa-4-9 is required. """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5817/head:pr5817 git checkout pr5817

1 1

[freeipa PR#5814][opened] ipatests: delete the replica before uninstallation
by flo-renaud 08 Jun '21

08 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5814 Author: flo-renaud Title: #5814: ipatests: delete the replica before uninstallation Action: opened PR body: """ The test test_installation.py::TestInstallWithCA1::test_install_with_bad_ldap_conf is uninstalling a replica by calling ipa-server-install --uninstall directly, instead of deleting the replica first. Use tasks.uninstall_replica instead of tasks.uninstall_master to perform a proper uninstallation. Fixes: https://pagure.io/freeipa/issue/8876 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5814/head:pr5814 git checkout pr5814

1 1

[freeipa PR#5816][opened] [Backport][ipa-4-9] ipatests: set selinux context for fips mode
by rcritten 08 Jun '21

08 Jun '21

URL: https://github.com/freeipa/freeipa/pull/5816 Author: rcritten Title: #5816: [Backport][ipa-4-9] ipatests: set selinux context for fips mode Action: opened PR body: """ This PR was opened automatically because PR #5812 was pushed to master and backport to ipa-4-9 is required. """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/5816/head:pr5816 git checkout pr5816

2 1

2024

2023

2022

2021

2020

2019

2018

2017

FreeIPA-devel June 2021