[SSSD] Design document - SSSD KCM server

Hi, I was working on a KCM server for SSSD for some time already in parallel with the files provider and had some discussions with Simo as well. Of course my intent wasn't to implement a feature secretly without a design review, but to have a prototype to base a proper design on :) However it

[SSSD] [sssd PR#84][opened] Socket-activation of SSSD responders

URL: https://github.com/SSSD/sssd/pull/84 Author: fidencio Title: #84: Socket-activation of SSSD responders Action: opened PR body: """ This series fixes [#2243](https://fedorahosted.org/sssd/ticket/2243) and [#3129](https://fedorahosted.org/sssd/ticket/3129), following what was discussed

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

Hi, I was out last week, sorry for the delay! The patch looks good to me, I'll try to test it too as soon as I find some time. Thanks, Victor ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] Design document - SSSD's files provider

Hi, I realized I never sent a design document about the files provider to the sssd-devel list - I'm sorry about that. Nonetheless, I discussed the design with Stephen quite some time ago, so hopefully it's not completely wrong. In general, the plan to "manage the users from files" will have

[SSSD] Re: [TESTS] test_secrets.py are failing on rawhide

On Mon, Nov 21, 2016 at 03:34:49PM +0100, Lukas Slebodnik wrote: > ehlo, > > FYI: There are failing integration tests on rawhide > > test_secrets.py::test_containers FAILED > > >

[SSSD] [TESTS] test_secrets.py are failing on rawhide

ehlo, FYI: There are failing integration tests on rawhide test_secrets.py::test_containers FAILED

[SSSD] [sssd PR#70][comment] check_duplicate: check name member before using it

URL: https://github.com/SSSD/sssd/pull/70 Title: #70: check_duplicate: check name member before using it celestian commented: """ @lslebodn, Lukas, are you satisfied by Sumit's explanation? """ See the full comment at https://github.com/SSSD/sssd/pull/70#issuecomment-261941593

[SSSD] [sssd PR#39][comment] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) celestian commented: """ So, I pushed new version. Now ```sysdb_get_sudo_filter()``` uses ```nameAlias``` values. (And after pushing #80 I will cherry-pick it to 1.13 too.) """ See

[SSSD] [sssd PR#39][-Changes requested] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#39][synchronized] RESPONDER: Enable sudoRule in case insen. domains (1.13)

URL: https://github.com/SSSD/sssd/pull/39 Author: celestian Title: #39: RESPONDER: Enable sudoRule in case insen. domains (1.13) Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/39/head:pr39 git checkout pr39 From

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file spbnick commented: """ Ah, I see. Then you can put your patterns into a variable and check against them in the loop, similarly to the way it's done above in the script. You can use extended globs (with

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file spbnick commented: """ Ah, I see. Then you can put your patterns into a variable and check against them in the loop, similarly to the way it's done above in the script. You can use extended globs (with

[SSSD] [sssd PR#81][comment] Please see the commit message, the fix is hopefully simple

URL: https://github.com/SSSD/sssd/pull/81 Title: #81: Please see the commit message, the fix is hopefully simple lslebodn commented: """ On (21/11/16 13:53), fidencio wrote: >For future interactions would be way simpler if the reviewer could just push >the patch with the simple fix and point

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file lslebodn commented: """ On (21/11/16 04:11), Nikolai Kondrashov wrote: >spbnick commented on this pull request. >> @@ -35,3 +35,15 @@ fi > exit found > } > " >+ >+declare

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file fidencio commented: """ Okay, then. As long as my preferences are respected during the review of my patches we're fine. :-) """ See the full comment at

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file fidencio commented: """ Okay, then. As long as my preferences are respected during the review my patches we're fine. :-) """ See the full comment at

[SSSD] [sssd PR#82][comment] KRB5: Remove spurious warning in logs

URL: https://github.com/SSSD/sssd/pull/82 Title: #82: KRB5: Remove spurious warning in logs fidencio commented: """ Acked-by: Fabiano Fidêncio """ See the full comment at https://github.com/SSSD/sssd/pull/82#issuecomment-261929601

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file lslebodn commented: """ On (21/11/16 04:46), fidencio wrote: >Also, please, split this patch in two parts: "removing new line at the end of >file" and "check new line at the end of file". > I prefer to

[SSSD] [sssd PR#81][comment] Please see the commit message, the fix is hopefully simple

URL: https://github.com/SSSD/sssd/pull/81 Title: #81: Please see the commit message, the fix is hopefully simple fidencio commented: """ So, AFAIU what's is missing is this small patch: ``` [ffidenci@cat x86_64]$ git diff diff --git a/src/external/inotify.m4 b/src/external/inotify.m4 index

[SSSD] [sssd PR#81][+Changes requested] Please see the commit message, the fix is hopefully simple

URL: https://github.com/SSSD/sssd/pull/81 Title: #81: Please see the commit message, the fix is hopefully simple Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file fidencio commented: """ Also, please, split this patch in two parts: "removing new line at the end of file" and "check new line at the end of file". """ See the full comment at

[SSSD] [sssd PR#83][+Changes requested] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#80][edited] SYSDB: Fixing of sudorule without a sudoUser

URL: https://github.com/SSSD/sssd/pull/80 Author: celestian Title: #80: SYSDB: Fixing of sudorule without a sudoUser Action: edited Changed field: title Original value: """ SYSDB: Sudorule without a sudoUser returns EINVAL """ ___ sssd-devel

[SSSD] [sssd PR#83][comment] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Title: #83: TESTS: Check new line at end of file spbnick commented: """ @lslebodn I left one suggestion, if that's not what you needed, could you please specify in which way it should be "better"? """ See the full comment at

[SSSD] [sssd PR#80][comment] SYSDB: Sudorule without a sudoUser returns EINVAL

URL: https://github.com/SSSD/sssd/pull/80 Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL celestian commented: """ New version pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/80#issuecomment-261919172 ___

[SSSD] [sssd PR#80][synchronized] SYSDB: Sudorule without a sudoUser returns EINVAL

URL: https://github.com/SSSD/sssd/pull/80 Author: celestian Title: #80: SYSDB: Sudorule without a sudoUser returns EINVAL Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/80/head:pr80 git checkout pr80 From

[SSSD] [sssd PR#83][opened] TESTS: Check new line at end of file

URL: https://github.com/SSSD/sssd/pull/83 Author: lslebodn Title: #83: TESTS: Check new line at end of file Action: opened PR body: """ @spbnick Do you have a better idea how to filter out exceptions in `src/tests/whitespace_test` """ To pull the PR as Git branch: git remote add ghsssd

[SSSD] [sssd PR#82][opened] KRB5: Remove spurious warning in logs

URL: https://github.com/SSSD/sssd/pull/82 Author: lslebodn Title: #82: KRB5: Remove spurious warning in logs Action: opened PR body: """ The option krb5_map_user is empty by default. Therefore we should not confuse users wih warning (Fri Nov 15 09:58:49 2016) [sssd[be[example.com]]]

[SSSD] Re: [Q] t3222 sssd still showing ipa user after removed from last group

On (15/11/16 10:03), Lukas Slebodnik wrote: >On (15/11/16 09:39), Jakub Hrozek wrote: >>On Wed, Nov 09, 2016 at 04:44:12PM +0100, Petr Cech wrote: >>> Hi all, >>> >>> I came back to ticket #3222 "sssd still showing ipa user after removed from >>> last group" [1]. And I have new knowledge. But I