[SSSD] [sssd PR#106][comment] Add a new "files" provider

2016-12-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/106 Title: #106: Add a new "files" provider jhrozek commented: """ I hope you noticed the earlier comment that says "I just set the Changes Requested label so that it's clear to reviewers new patch set is coming up.." """ See the full comment at

[SSSD] [sssd PR#106][comment] Add a new "files" provider

2016-12-20 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/106 Title: #106: Add a new "files" provider lslebodn commented: """ I hope you noticed that this PR need to be rebased (due to refactoring of nss responder) """ See the full comment at https://github.com/SSSD/sssd/pull/106#issuecomment-268347295

[SSSD] [sssd PR#107][comment] WATCHDOG: Avoid non async-signal-safe from the signal_handler

2016-12-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/107 Title: #107: WATCHDOG: Avoid non async-signal-safe from the signal_handler fidencio commented: """ CI: http://sssd-ci.duckdns.org/logs/job/59/63/summary.html Rawhide failure is unrelated. """ See the full comment at

[SSSD] [sssd PR#107][-Changes requested] WATCHDOG: Avoid non async-signal-safe from the signal_handler

2016-12-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/107 Title: #107: WATCHDOG: Avoid non async-signal-safe from the signal_handler Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#107][comment] WATCHDOG: Avoid non async-signal-safe from the signal_handler

2016-12-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/107 Title: #107: WATCHDOG: Avoid non async-signal-safe from the signal_handler fidencio commented: """ Patch updated according to Simo's suggestion. Removing the label "Changes requested" as it's done for review. """ See the full comment at

[SSSD] [sssd PR#107][synchronized] WATCHDOG: Avoid non async-signal-safe from the signal_handler

2016-12-20 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/107 Author: fidencio Title: #107: WATCHDOG: Avoid non async-signal-safe from the signal_handler Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/107/head:pr107 git checkout pr107

[SSSD] Re: [PATCH] Create pidfile after responders have started

2016-12-20 Thread Victor Tapia
Hi Lukas, Just wanted to confirm your patch works for me. Kind regards, Victor ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

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

2016-12-20 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req spbnick commented: """ Another question is cache_req going to be used by PAM (if this question even makes sense)? """ See the full comment at

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

2016-12-20 Thread spbnick
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req spbnick commented: """ Not a review, but rather a question to @pbrezina: why do some "plugin" functions receive both cache_req_data and cache_req, which also references cache_req_data, at the

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

2016-12-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/89 Title: #89: nss: rewrite nss responder so it uses cache_req jhrozek commented: """ @pbrezina @lslebodn is there any more work needed on https://fedorahosted.org/sssd/ticket/1126 or https://fedorahosted.org/sssd/ticket/2320 ? Do we anticipate to work

[SSSD] [sssd PR#109][-Changes requested] SSSCTL: fix netgroup-show parsing

2016-12-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/109 Title: #109: SSSCTL: fix netgroup-show parsing Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#112][comment] FAILOVER: Improve port status log messages

2016-12-20 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/112 Title: #112: FAILOVER: Improve port status log messages jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/112#issuecomment-268183489 ___ sssd-devel mailing list