[SSSD] [sssd PR#715][comment] Use 120 second default timeout for dbus (#1654537)

2018-12-13 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/715 Title: #715: Use 120 second default timeout for dbus (#1654537) jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/715#issuecomment-446878401

[SSSD] [sssd PR#713][comment] krb5_child: fix permissions during SC auth

2018-12-12 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/713 Title: #713: krb5_child: fix permissions during SC auth jhrozek commented: """ Thank you. As discussed on IRC, I filed a separate ticket https://pagure.io/SSSD/sssd/issue/3903 as the issue is a bit different from the generic 'can'

[SSSD] [sssd PR#713][+Changes requested] krb5_child: fix permissions during SC auth

2018-12-12 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/713 Title: #713: krb5_child: fix permissions during SC auth Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#714][comment] p11_child(openssl): do not free static memory

2018-12-12 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/714 Title: #714: p11_child(openssl): do not free static memory jhrozek commented: """ For some reason, coverity didn't see the defect as fixed, but OTOH it also didn't find any new defects :-) """ See the full comment at

[SSSD] [sssd PR#713][comment] krb5_child: fix permissions during SC auth

2018-12-12 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/713 Title: #713: krb5_child: fix permissions during SC auth jhrozek commented: """ In general this looks good, but please fix this clang warning: ``` Error: CLANG_WARNING: sssd-2.0.99/src/providers/krb5/krb5_child.c:1759:13: warning: Value

[SSSD] [sssd PR#703][closed] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/703/head:pr703 git checkout pr703

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ * sssd-1-16: 28792523a01a7d21bcc8931794164f253e691a68 """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#703][+Pushed] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ * master: 90f32399b4100ce39cf665649fde82d215e5eb49 """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#711][comment] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root jhrozek commented: """ * sssd-1-16: * 74568bdde833f752187cb1a38b39715556c91279 * d33ec64423087261fcc14acb5922793fadb83342 """ See the full comme

[SSSD] [sssd PR#711][+Pushed] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#711][closed] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Author: sumit-bose Title: #711: ipa: use only the global catalog service of the forest root Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/711/head:pr711 git checkout pr711

[SSSD] [sssd PR#711][comment] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root jhrozek commented: """ * master: * 62d671b874a66101c0f4bff39

[SSSD] [sssd PR#677][comment] pcre: port to pcre2

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/677 Title: #677: pcre: port to pcre2 jhrozek commented: """ I'm sorry to keep beating the error code from *sss_regexp_new...but why not just return int from that function, which would be EOK on success, in which case a **self poi

[SSSD] [sssd PR#705][-Changes requested] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#705][synchronized] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Author: jhrozek Title: #705: KCM: Add configurable quotas Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/705/head:pr705 git checkout pr705 From

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ OK, I'll squash this diff: ``` diff --git a/src/responder/secrets/secsrv.c b/src/responder/s

[SSSD] [sssd PR#711][+Accepted] ipa: use only the global catalog service of the forest root

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/711 Title: #711: ipa: use only the global catalog service of the forest root Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#705][+Changes requested] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ I'm adding changes requested, but since the issue is 'only' in tests, I would still appreciate a review """ See the full comment at https://github.com/S

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ I still need to amend the tests: ``` === short test summary info FAIL test_kcm.py::test_kcm_secrets_quota FAIL

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ The patch does change the behaviour, but it's also just a fallback, so whatever you had defined in AD LDAP is still used. Let me give an example: - b

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/705#issuecomment-446184702 _

[SSSD] [sssd PR#703][+Accepted] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ Seems to work fine, by default I get /home/domain/username for all admins, when I set fallback_homedir=%o then the unixHomeDirectory attribute

[SSSD] [sssd PR#696][closed] DYNDNS: SSSD does not batch DDNS update requests

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/696 Author: thalman Title: #696: DYNDNS: SSSD does not batch DDNS update requests Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/696/head:pr696 git checkout pr696

[SSSD] [sssd PR#696][comment] DYNDNS: SSSD does not batch DDNS update requests

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/696 Title: #696: DYNDNS: SSSD does not batch DDNS update requests jhrozek commented: """ * master: 5565dd365e704f6ded4f95db5bfbefd5dffc888b """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#696][+Pushed] DYNDNS: SSSD does not batch DDNS update requests

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/696 Title: #696: DYNDNS: SSSD does not batch DDNS update requests Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#707][comment] build: remove hardcoded samba include path

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Title: #707: build: remove hardcoded samba include path jhrozek commented: """ * master: 7354e59e010197ab5be3440a0e2c24302298a237 Please let me know if you'd like to have the patch backported to sssd-1-16 as well -- on one h

[SSSD] [sssd PR#707][closed] build: remove hardcoded samba include path

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Author: gmccollister Title: #707: build: remove hardcoded samba include path Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/707/head:pr707 git checkout pr707

[SSSD] [sssd PR#707][+Pushed] build: remove hardcoded samba include path

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Title: #707: build: remove hardcoded samba include path Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ The test failed because of pep8 failures in the test. Now it will hopefully come back clean. Still looking for a reviewer.. """ See the full comment at

[SSSD] [sssd PR#705][synchronized] KCM: Add configurable quotas

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Author: jhrozek Title: #705: KCM: Add configurable quotas Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/705/head:pr705 git checkout pr705 From

[SSSD] [sssd PR#702][-Changes requested] NSS: Avoid changing the memory cache ownership away from the SSSD user

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/702 Title: #702: NSS: Avoid changing the memory cache ownership away from the SSSD user Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#702][comment] NSS: Avoid changing the memory cache ownership away from the SSSD user

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/702 Title: #702: NSS: Avoid changing the memory cache ownership away from the SSSD user jhrozek commented: """ OK, the comment is there. I also did some more changes, because the integration tests started failing intermittently. This

[SSSD] [sssd PR#702][synchronized] NSS: Avoid changing the memory cache ownership away from the SSSD user

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/702 Author: jhrozek Title: #702: NSS: Avoid changing the memory cache ownership away from the SSSD user Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/702/head:pr702 git

[SSSD] [sssd PR#708][+Changes requested] TESTS: ldb-tools are required for multihost tests

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#707][+Accepted] build: remove hardcoded samba include path

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Title: #707: build: remove hardcoded samba include path Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#708][comment] TESTS: ldb-tools are required for multihost tests

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests jhrozek commented: """ Well I do test on Fedora, so I care about the patch. The elif RHEL branch is just a bug, we need to fix the dependency detection better, but for now

[SSSD] [sssd PR#710][comment] data_provider_fo: fix error in hostname retrieval

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/710 Title: #710: data_provider_fo: fix error in hostname retrieval jhrozek commented: """ I think this would fix the error, but I have two more questions: 1) Do you agree it would be nice to fix all gethostname() calls in a similar ma

[SSSD] [sssd PR#710][comment] data_provider_fo: fix error in hostname retrieval

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/710 Title: #710: data_provider_fo: fix error in hostname retrieval jhrozek commented: """ add to whitelist """ See the full comment at https://github.com/SSSD/sssd/pull/710#issuecomment-445745812

[SSSD] [sssd PR#710][comment] data_provider_fo: fix error in hostname retrieval

2018-12-10 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/710 Title: #710: data_provider_fo: fix error in hostname retrieval jhrozek commented: """ (Your patches should no longer by gated by centos CI) """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#709][comment] Regex fails if there's a whitespace before option name

2018-12-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/709 Title: #709: Regex fails if there's a whitespace before option name jhrozek commented: """ @mzidek-rh do you have some opinion on the patch? """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#709][comment] Regex fails if there's a whitespace before option name

2018-12-07 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/709 Title: #709: Regex fails if there's a whitespace before option name jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/709#issuecomment-445157651

[SSSD] [sssd PR#707][comment] build: remove hardcoded samba include path

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Title: #707: build: remove hardcoded samba include path jhrozek commented: """ OK, so the CI 'passed', meaning we have ran into a totally unrelated issue. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#707][comment] build: remove hardcoded samba include path

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Title: #707: build: remove hardcoded samba include path jhrozek commented: """ thanks, submitted to our internal CI (that also runs different RHEL releases and Debian), will ack if the build comes through """

[SSSD] [sssd PR#704][+Pushed] PROXY: Copy the response to the caller

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/704 Title: #704: PROXY: Copy the response to the caller Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#704][closed] PROXY: Copy the response to the caller

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/704 Author: jhrozek Title: #704: PROXY: Copy the response to the caller Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/704/head:pr704 git checkout pr704

[SSSD] [sssd PR#704][comment] PROXY: Copy the response to the caller

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/704 Title: #704: PROXY: Copy the response to the caller jhrozek commented: """ * master: 807bbce25ffedb6f0d2d61831b5d5133e11aa84a """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#701][comment] ci: add ability to run tests in jenkins

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/701 Title: #701: ci: add ability to run tests in jenkins jhrozek commented: """ * master: 36255b893a8a55588309a7c5729560c48a30018d """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#701][closed] ci: add ability to run tests in jenkins

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/701 Author: pbrezina Title: #701: ci: add ability to run tests in jenkins Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/701/head:pr701 git checkout pr701

[SSSD] [sssd PR#701][+Pushed] ci: add ability to run tests in jenkins

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/701 Title: #701: ci: add ability to run tests in jenkins Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#695][comment] sss_iface: prevent from using invalid names that start with digits

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/695 Title: #695: sss_iface: prevent from using invalid names that start with digits jhrozek commented: """ * master: f47940356462a3f477fe462e71d7680c959300db """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#695][+Pushed] sss_iface: prevent from using invalid names that start with digits

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/695 Title: #695: sss_iface: prevent from using invalid names that start with digits Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#695][closed] sss_iface: prevent from using invalid names that start with digits

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/695 Author: pbrezina Title: #695: sss_iface: prevent from using invalid names that start with digits Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/695/head:pr695 git checkout pr695

[SSSD] [sssd PR#690][closed] DYNDNS: Convert dyndns timer to be_ptask

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/690 Author: thalman Title: #690: DYNDNS: Convert dyndns timer to be_ptask Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/690/head:pr690 git checkout pr690

[SSSD] [sssd PR#690][comment] DYNDNS: Convert dyndns timer to be_ptask

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask jhrozek commented: """ * master: df9e4802c060fc21d38f238265805092352e5c95 """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#690][+Pushed] DYNDNS: Convert dyndns timer to be_ptask

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#701][+Accepted] ci: add ability to run tests in jenkins

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/701 Title: #701: ci: add ability to run tests in jenkins Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#701][comment] ci: add ability to run tests in jenkins

2018-12-05 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/701 Title: #701: ci: add ability to run tests in jenkins jhrozek commented: """ I'll push the patch if the builds succeed here. What we need next is some documentation.. """ See the full comment at https://github.com/S

[SSSD] [sssd PR#702][+Changes requested] NSS: Avoid changing the memory cache ownership away from the SSSD user

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/702 Title: #702: NSS: Avoid changing the memory cache ownership away from the SSSD user Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ I also had a follow-up discussion with simo on IRC, let me paste rephrasing: - the AD provider should have an AD specific internal option that generat

[SSSD] [sssd PR#707][comment] build: remove hardcoded samba include path

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/707 Title: #707: build: remove hardcoded samba include path jhrozek commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/707#issuecomment-444129009 _

[SSSD] [sssd PR#706][opened] KCM: Fall back to using the first ccache if the default does not exist

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/706 Author: jhrozek Title: #706: KCM: Fall back to using the first ccache if the default does not exist Action: opened PR body: """ Resolves: https://pagure.io/SSSD/sssd/issue/3838 KCM stores the default ccache in a separate DB entry.

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ I thought that fallback_homedir = "" would work but it doesn't, not even with escaping quotes. An empty attribute is silently ignored. Ab

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ > On Tue, 2018-12-04 at 04:51 -0800, Jakub Hrozek wrote: Thanks, this passes > the test. And of course the patch is correct, but after some more

[SSSD] [sssd PR#705][comment] KCM: Add configurable quotas

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/pull/705#issuecomment-444096360 _

[SSSD] [sssd PR#702][comment] NSS: Avoid changing the memory cache ownership away from the SSSD user

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/702 Title: #702: NSS: Avoid changing the memory cache ownership away from the SSSD user jhrozek commented: """ retest this please """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ Thanks, this passes the test. And of course the patch is correct, but after some more testing, I wonder if we should at least for one

[SSSD] [sssd PR#703][comment] nss: sssd returns '/' for emtpy home directories

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories jhrozek commented: """ You also need to amend `test_user_no_dir` in `src/tests/intg/test_files_provider.py` """ See the full comment at https://github.com/S

[SSSD] [sssd PR#705][opened] KCM: Add configurable quotas

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Author: jhrozek Title: #705: KCM: Add configurable quotas Action: opened PR body: """ This PR adds several patches that let the user configure quotas to store their ccaches. Please see the commit messages, I hope they are verbose en

[SSSD] [sssd PR#695][+Accepted] sss_iface: prevent from using invalid names that start with digits

2018-12-04 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/695 Title: #695: sss_iface: prevent from using invalid names that start with digits Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#704][opened] PROXY: Copy the response to the caller

2018-12-03 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/704 Author: jhrozek Title: #704: PROXY: Copy the response to the caller Action: opened PR body: """ Resolves: https://pagure.io/SSSD/sssd/issue/3892 """ To pull the PR as Git branch: git remote add ghsssd https://github.

[SSSD] [sssd PR#702][opened] NSS: Avoid changing the memory cache ownership away from the SSSD user

2018-11-30 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/702 Author: jhrozek Title: #702: NSS: Avoid changing the memory cache ownership away from the SSSD user Action: opened PR body: """ Resolves: https://pagure.io/SSSD/sssd/issue/3890 In case SSSD is compiled --with-sssd-user but run

[SSSD] [sssd PR#700][comment] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information jhrozek commented: """ btw I opened https://pagure.io/SSSD/sssd/issue/3889 to track the additional hardening. Maybe it would be a ni

[SSSD] [sssd PR#700][comment] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information jhrozek commented: """ * sssd-1-16: 1a7c6ab6efce3720d27def426

[SSSD] [sssd PR#700][comment] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information jhrozek commented: """ * master: * 6f113c7ddeaa5c82558e10118

[SSSD] [sssd PR#700][+Pushed] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#700][closed] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Author: jhrozek Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/700/head:pr700 git

[SSSD] [sssd PR#700][synchronized] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Author: jhrozek Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/700

[SSSD] [sssd PR#700][comment] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information jhrozek commented: """ thanks, I added your newest patch version to this PR """ See the full comment at https:/

[SSSD] [sssd PR#700][comment] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information jhrozek commented: """ Thank you, this is nice. I added your patch to the PR, fixed one typo and one minor style issue. If you're OK wi

[SSSD] [sssd PR#700][synchronized] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Author: jhrozek Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/700

[SSSD] [sssd PR#699][closed] Fixes for MIT Kerberos 1.17 and valgind CI runs

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/699 Author: sumit-bose Title: #699: Fixes for MIT Kerberos 1.17 and valgind CI runs Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/699/head:pr699 git checkout pr699

[SSSD] [sssd PR#699][+Pushed] Fixes for MIT Kerberos 1.17 and valgind CI runs

2018-11-26 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/699 Title: #699: Fixes for MIT Kerberos 1.17 and valgind CI runs Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#700][-Changes requested] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send

[SSSD] [sssd PR#700][synchronized] LDAP: Only authenticate the auth connection if we need to look up user information

2018-11-23 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/700 Author: jhrozek Title: #700: LDAP: Only authenticate the auth connection if we need to look up user information Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/700

[SSSD] [sssd PR#699][comment] Fixes for MIT Kerberos 1.17 and valgind CI runs

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/699 Title: #699: Fixes for MIT Kerberos 1.17 and valgind CI runs jhrozek commented: """ CI passed completely: http://vm-031.$ABC/logs/job/94/94/summary.html """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#699][+Accepted] Fixes for MIT Kerberos 1.17 and valgind CI runs

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/699 Title: #699: Fixes for MIT Kerberos 1.17 and valgind CI runs Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#644][closed] When multiple UIDs exist, use the username provided by the user as the first lookup

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/644 Author: joeFischetti Title: #644: When multiple UIDs exist, use the username provided by the user as the first lookup Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd

[SSSD] [sssd PR#644][+Rejected] When multiple UIDs exist, use the username provided by the user as the first lookup

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/644 Title: #644: When multiple UIDs exist, use the username provided by the user as the first lookup Label: +Rejected ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email

[SSSD] [sssd PR#698][comment] Add support for EC keys

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/698 Title: #698: Add support for EC keys jhrozek commented: """ * master: 3906e5f41a00063127e07f5ca

[SSSD] [sssd PR#698][closed] Add support for EC keys

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/698 Author: sumit-bose Title: #698: Add support for EC keys Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/698/head:pr698 git checkout pr698

[SSSD] [sssd PR#698][+Pushed] Add support for EC keys

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/698 Title: #698: Add support for EC keys Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code of

[SSSD] [sssd PR#697][comment] RESPONDER: Log failures from bind() and listen()

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/697 Title: #697: RESPONDER: Log failures from bind() and listen() jhrozek commented: """ * master: 75696ddc84ab08c8c885dacc7796ebc8afc429ec """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#697][closed] RESPONDER: Log failures from bind() and listen()

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/697 Author: jhrozek Title: #697: RESPONDER: Log failures from bind() and listen() Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/697/head:pr697 git checkout pr697

[SSSD] [sssd PR#697][+Pushed] RESPONDER: Log failures from bind() and listen()

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/697 Title: #697: RESPONDER: Log failures from bind() and listen() Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#694][comment] SSSCTL: user-show says that user is expired

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/694 Title: #694: SSSCTL: user-show says that user is expired jhrozek commented: """ * master: 291071cb3c04eda7606d62bbff123a0a125c7d60 """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#694][+Pushed] SSSCTL: user-show says that user is expired

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/694 Title: #694: SSSCTL: user-show says that user is expired Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org

[SSSD] [sssd PR#694][closed] SSSCTL: user-show says that user is expired

2018-11-22 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/694 Author: thalman Title: #694: SSSCTL: user-show says that user is expired Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/694/head:pr694 git checkout pr694

<    4   5   6   7   8   9   10   11   12   13   >