Re: [SSSD] [PATCH] Typo in debug message (SSSd -> SSSD).

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 10:54:41AM -0400, Stephen Gallagher wrote: > On Thu, 2012-08-23 at 16:48 +0200, Michal Židek wrote: > > One line (or better, one character) change in debug message. > > > > https://fedorahosted.org/sssd/ticket/1434 > > > > Patch is attached. > > Ack Pushed to master. ___

Re: [SSSD] [PATCH] (2nd wave) Removing bad examples of usage of, sysdb_transaction_start/commit/end functions and making it, more consistent (all files except of src/db/sysdb_*.c)

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 06:12:27PM +0200, Jakub Hrozek wrote: > > New patch is attached. > > > > Thanks > > Michal > > > > Ack I amended the commit message (it was too long) and pushed the patch to master. ___ sssd-devel mailing list sssd-devel@lists.

Re: [SSSD] [PATCH] Fix: IPv6 address with square brackets doesn't work.

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 06:24:13PM +0200, Jakub Hrozek wrote: > On Thu, Aug 23, 2012 at 11:50:24AM +0200, Michal Židek wrote: > > On 08/22/2012 03:50 PM, Stephen Gallagher wrote: > > >On Wed, 2012-08-22 at 15:43 +0200, Michal Židek wrote: > > >>https://fedorahosted.org/sssd/ticket/1365 > > >> > > >

Re: [SSSD] [PATCH] Fix: IPv6 address with square brackets doesn't work.

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 11:50:24AM +0200, Michal Židek wrote: > On 08/22/2012 03:50 PM, Stephen Gallagher wrote: > >On Wed, 2012-08-22 at 15:43 +0200, Michal Židek wrote: > >>https://fedorahosted.org/sssd/ticket/1365 > >> > >>Note: It affected not only krb5, but other providers as well. > >> > >>Pa

Re: [SSSD] [PATCH] (2nd wave) Removing bad examples of usage of, sysdb_transaction_start/commit/end functions and making it, more consistent (all files except of src/db/sysdb_*.c)

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 12:48:41PM +0200, Michal Židek wrote: > On 08/22/2012 04:48 PM, Jakub Hrozek wrote: > >On Tue, Aug 21, 2012 at 01:40:27PM +0200, Michal Židek wrote: > >>On 08/21/2012 10:39 AM, Pavel Březina wrote: > >>>On 08/20/2012 05:03 PM, Michal Židek wrote: > This is the second and

Re: [SSSD] [PATCH] Typo in debug message (SSSd -> SSSD).

2012-08-23 Thread Stephen Gallagher
On Thu, 2012-08-23 at 16:48 +0200, Michal Židek wrote: > One line (or better, one character) change in debug message. > > https://fedorahosted.org/sssd/ticket/1434 > > Patch is attached. Ack signature.asc Description: This is a digitally signed message part

[SSSD] [PATCH] Typo in debug message (SSSd -> SSSD).

2012-08-23 Thread Michal Židek
One line (or better, one character) change in debug message. https://fedorahosted.org/sssd/ticket/1434 Patch is attached. Thanks Michal >From f73b8d18cdd696b4241d0d59dca739332b9c8d30 Mon Sep 17 00:00:00 2001 From: Michal Zidek Date: Thu, 23 Aug 2012 13:58:56 +0200 Subject: [PATCH] Typo in debu

Re: [SSSD] [PATCH] delete entries with entryUSN > lastUSN when lastUSN changes

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 01:08:21PM +0200, Jakub Hrozek wrote: > On Thu, Aug 23, 2012 at 10:12:57AM +0200, Pavel Březina wrote: > > On 08/22/2012 05:37 PM, Jakub Hrozek wrote: > > >On Thu, Aug 16, 2012 at 05:42:45PM +0200, Pavel Březina wrote: > > >>On 08/07/2012 04:04 PM, Simo Sorce wrote: > > >>>O

Re: [SSSD] [PATCH] AD context was set to null due to type mismatch

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 01:54:00PM +0200, Ondrej Kos wrote: > fixed, patch is attached > > O. > Ack and pushed to master. ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel

Re: [SSSD] [PATCH] Consolidation of functions that make realm upper-case

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 01:15:11PM +0200, Jakub Hrozek wrote: > On Thu, Aug 23, 2012 at 08:25:43AM +0200, Ondrej Kos wrote: > > hmm, those warnings didn't appear, flags probably got somehow mixed up. > > > > fixed patch is attached > > > > O. > > > > Ack Pushed to master. _

Re: [SSSD] [PATCH] AD context was set to null due to type mismatch

2012-08-23 Thread Ondrej Kos
fixed, patch is attached O. On 08/23/2012 01:14 PM, Jakub Hrozek wrote: On Thu, Aug 23, 2012 at 11:23:52AM +0200, Ondrej Kos wrote: https://fedorahosted.org/sssd/ticket/1416 Fixed by wrap-up function re-used from the IPA provider Ondrej The code is fine, but can you change the commit messa

Re: [SSSD] message "Could not reconnect to LDAP provider"

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 11:24:55AM +0200, Franky Van Liedekerke wrote: > On 2012-08-23 11:18, Jakub Hrozek wrote: > >On Thu, Aug 23, 2012 at 11:07:31AM +0200, Franky Van Liedekerke > >wrote: > >>Hi, > >> > >>I just had a weird situation: one of my servers suddenly no longer > >>allowed me to log in

Re: [SSSD] [PATCH] Consolidation of functions that make realm upper-case

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 08:25:43AM +0200, Ondrej Kos wrote: > hmm, those warnings didn't appear, flags probably got somehow mixed up. > > fixed patch is attached > > O. > Ack ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.

Re: [SSSD] [PATCH] AD provider was crashing with inappropriate keytab

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 11:23:52AM +0200, Ondrej Kos wrote: > https://fedorahosted.org/sssd/ticket/1416 > > Fixed by wrap-up function re-used from the IPA provider > > Ondrej The code is fine, but can you change the commit message now that it turned out that the problem wasn't caused by the keyt

Re: [SSSD] [PATCH] delete entries with entryUSN > lastUSN when lastUSN changes

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 10:12:57AM +0200, Pavel Březina wrote: > On 08/22/2012 05:37 PM, Jakub Hrozek wrote: > >On Thu, Aug 16, 2012 at 05:42:45PM +0200, Pavel Březina wrote: > >>On 08/07/2012 04:04 PM, Simo Sorce wrote: > >>>On Tue, 2012-08-07 at 15:23 +0200, Pavel Březina wrote: > https://fed

Re: [SSSD] [PATCH] Release mutex if a thread gets cancelled

2012-08-23 Thread Jakub Hrozek
On Wed, Aug 22, 2012 at 07:00:02PM +0200, Jakub Hrozek wrote: > On Tue, Aug 21, 2012 at 12:02:43PM +0200, Jakub Hrozek wrote: > > I haven't tested F-16 yet, but I'll spin up a VM. > > The patch worked fine for me on a 64bit F-16 machine..feel free to ping > me off-list for access details. Simo re

Re: [SSSD] [PATCH] (2nd wave) Removing bad examples of usage of, sysdb_transaction_start/commit/end functions and making it, more consistent (all files except of src/db/sysdb_*.c)

2012-08-23 Thread Michal Židek
On 08/22/2012 04:48 PM, Jakub Hrozek wrote: On Tue, Aug 21, 2012 at 01:40:27PM +0200, Michal Židek wrote: On 08/21/2012 10:39 AM, Pavel Březina wrote: On 08/20/2012 05:03 PM, Michal Židek wrote: This is the second and final patch to make the usage of sysdb_transaction_* functions more consiste

[SSSD] realmd: Faster discovery, generic kerberos discovery

2012-08-23 Thread Stef Walter
Some more patches for realmd. The first patch is one that makes the discovery of kerberos realms much faster: https://bugs.freedesktop.org/show_bug.cgi?id=53956 In particular discovery of IPA is harder to do in a fixed amount of time. We actually try to connect to the server to retrieve its cert

Re: [SSSD] [PATCH] Fix: IPv6 address with square brackets doesn't work.

2012-08-23 Thread Michal Židek
On 08/22/2012 03:50 PM, Stephen Gallagher wrote: On Wed, 2012-08-22 at 15:43 +0200, Michal Židek wrote: https://fedorahosted.org/sssd/ticket/1365 Note: It affected not only krb5, but other providers as well. Patch is attached. Nack. Please do not copy-paste everywhere. This should be a functi

Re: [SSSD] message "Could not reconnect to LDAP provider"

2012-08-23 Thread Franky Van Liedekerke
On 2012-08-23 11:18, Jakub Hrozek wrote: On Thu, Aug 23, 2012 at 11:07:31AM +0200, Franky Van Liedekerke wrote: Hi, I just had a weird situation: one of my servers suddenly no longer allowed me to log in (pam auth via sssd). Looking in the log for sssd, I had this message once every minute: [s

[SSSD] [PATCH] AD provider was crashing with inappropriate keytab

2012-08-23 Thread Ondrej Kos
https://fedorahosted.org/sssd/ticket/1416 Fixed by wrap-up function re-used from the IPA provider Ondrej -- Ondrej Kos Associate Software Engineer Identity Management Red Hat Czech cell: +420-736-417-909 phone: +420-532-294-558 ext.: 82-62558 irc: okos @ #brno >From 92b22b7544b85ee08a880183

Re: [SSSD] message "Could not reconnect to LDAP provider"

2012-08-23 Thread Jakub Hrozek
On Thu, Aug 23, 2012 at 11:07:31AM +0200, Franky Van Liedekerke wrote: > Hi, > > I just had a weird situation: one of my servers suddenly no longer > allowed me to log in (pam auth via sssd). > Looking in the log for sssd, I had this message once every minute: > > [sssd[pam]] [pam_dp_reconnect_in

[SSSD] message "Could not reconnect to LDAP provider"

2012-08-23 Thread Franky Van Liedekerke
Hi, I just had a weird situation: one of my servers suddenly no longer allowed me to log in (pam auth via sssd). Looking in the log for sssd, I had this message once every minute: [sssd[pam]] [pam_dp_reconnect_init] (0): Could not reconnect to LDAP provider. Also, "getent passwd" no longer

Re: [SSSD] [PATCH] delete entries with entryUSN > lastUSN when lastUSN changes

2012-08-23 Thread Pavel Březina
On 08/22/2012 05:37 PM, Jakub Hrozek wrote: On Thu, Aug 16, 2012 at 05:42:45PM +0200, Pavel Březina wrote: On 08/07/2012 04:04 PM, Simo Sorce wrote: On Tue, 2012-08-07 at 15:23 +0200, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/734 Patches 1 and 2 adds support sysdb functions. P