[SSSD] [sssd PR#636][+Pushed] failover: tune up default timeouts

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/636 Title: #636: failover: tune up default timeouts Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Co

[SSSD] [sssd PR#636][closed] failover: tune up default timeouts

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/636 Author: pbrezina Title: #636: failover: tune up default timeouts Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/636/head:pr636 git checkout pr636 __

[SSSD] [sssd PR#636][comment] failover: tune up default timeouts

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/636 Title: #636: failover: tune up default timeouts jhrozek commented: """ * master: * 049f3906b9ef2041b5e1df666bd570379ae60718 * e97ff0adb62c89cfc7e75858b7e592e0303720b0 * 99e2a107f01c625cb59cb88589db87294176d6c6 * 3807de1d97fc87cf7c25af264a8b1bbabdef5

[SSSD] [sssd PR#847][comment] systemd: add Restart=on-failure to sssd.service

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/847 Title: #847: systemd: add Restart=on-failure to sssd.service jhrozek commented: """ * master: b1ea33eca64a0429513fcfe2ba7402ff56889b46 """ See the full comment at https://github.com/SSSD/sssd/pull/847#issuecomment-516996775 ___

[SSSD] [sssd PR#847][closed] systemd: add Restart=on-failure to sssd.service

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/847 Author: pbrezina Title: #847: systemd: add Restart=on-failure to sssd.service Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/847/head:pr847 git checkout pr847 _

[SSSD] [sssd PR#847][+Pushed] systemd: add Restart=on-failure to sssd.service

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/847 Title: #847: systemd: add Restart=on-failure to sssd.service Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.

[SSSD] [sssd PR#853][+Pushed] DYNDNS: dyndns_update is not enough

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/853 Title: #853: DYNDNS: dyndns_update is not enough Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora

[SSSD] [sssd PR#853][comment] DYNDNS: dyndns_update is not enough

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/853 Title: #853: DYNDNS: dyndns_update is not enough jhrozek commented: """ * master: * f2c69a67ad0cd9d4db94aa66e46ede0cb0790480 * 1c7521898f1cb13607c536977029561f89573c7c * 5b235bbdbea355923e4f2aeb745c8e514b423984 """ See the full comment at https:

[SSSD] [sssd PR#853][closed] DYNDNS: dyndns_update is not enough

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/853 Author: thalman Title: #853: DYNDNS: dyndns_update is not enough Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/853/head:pr853 git checkout pr853

[SSSD] [sssd PR#855][+Pushed] nss: Fix command 'endservent' resetting wrong struct member

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/855 Title: #855: nss: Fix command 'endservent' resetting wrong struct member Label: +Pushed ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.f

[SSSD] [sssd PR#855][comment] nss: Fix command 'endservent' resetting wrong struct member

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/855 Title: #855: nss: Fix command 'endservent' resetting wrong struct member jhrozek commented: """ * sssd-1-16: 9673ca8 * master: 06479a1 """ See the full comment at https://github.com/SSSD/sssd/pull/855#issuecomment-516995456 __

[SSSD] [sssd PR#855][closed] nss: Fix command 'endservent' resetting wrong struct member

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/855 Author: scabrero Title: #855: nss: Fix command 'endservent' resetting wrong struct member Action: closed To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/855/head:pr855 git checkout pr855 _

[SSSD] [sssd PR#636][+Accepted] failover: tune up default timeouts

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/636 Title: #636: failover: tune up default timeouts Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora

[SSSD] [sssd PR#636][comment] failover: tune up default timeouts

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/636 Title: #636: failover: tune up default timeouts jhrozek commented: """ F-30 failed CI and there are no logs. But I don't see anything OS-specific in the patches and at the same time all my concerns were addressed. Thank you. ACK. """ See the full com

[SSSD] [sssd PR#837][comment] p11_child: make OCSP digest configurable

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/837 Title: #837: p11_child: make OCSP digest configurable jhrozek commented: """ OK, I tried also different values of the `ocsp_dgst` option and at least I see that the revoked cert is still revoked..and invalid value seems to fall back to the default whi

[SSSD] [sssd PR#837][comment] p11_child: make OCSP digest configurable

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/837 Title: #837: p11_child: make OCSP digest configurable jhrozek commented: """ Sorry it took me almost four weeks to test the PR. I think OSCP in general works fine. With a valid certificate I was getting: ``` (Wed Jul 31 15:34:44 2019) [[sssd[p11_chil

[SSSD] [sssd PR#856][comment] pam_sss: Add missing colon to the PIN prompt

2019-07-31 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/856 Title: #856: pam_sss: Add missing colon to the PIN prompt sumit-bose commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/856#issuecomment-516793690 ___ sssd-devel mailing

[SSSD] [sssd PR#856][opened] pam_sss: Add missing colon to the PIN prompt

2019-07-31 Thread Jakuje
URL: https://github.com/SSSD/sssd/pull/856 Author: Jakuje Title: #856: pam_sss: Add missing colon to the PIN prompt Action: opened PR body: """ This can be noticed in the sudo prompt, when the system is configured to authenticate users using smart cards. Resolves: Pagure#4049 [1] [1] https:/

[SSSD] [sssd PR#856][comment] pam_sss: Add missing colon to the PIN prompt

2019-07-31 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/856 Title: #856: pam_sss: Add missing colon to the PIN prompt centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/856#issuecomment-516791345 __

[SSSD] [sssd PR#851][comment] Update __init__.py.in

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/851 Title: #851: Update __init__.py.in jhrozek commented: """ I don't know if it's easy or possible but wouldn't it be better to amend the config API to internally synthetize the provider values, but does not write them to the config file? Otherwise I'm s

[SSSD] [sssd PR#841][+Changes requested] DEBUG: Add debug to display ldapsearch requests

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/841 Title: #841: DEBUG: Add debug to display ldapsearch requests Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fe

[SSSD] [sssd PR#841][comment] DEBUG: Add debug to display ldapsearch requests

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/841 Title: #841: DEBUG: Add debug to display ldapsearch requests jhrozek commented: """ The code looks OK. I haven't tried it, but looks simple enough. But most importantly, it is not enough to resolve the issue, I think. What the original reporter asked

[SSSD] [sssd PR#847][comment] systemd: add Restart=on-failure to sssd.service

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/847 Title: #847: systemd: add Restart=on-failure to sssd.service jhrozek commented: """ ACK, tested by sending SIGV to the main sssd process, systemd spawned another one. """ See the full comment at https://github.com/SSSD/sssd/pull/847#issuecomment-5167

[SSSD] [sssd PR#636][-Changes requested] failover: tune up default timeouts

2019-07-31 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/636 Title: #636: failover: tune up default timeouts Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.or

[SSSD] [sssd PR#636][comment] failover: tune up default timeouts

2019-07-31 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/636 Title: #636: failover: tune up default timeouts pbrezina commented: """ It should be fixed now. """ See the full comment at https://github.com/SSSD/sssd/pull/636#issuecomment-516762783 ___ sssd-devel mailing

[SSSD] [sssd PR#847][+Accepted] systemd: add Restart=on-failure to sssd.service

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/847 Title: #847: systemd: add Restart=on-failure to sssd.service Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahoste

[SSSD] [sssd PR#636][synchronized] failover: tune up default timeouts

2019-07-31 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/636 Author: pbrezina Title: #636: failover: tune up default timeouts Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/636/head:pr636 git checkout pr636 From 77236ad720a8ce4208fe284

[SSSD] [sssd PR#853][comment] DYNDNS: dyndns_update is not enough

2019-07-31 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/853 Title: #853: DYNDNS: dyndns_update is not enough pbrezina commented: """ Ack. """ See the full comment at https://github.com/SSSD/sssd/pull/853#issuecomment-516754770 ___ sssd-devel mailing list -- sssd-de

[SSSD] [sssd PR#853][+Accepted] DYNDNS: dyndns_update is not enough

2019-07-31 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/853 Title: #853: DYNDNS: dyndns_update is not enough Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedo

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

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/705 Title: #705: KCM: Add configurable quotas jhrozek commented: """ Now some pep8 errors found by CI were fixed. """ See the full comment at https://github.com/SSSD/sssd/pull/705#issuecomment-516752179 ___ sssd

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

2019-07-31 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 7584b16dd738a982595519354d2441

[SSSD] [sssd PR#855][+Accepted] nss: Fix command 'endservent' resetting wrong struct member

2019-07-31 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/855 Title: #855: nss: Fix command 'endservent' resetting wrong struct member Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists

[SSSD] [sssd PR#851][comment] Update __init__.py.in

2019-07-31 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/851 Title: #851: Update __init__.py.in pbrezina commented: """ When we are touching the code, I believe also `subdomains_provider` is missing. Also most of the providers defaults to `id_provider`, few have other defaults so I think it should be handled he