[SSSD] Re: [PATCH] PROXY: Adding proxy_max_children option

2016-09-14 Thread Lukas Slebodnik
On (13/09/16 17:57), Petr Cech wrote: >On 09/13/2016 04:27 PM, Lukas Slebodnik wrote: >> On (13/09/16 16:24), Lukas Slebodnik wrote: >> > On (13/09/16 14:11), Fabiano FidĂȘncio wrote: >> > > On Mon, Sep 12, 2016 at 9:40 AM, Petr Cech wrote: >> > > > Bump. >> > > > >> > > > >> >

[SSSD] [sssd PR#25] TESTS: Add integration tests for the proxy provider of sssd-secrets (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ btw here are CI tests on all supported systems: http://sssd-ci.duckdns.org/logs/job/53/41/summary.html """ See the full comment at https://github.com/SSSD/sssd/pull/25#issuecomment-246998696 ___ sssd-devel

[SSSD] [sssd PR#26] KRB5: Fixing FQ name of user in krb5_setup() (opened)

2016-09-14 Thread celestian
celestian's pull request #26: "KRB5: Fixing FQ name of user in krb5_setup()" was opened PR body: """ This patch fixes creation of FQ username if krb5_map_user option ise used. Resolves: https://fedorahosted.org/sssd/ticket/3188 """ See the full pull-request at

[SSSD] [sssd PR#26] KRB5: Fixing FQ name of user in krb5_setup() (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ Why not do it the other way around and qualify all names in the mapping? That needs to be done only once unlike this approach that needs to unqualify the names every time they are used. """ See the full comment at

[SSSD] [sssd PR#26] KRB5: Fixing FQ name of user in krb5_setup() (+Changes requested)

2016-09-14 Thread jhrozek
celestian's pull request #26: "KRB5: Fixing FQ name of user in krb5_setup()" label *Changes requested* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/26 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] Re: [PATCH SET] SYSDB: Adding message to inform about cache

2016-09-14 Thread Lukas Slebodnik
On (06/09/16 13:15), Petr Cech wrote: >On 09/05/2016 02:31 PM, Fabiano FidĂȘncio wrote: >> On Mon, Sep 5, 2016 at 11:59 AM, Fabiano FidĂȘncio >> wrote: >> > Petr, >> > >> > I went through your patches and in general they look good to me. >> > However, I haven't done any

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (+Accepted)

2016-09-14 Thread jhrozek
fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" label *Accepted* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/11 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#20] sss_override fails to export (closed)

2016-09-14 Thread jhrozek
mzidek-rh's pull request #20: "sss_override fails to export" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/20 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/20/head:pr20 git checkout pr20

[SSSD] [sssd PR#24] MAN: Add a manpage for the sssd-secrets responder (opened)

2016-09-14 Thread jhrozek
jhrozek's pull request #24: "MAN: Add a manpage for the sssd-secrets responder" was opened PR body: """ The manpage lists the options and adds API examples. """ See the full pull-request at https://github.com/SSSD/sssd/pull/24 ... or pull the PR as Git branch: git remote add ghsssd

[SSSD] [sssd PR#25] TESTS: Add integration tests for the proxy provider of sssd-secrets (opened)

2016-09-14 Thread jhrozek
jhrozek's pull request #25: "TESTS: Add integration tests for the proxy provider of sssd-secrets" was opened PR body: """ Execrcies the basic operations of the sssd-secrets responder and can be used as a basis to add more tests. """ See the full pull-request at

[SSSD] [sssd PR#20] sss_override fails to export (+Pushed)

2016-09-14 Thread pbrezina
mzidek-rh's pull request #20: "sss_override fails to export" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/20 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#20] sss_override fails to export (-Changes requested)

2016-09-14 Thread pbrezina
mzidek-rh's pull request #20: "sss_override fails to export" label *Changes requested* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/20 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ The first patch was already pushed. The second looks good to me visually and I wrote a simple patch to verify the fix: https://github.com/jhrozek/sssd/commit/374eff89472a3181548495ed74deadaa690e921e """ See the full comment at

[SSSD] [sssd PR#11] SECRETS: Don't remove a container when it has children (-Changes requested)

2016-09-14 Thread fidencio
fidencio's pull request #11: "SECRETS: Don't remove a container when it has children" label *Changes requested* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/11 ___ sssd-devel mailing list

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ On Tue, Sep 13, 2016 at 01:33:49AM -0700, mzidek-rh wrote: > Makes sense, but I would prefer to add new "else if" branch with > ADMINLIMIT_EXCEEDED specific debug message. > > It would also make sense to change the SDAP_SRCH_FLG_SIZELIMIT_SILENT into >

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (closed)

2016-09-14 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" was closed See the full pull-request at https://github.com/SSSD/sssd/pull/22 ... or pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/22/head:pr22 git checkout

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (comment)

2016-09-14 Thread jhrozek
jhrozek commented on a pull request """ * master: 3319d964721396c07daba383ded6aaaf33ed6e3b """ See the full comment at https://github.com/SSSD/sssd/pull/22#issuecomment-246954357 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (+Pushed)

2016-09-14 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Pushed* has been added See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org

[SSSD] [sssd PR#22] LDAP: Return partial results from adminlimit exceeded (-Accepted)

2016-09-14 Thread jhrozek
jhrozek's pull request #22: "LDAP: Return partial results from adminlimit exceeded" label *Accepted* has been removed See the full pull-request at https://github.com/SSSD/sssd/pull/22 ___ sssd-devel mailing list sssd-devel@lists.fedorahosted.org