[SSSD] [sssd PR#100][synchronized] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2016-12-07 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/100 Author: amitkumar50 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/100/head:pr10

[SSSD] [sssd PR#100][synchronized] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2016-12-07 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/100 Author: amitkumar50 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/100/head:pr10

[SSSD] [sssd PR#104][+Accepted] Suppress a confusing error message from timestamp cache updates

2016-12-07 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/104 Title: #104: Suppress a confusing error message from timestamp cache updates Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@l

[SSSD] [sssd PR#104][comment] Suppress a confusing error message from timestamp cache updates

2016-12-07 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/104 Title: #104: Suppress a confusing error message from timestamp cache updates mzidek-rh commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/104#issuecomment-265496840 ___ sssd-de

[SSSD] [sssd PR#105][+Accepted] RESPONDER: Remove dead assignment to the variable ret

2016-12-07 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/105 Title: #105: RESPONDER: Remove dead assignment to the variable ret Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedor

[SSSD] [sssd PR#105][comment] RESPONDER: Remove dead assignment to the variable ret

2016-12-07 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/105 Title: #105: RESPONDER: Remove dead assignment to the variable ret mzidek-rh commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/105#issuecomment-265487170 ___ sssd-devel mailin

[SSSD] [sssd PR#105][opened] RESPONDER: Remove dead assignment to the variable ret

2016-12-07 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/105 Author: lslebodn Title: #105: RESPONDER: Remove dead assignment to the variable ret Action: opened PR body: """ """ To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/105/head:pr105 git checkout pr

[SSSD] [sssd PR#104][comment] Suppress a confusing error message from timestamp cache updates

2016-12-07 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/104 Title: #104: Suppress a confusing error message from timestamp cache updates mzidek-rh commented: """ Tentative ACK. Testing now. """ See the full comment at https://github.com/SSSD/sssd/pull/104#issuecomment-265478161

[SSSD] [sssd PR#89][comment] nss: rewrite nss responder so it uses cache_req

2016-12-07 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req pbrezina commented: """ Enumeration, services and memory leak were fixed. * Enumeration issue was caused by calling `talloc_get_type()` on non-talloc pointer. * Services issue was caused by no

[SSSD] [sssd PR#104][opened] Suppress a confusing error message from timestamp cache updates

2016-12-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/104 Author: jhrozek Title: #104: Suppress a confusing error message from timestamp cache updates Action: opened PR body: """ 0 is not an error, let's use a second variable instead to track the timestamp update retval and only emit a MINOR-level failure if

[SSSD] Re: [RFCv3] NSS tlog integration

2016-12-07 Thread Nikolai Kondrashov
On 08/30/2016 02:21 PM, Sumit Bose wrote: On Mon, Aug 29, 2016 at 05:18:58PM -0400, Simo Sorce wrote: On Mon, 2016-08-29 at 22:47 +0200, Sumit Bose wrote: About Simo's concern. If there are groups defined in the session_recording configuration you look up those groups with cache_req_group_by_na

[SSSD] [sssd PR#100][comment] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds

2016-12-07 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/100 Title: #100: Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds pbrezina commented: """ It sounds good to me. Thank you for your contribution. I will leave final ack for some native speaker. """ See the full comment at h

[SSSD] Updation of sssd-ad man page for case when dyndns_refresh_interval < 60 seconds #100

2016-12-07 Thread amit kumar
Hi, https://github.com/SSSD/sssd/pull/100 I am planning to use this: Note that lowest possible value is 60 seconds in-case if value is provided less than 60, parameter will assume lowest value only. Thanks ___ sssd-devel mailing list -- sssd-devel@lis

[SSSD] Re: Feedback About probable Fix for sssd/ticket/1149

2016-12-07 Thread Pavel Březina
On 12/06/2016 06:21 PM, amit kumar wrote: Hello, _https://fedorahosted.org/sssd/ticket/1149 _Yes replacement of talloc_get_type() with talloc_get_type_abort() is good move, reasons mentioned in description. Current sssd contains 773 occurrences of talloc_get_type. I suppose Fix is: Replacing a