Re: [SSSD] [PATCH v3] Add integration tests

2015-05-12 Thread Michal Židek
On 05/12/2015 09:51 PM, Michal Židek wrote: On 05/06/2015 02:51 PM, Nikolai Kondrashov wrote: Hi everyone, Here is the third version of the integration test patches. Changes from the second version include the following. * Rebased on latest master. * Moved verbs to the start of ent.py functio

Re: [SSSD] [PATCH v3] Add integration tests

2015-05-12 Thread Michal Židek
On 05/06/2015 02:51 PM, Nikolai Kondrashov wrote: Hi everyone, Here is the third version of the integration test patches. Changes from the second version include the following. * Rebased on latest master. * Moved verbs to the start of ent.py function names, so assertion function names start

Re: [SSSD] [PATCH] Amend the man page for refresh_expired_interval

2015-05-12 Thread Stephen Gallagher
On Mon, 2015-05-11 at 09:52 +0200, Jakub Hrozek wrote: > Hi, > > while triaging a performance-related issue, I realized our manpage > doesn't say also users and groups are now supported by the background > refresh. The attached patch fixes that. I'd recommend the phrasing: "The background refre

[SSSD] [PATCH] krb5: remove field run_as_user

2015-05-12 Thread Pavel Reichl
Hello, this patch was already ACKed in another thread, I decided to move it here as that thread is going to take some more time to finish. From ffe3d9f0e0e697f565fb28861df41fdd9c7bbbe2 Mon Sep 17 00:00:00 2001 From: Pavel Reichl Date: Wed, 29 Apr 2015 06:03:04 -0400 Subject: [PATCH 3/3] krb5:

Re: [SSSD] [PATCH] LDAP: warn about lockout option being deprecated

2015-05-12 Thread Pavel Reichl
On 05/11/2015 07:28 PM, Jakub Hrozek wrote: On Mon, May 11, 2015 at 02:35:35PM +0200, Pavel Reichl wrote: On 05/11/2015 12:24 PM, Jakub Hrozek wrote: On Wed, Mar 25, 2015 at 10:19:39AM +0100, Pavel Reichl wrote: Hello, attached patch deprecates lockout option in 1-12 branch. This was discu

Re: [SSSD] [PATCH] LDAP: disable the cleanup task by default

2015-05-12 Thread Jakub Hrozek
On Mon, May 11, 2015 at 03:04:43PM -0400, Stephen Gallagher wrote: > > > Code wise ACK. > > > http://sssd-ci.duckdns.org/logs/job/15/08/summary.html > > > > > > It would be good to have a blessing for man page from native > > > speaker. > > > > Stephen, do you have a minute to check if the manpa

Re: [SSSD] [PATCH] IPA: do not fail if view name lookup failed on older versions

2015-05-12 Thread Jakub Hrozek
On Tue, May 12, 2015 at 09:55:53AM +0200, Jakub Hrozek wrote: > > It comes from sdap_get_generic_op_finished(). I was thinking about > > adding special handling for LDAP_PROTOCOL_ERROR here, but I thought it > > might be too risky because it might break some other code which expects > > EIO. > >

Re: [SSSD] design discussion: Authenticate against cache in SSSD

2015-05-12 Thread Pavel Reichl
On 05/04/2015 01:34 PM, Pavel Reichl wrote: Hello, thank you all for comments. I have updated the design page. You can find it there: https://fedorahosted.org/sssd/wiki/DesignDocs/CachedAuthentication I have also pasted it here. Thanks! There was no activity in this thread for longer than a

Re: [SSSD] [PATCH] IPA: do not fail if view name lookup failed on older versions

2015-05-12 Thread Jakub Hrozek
On Mon, May 11, 2015 at 08:14:47PM +0200, Sumit Bose wrote: > On Mon, May 11, 2015 at 07:13:17PM +0200, Jakub Hrozek wrote: > > On Mon, May 11, 2015 at 12:01:52PM +0200, Sumit Bose wrote: > > > On Mon, May 11, 2015 at 12:01:12PM +0200, Sumit Bose wrote: > > > > Hi, > > > > > > and now with patch .