[SSSD] Re: [PATCH] DEBUG: Apend line feed to messages from libsemanage

2016-08-31 Thread Fabiano Fidêncio
On Thu, Sep 1, 2016 at 8:16 AM, Lukas Slebodnik wrote: > ehlo, > > @see > (Wed Aug 31 14:12:18 2016) [[sssd[selinux_child[7088 [libsemanage] > (0x0020): could not query record value(Wed Aug 31 14:12:18 2016) > [[sssd[selinux_child[7088 [get_seuser] (0x0020): Cannot query for dennis > (We

[SSSD] [PATCH] DEBUG: Apend line feed to messages from libsemanage

2016-08-31 Thread Lukas Slebodnik
ehlo, @see (Wed Aug 31 14:12:18 2016) [[sssd[selinux_child[7088 [libsemanage] (0x0020): could not query record value(Wed Aug 31 14:12:18 2016) [[sssd[selinux_child[7088 [get_seuser] (0x0020): Cannot query for dennis (Wed Aug 31 14:12:18 2016) [[sssd[selinux_child[7088 [seuser_needs_u

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (comment)

2016-08-31 Thread jhrozek
jhrozek commented on a pull request """ ACK, CI pending """ See the full comment at https://github.com/SSSD/sssd/pull/9#issuecomment-243902393 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/admin/lists/sssd

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (+ack)

2016-08-31 Thread jhrozek
celestian's pull request #9: "SYSDB: Removing of unused parametr" label *ack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/9 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/ad

[SSSD] Re: [RFC] Cleaning up the IFP responder (mainly) and socket-activatable responders

2016-08-31 Thread Jakub Hrozek
On Wed, Aug 31, 2016 at 07:40:32PM +0200, Fabiano Fidêncio wrote: > Howdy! > > Taking a look on https://fedorahosted.org/sssd/ticket/2395 seems that > there are a few ways to achieve what's proposed by Simo and I'd like > to discuss one of those before start implementing it. > > As far as I under

[SSSD] Re: [PATCH] GPO: Cat vals with same key from different GPOs

2016-08-31 Thread Stephen Gallagher
On 08/31/2016 01:24 PM, Simo Sorce wrote: > On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote: >> Hi, >> >> here is patch for ticket #3161. >> >> See more in the ticket description. >> >> I was thinking why we originally replaced >> the lists and I think it comes from confusion >> on how we han

[SSSD] [RFC] Cleaning up the IFP responder (mainly) and socket-activatable responders

2016-08-31 Thread Fabiano Fidêncio
Howdy! Taking a look on https://fedorahosted.org/sssd/ticket/2395 seems that there are a few ways to achieve what's proposed by Simo and I'd like to discuss one of those before start implementing it. As far as I understand the bug, SSSD cannot be run as a non-root user because of IFP responder, w

[SSSD] Re: [PATCH] GPO: Cat vals with same key from different GPOs

2016-08-31 Thread Lukas Slebodnik
On (31/08/16 13:24), Simo Sorce wrote: >On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote: >> Hi, >> >> here is patch for ticket #3161. >> >> See more in the ticket description. >> >> I was thinking why we originally replaced >> the lists and I think it comes from confusion >> on how we hand

[SSSD] Re: [PATCH] GPO: Cat vals with same key from different GPOs

2016-08-31 Thread Simo Sorce
On Wed, 2016-08-31 at 17:41 +0200, Michal Židek wrote: > Hi, > > here is patch for ticket #3161. > > See more in the ticket description. > > I was thinking why we originally replaced > the lists and I think it comes from confusion > on how we handle the same keys in single > GPO ini file, howeve

[SSSD] [PATCH] GPO: Cat vals with same key from different GPOs

2016-08-31 Thread Michal Židek
Hi, here is patch for ticket #3161. See more in the ticket description. I was thinking why we originally replaced the lists and I think it comes from confusion on how we handle the same keys in single GPO ini file, however that is handled by libini not by SSSD. Michal >From f1a54ec427fe109c8c1

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread lslebodn
lslebodn commented on a pull request """ On (31/08/16 01:47), sumit-bose wrote: >On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote: >> On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: >> > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: >> > > On Tue, Aug 30, 20

[SSSD] Re: [PATCH] sss_override: improve --debug description

2016-08-31 Thread Justin Stephenson
On 08/31/2016 06:59 AM, Pavel Březina wrote: On 08/30/2016 08:52 PM, Justin Stephenson wrote: On 08/05/2016 11:45 AM, Lukas Slebodnik wrote: On (15/03/16 12:40), Pavel Březina wrote: On 12/09/2015 01:16 PM, Jakub Hrozek wrote: On Wed, Dec 09, 2015 at 01:07:10PM +0100, Pavel Březina wrote: ht

[SSSD] [sssd PR#9] SYSDB: Removing of unused parametr (opened)

2016-08-31 Thread celestian
celestian's pull request #9: "SYSDB: Removing of unused parametr" was opened PR body: """ There were unused parameter struct ldb_message *cached_group in sysdb_store_group_attrs(). This parameter was introduced by 40de79d69860ec7f04bf7795bd88b641ec42fd23 SYSDB: Check if group attributes differ be

[SSSD] Re: [PATCH] sss_override: improve --debug description

2016-08-31 Thread Pavel Březina
On 08/30/2016 08:52 PM, Justin Stephenson wrote: On 08/05/2016 11:45 AM, Lukas Slebodnik wrote: On (15/03/16 12:40), Pavel Březina wrote: On 12/09/2015 01:16 PM, Jakub Hrozek wrote: On Wed, Dec 09, 2015 at 01:07:10PM +0100, Pavel Březina wrote: https://fedorahosted.org/sssd/ticket/2813 I wan

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (+ack)

2016-08-31 Thread pbrezina
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option" label *ack* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/8 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahost

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread sumit-bose
sumit-bose commented on a pull request """ On Wed, Aug 31, 2016 at 01:30:12AM -0700, Jakub Hrozek wrote: > On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: > > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > > > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: >

[SSSD] Re: [PATCH] knownhostsproxy: use all of the getaddrinfo()

2016-08-31 Thread Lukas Slebodnik
On (31/08/16 10:23), Pavel Březina wrote: >On 08/30/2016 10:06 PM, Lukas Slebodnik wrote: >> On (30/08/16 21:22), Jakub Hrozek wrote: >> > On Tue, Aug 30, 2016 at 06:51:43PM +0200, Lukas Slebodnik wrote: >> > > On (30/08/16 13:01), Pavel Březina wrote: >> > > > On 08/19/2016 02:03 PM, Jakub Hrozek

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread jhrozek
jhrozek commented on a pull request """ On Wed, Aug 31, 2016 at 12:36:37AM -0700, sumit-bose wrote: > On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > > > > About the discussion I saw on #sssd in backscroll, the rfc2307b

[SSSD] RFC: github PR workflow

2016-08-31 Thread Jakub Hrozek
Hi, I documented workflow that we could use for submitting PRs: https://fedorahosted.org/sssd/wiki/GithubWorkflow It's quite similar to what the FreeIPA team uses (although I don't think they publicly document it yet). Comments or edits welcome. If there are none, I'll link the page from the

[SSSD] Re: [PATCH] knownhostsproxy: use all of the getaddrinfo()

2016-08-31 Thread Pavel Březina
On 08/30/2016 10:06 PM, Lukas Slebodnik wrote: On (30/08/16 21:22), Jakub Hrozek wrote: On Tue, Aug 30, 2016 at 06:51:43PM +0200, Lukas Slebodnik wrote: On (30/08/16 13:01), Pavel Březina wrote: On 08/19/2016 02:03 PM, Jakub Hrozek wrote: Hi, I'm sending this patch on behalf of Nalin, who at

[SSSD] [sssd PR#8] MAN: Document the ldap_user_gid_number option (opened)

2016-08-31 Thread jhrozek
jhrozek's pull request #8: "MAN: Document the ldap_user_gid_number option" was opened PR body: """ Some users, mostly those who cannot join the Linux machines to their AD domain resort to using the LDAP provider with ID mapping. But in order for that to work correctly, the ldap_user_gid_number

[SSSD] Re: [PATCH] LDAP: Improving debug message

2016-08-31 Thread Petr Cech
On 08/31/2016 09:36 AM, Lukas Slebodnik wrote: On (31/08/16 08:28), Petr Cech wrote: From b3ae463a7544bb9561126c5e05475d5b98928edc Mon Sep 17 00:00:00 2001 From: Petr Cech Date: Fri, 15 Jul 2016 14:54:35 +0200 Subject: [PATCH] LDAP: Improving debug message There were debug messges refering to

[SSSD] [sssd PR#7] Fix initgroups with nested groups (comment)

2016-08-31 Thread sumit-bose
sumit-bose commented on a pull request """ On Tue, Aug 30, 2016 at 12:36:20PM -0700, Jakub Hrozek wrote: > On Tue, Aug 30, 2016 at 11:47:09AM -0700, lslebodn wrote: > > About the discussion I saw on #sssd in backscroll, the rfc2307bis schema > only uses the member attribute because IIRC the RFC d

[SSSD] Re: [PATCH] LDAP: Improving debug message

2016-08-31 Thread Lukas Slebodnik
On (31/08/16 08:28), Petr Cech wrote: >From b3ae463a7544bb9561126c5e05475d5b98928edc Mon Sep 17 00:00:00 2001 >From: Petr Cech >Date: Fri, 15 Jul 2016 14:54:35 +0200 >Subject: [PATCH] LDAP: Improving debug message > >There were debug messges refering to user by for loop variable. >We might obtain

[SSSD] Re: [PATCHES] Remove leftovers from diag_cmd and force_timeout

2016-08-31 Thread Lukas Slebodnik
On (31/08/16 08:43), Petr Cech wrote: >On 08/31/2016 07:44 AM, Petr Cech wrote: >> On 08/30/2016 06:35 PM, Fabiano Fidêncio wrote: >> > Seems that when I sent the v2 of 7579cf99 and ac35fe74 I attached the >> > wrong patches that ended up being pushed. >> > Those patches were incomplete as there ar