[SSSD] [sssd PR#566][comment] NSS: nss_clear_netgroup_hash_table() do not free data

2018-05-06 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/566 Title: #566: NSS: nss_clear_netgroup_hash_table() do not free data fidencio commented: """ Okay, I'm just running internal CI for the sake of the process and I'll add the accepted label as soon as I get the results. """ See the full comment at https:

[SSSD] [sssd PR#567][comment] TESTS: Increase test_resp_idle_timeout* timeout

2018-05-06 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/567 Title: #567: TESTS: Increase test_resp_idle_timeout* timeout fidencio commented: """ I've ran CI 20 times with the patch with no failure on those specific tests. """ See the full comment at https://github.com/SSSD/sssd/pull/567#issuecomment-386890537

[SSSD] [sssd PR#567][comment] TESTS: Increase test_resp_idle_timeout* timeout

2018-05-06 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/567 Title: #567: TESTS: Increase test_resp_idle_timeout* timeout fidencio commented: """ I've run CI 20 times with the patch with no failure on those specific tests. """ See the full comment at https://github.com/SSSD/sssd/pull/567#issuecomment-386890537

[SSSD] [sssd PR#567][opened] TESTS: Increase test_resp_idle_timeout* timeout

2018-05-06 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/567 Author: fidencio Title: #567: TESTS: Increase test_resp_idle_timeout* timeout Action: opened PR body: """ As suggested by Sumit, let's increase the timeout in the test_resp_idle_timeout* as those are a little bit on the edge. Related: https://pagure.

[SSSD] Re: handling whitespaces in netgroup triples

2018-05-06 Thread Jakub Hrozek
On Sun, May 06, 2018 at 03:31:03PM +0200, Jakub Hrozek wrote: > On Thu, May 03, 2018 at 02:44:40PM +0200, Josef Cejka wrote: > > Hi, > > I have noticed that sssd does not trim whitespaces from strings while > > parsing netgroup triples. Comment inside code explains that it follow > > the nss_ldap i

[SSSD] Re: handling whitespaces in netgroup triples

2018-05-06 Thread Jakub Hrozek
On Thu, May 03, 2018 at 02:44:40PM +0200, Josef Cejka wrote: > Hi, > I have noticed that sssd does not trim whitespaces from strings while > parsing netgroup triples. Comment inside code explains that it follow > the nss_ldap implementation: > > src/db/sysdb_search.c: > 1687 /* This function split

[SSSD] Re: Shall we revert test_resp_idle_timeout_shutdown_slow()?

2018-05-06 Thread Jakub Hrozek
On Fri, May 04, 2018 at 11:42:48AM +0200, Sumit Bose wrote: > On Fri, May 04, 2018 at 11:17:26AM +0200, Fabiano FidĂȘncio wrote: > > On Fri, May 4, 2018 at 10:20 AM, Sumit Bose wrote: > > > On Fri, May 04, 2018 at 09:57:51AM +0200, Fabiano FidĂȘncio wrote: > > >> This test was introduced in > > >> h