[SSSD] [sssd PR#669][opened] test_config: Test for invalid characker in domain

2018-10-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/669 Author: thalman Title: #669: test_config: Test for invalid characker in domain Action: opened PR body: """ There was bug allowing forbidden characters in config file section name. Bug has been fixed meantime but we decided to write th

[SSSD] [sssd PR#667][-Changes requested] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Title: #667: UTIL: rename & cleanup sysdb_error_to_errno Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#667][comment] WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno

2018-10-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Title: #667: WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno thalman commented: """ @mzidek-rh, @jhrozek PR updated, I moved #define to sysdb.h. I also did the cleanup. Please take a look at cleanup c

[SSSD] [sssd PR#667][comment] WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno

2018-10-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Title: #667: WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno thalman commented: """ @mzidek-rh, @jhrozek PR updated, I moved #define to sysdb.h. I also did the cleanup. Please take a look at cleanup c

[SSSD] [sssd PR#667][edited] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Author: thalman Title: #667: UTIL: rename & cleanup sysdb_error_to_errno Action: edited Changed field: title Original value: """ WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno """ __

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

2018-10-15 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: opened PR body: """ Some distributions want to drop pcre support. Sssd should work with pcre2. With this patch sssd tries to use pcre2 if pcre is not present. Resolves: https:/

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

2018-10-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Title: #677: pcre: port to pcre2 thalman commented: """ Re-formatted to 80 columns """ See the full comment at https://github.com/SSSD/sssd/pull/677#issuecomment-431803305 _

[SSSD] [sssd PR#682][opened] DYNDNS: Drop support for legacy NSUPDATE

2018-10-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/682 Author: thalman Title: #682: DYNDNS: Drop support for legacy NSUPDATE Action: opened PR body: """ We should drop support for legacy versions of NSUPDATE that doesn't support 'realm' option. The option 'realm' was added in BIND 9.8

[SSSD] [sssd PR#677][-Changes requested] pcre: port to pcre2

2018-10-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Title: #677: pcre: port to pcre2 Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code

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

2018-10-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

[SSSD] [sssd PR#677][-Changes requested] pcre: port to pcre2

2018-10-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Title: #677: pcre: port to pcre2 Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org Fedora Code

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

2018-10-17 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

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

2018-11-05 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Author: thalman Title: #690: DYNDNS: Convert dyndns timer to be_ptask Action: opened PR body: """ We have API for periodic tasks and we want to use it instead of tevent timers. Resolves: https://pagure.io/SSSD/sssd/issue/1944 &quo

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

2018-11-13 Thread thalman
URL: https://github.com/SSSD/sssd/pull/694 Author: thalman Title: #694: SSSCTL: user-show says that user is expired Action: opened PR body: """ sssctl user-show says that user is expired if the user comes from files provider. This is ok because files user's expiration time is

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

2018-11-15 Thread thalman
URL: https://github.com/SSSD/sssd/pull/696 Author: thalman Title: #696: DYNDNS: SSSD does not batch DDNS update requests Action: opened PR body: """ SSSD includes a 'send' command in between each record modification and does not batch DDNS update requests. This is problemati

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

2018-11-16 Thread thalman
URL: https://github.com/SSSD/sssd/pull/694 Author: thalman Title: #694: SSSCTL: user-show says that user is expired Action: synchronized 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 From

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

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

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

2018-11-16 Thread thalman
URL: https://github.com/SSSD/sssd/pull/694 Author: thalman Title: #694: SSSCTL: user-show says that user is expired Action: synchronized 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 From

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

2018-11-16 Thread thalman
URL: https://github.com/SSSD/sssd/pull/694 Title: #694: SSSCTL: user-show says that user is expired thalman commented: """ Updated. """ See the full comment at https://github.com/SSSD/sssd/pull/694#issuecomment-439407410 _

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

2018-11-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask thalman commented: """ This limit was not in code, only in documentation (sssd-ad.5.xml). It is also inconsistent with sssd-ipa where there is no such limit and also

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

2018-11-09 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Author: thalman Title: #690: DYNDNS: Convert dyndns timer to be_ptask Action: synchronized 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 From

[SSSD] [sssd PR#667][edited] WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno

2018-10-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Author: thalman Title: #667: WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno Action: edited Changed field: title Original value: """ UTIL: rename & cleanup sysdb_error_to_errno """ __

[SSSD] [sssd PR#667][synchronized] WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno

2018-10-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Author: thalman Title: #667: WIP, do not merge, UTIL: rename & cleanup sysdb_error_to_errno Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/667/head:pr667 git checkout p

[SSSD] [sssd PR#667][comment] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Title: #667: UTIL: rename & cleanup sysdb_error_to_errno thalman commented: """ I will take a look at it and extend PR """ See the full comment at https://github.com/SSSD/ss

[SSSD] [sssd PR#662][-Changes requested] doc: Add nsswitch.conf note to manpage

2018-10-05 Thread thalman
URL: https://github.com/SSSD/sssd/pull/662 Title: #662: doc: Add nsswitch.conf note to manpage Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#662][comment] doc: Add nsswitch.conf note to manpage

2018-10-05 Thread thalman
URL: https://github.com/SSSD/sssd/pull/662 Title: #662: doc: Add nsswitch.conf note to manpage thalman commented: """ Documentation updated """ See the full comment at https://github.com/SSSD/sssd/pull/662#issuecomment-427293559

[SSSD] [sssd PR#667][comment] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-09 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Title: #667: UTIL: rename & cleanup sysdb_error_to_errno thalman commented: """ This is unfortunate and after cleanup hard to isolate. In fact the only reason for linking with ldb is DEBUG() message in sss_ldb_error_to_e

[SSSD] [sssd PR#667][synchronized] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-09 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Author: thalman Title: #667: UTIL: rename & cleanup sysdb_error_to_errno Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/667/head:pr667 git checkout pr667

[SSSD] [sssd PR#669][edited] test_config: Test for invalid character in domain

2018-10-09 Thread thalman
URL: https://github.com/SSSD/sssd/pull/669 Author: thalman Title: #669: test_config: Test for invalid character in domain Action: edited Changed field: title Original value: """ test_config: Test for invalid characker in domain """

[SSSD] [sssd PR#667][comment] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-09 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Title: #667: UTIL: rename & cleanup sysdb_error_to_errno thalman commented: """ PR updated """ See the full comment at https://github.com/SSSD/sssd/pull/667#issuecomment-428215285

[SSSD] [sssd PR#662][synchronized] doc: Add nsswitch.conf note to manpage

2018-10-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/662 Author: thalman Title: #662: doc: Add nsswitch.conf note to manpage Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/662/head:pr662 git checkout pr662 From

[SSSD] [sssd PR#661][comment] TESTS: Add a test for whitespace trimming in netgroup entries

2018-10-11 Thread thalman
URL: https://github.com/SSSD/sssd/pull/661 Title: #661: TESTS: Add a test for whitespace trimming in netgroup entries thalman commented: """ It looks good. I tested it with suggested revert and it works. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#661][+Accepted] TESTS: Add a test for whitespace trimming in netgroup entries

2018-10-11 Thread thalman
URL: https://github.com/SSSD/sssd/pull/661 Title: #661: TESTS: Add a test for whitespace trimming in netgroup entries Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#662][opened] doc: Add nsswitch.conf note to manpage

2018-10-01 Thread thalman
URL: https://github.com/SSSD/sssd/pull/662 Author: thalman Title: #662: doc: Add nsswitch.conf note to manpage Action: opened PR body: """ We want to add note about nsswitch.conf configuration into sssd-files manpage. Resolves: https://pagure.io/SSSD/sssd/issue/3750 &quo

[SSSD] [sssd PR#663][opened] confdb: log an error when domain is misconfigured

2018-10-01 Thread thalman
URL: https://github.com/SSSD/sssd/pull/663 Author: thalman Title: #663: confdb: log an error when domain is misconfigured Action: opened PR body: """ We need to inform user that there is misconfiguration and particular domain will not be started. Resolves: https://pagure.io/

[SSSD] [sssd PR#667][opened] UTIL: rename & cleanup sysdb_error_to_errno

2018-10-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/667 Author: thalman Title: #667: UTIL: rename & cleanup sysdb_error_to_errno Action: opened PR body: """ * UTIL: move and rename sysdb_error_to_errno to utils * UTIL: Use new ldb_error_to_errno function in code """

[SSSD] [sssd PR#662][comment] doc: Add nsswitch.conf note to manpage

2018-10-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/662 Title: #662: doc: Add nsswitch.conf note to manpage thalman commented: """ Thanks @justin-stephenson . @jhrozek PR updated """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#662][synchronized] doc: Add nsswitch.conf note to manpage

2018-10-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/662 Author: thalman Title: #662: doc: Add nsswitch.conf note to manpage Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/662/head:pr662 git checkout pr662 From

[SSSD] [sssd PR#719][synchronized] ifp: extraAttributes is UnknownProperty

2019-01-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/719 Author: thalman Title: #719: ifp: extraAttributes is UnknownProperty Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/719/head:pr719 git checkout pr719 From

[SSSD] [sssd PR#716][synchronized] CACHE: SSSD doesn't clear cache entries

2019-01-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/716 Author: thalman Title: #716: CACHE: SSSD doesn't clear cache entries Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/716/head:pr716 git checkout pr716 From

[SSSD] [sssd PR#716][synchronized] CACHE: SSSD doesn't clear cache entries

2019-01-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/716 Author: thalman Title: #716: CACHE: SSSD doesn't clear cache entries Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/716/head:pr716 git checkout pr716 From

[SSSD] [sssd PR#730][synchronized] confdb: sssd tools don't handle the implicit domain

2019-01-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/730 Author: thalman Title: #730: confdb: sssd tools don't handle the implicit domain Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/730/head:pr730 git checkout pr730 From

[SSSD] [sssd PR#730][comment] confdb: sssd tools don't handle the implicit domain

2019-01-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/730 Title: #730: confdb: sssd tools don't handle the implicit domain thalman commented: """ @alexey-tikhonov thanks for comment. PR updated. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#721][+Accepted] AD/IPA: Reset subdomain service name, not domain name

2019-01-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/721 Title: #721: AD/IPA: Reset subdomain service name, not domain name Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

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

2018-12-13 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: WIP pcre: port to pcre2 Action: edited Changed field: title Original value: """ pcre: port to pcre2 """ ___ sssd-devel mailing list -- sss

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

2018-12-13 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Title: #677: WIP pcre: port to pcre2 thalman commented: """ @jhrozek Please take a look at this update. Let me know if I get your suggestion right and I wil rebase it to one commit. """ See the full comment at https:/

[SSSD] [sssd PR#716][synchronized] CACHE: SSSD doesn't clear cache entries

2018-12-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/716 Author: thalman Title: #716: CACHE: SSSD doesn't clear cache entries Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/716/head:pr716 git checkout pr716 From

[SSSD] [sssd PR#719][opened] ifp: extraAttributes is UnknownProperty

2018-12-18 Thread thalman
URL: https://github.com/SSSD/sssd/pull/719 Author: thalman Title: #719: ifp: extraAttributes is UnknownProperty Action: opened PR body: """ Attempting to get extraAttributes via SSSD's ifp fails. Here I uncomment interface function for extraAttributes. also right for querying

[SSSD] [sssd PR#712][opened] SSSCTL: user-checks does not show custom attributes

2018-12-11 Thread thalman
URL: https://github.com/SSSD/sssd/pull/712 Author: thalman Title: #712: SSSCTL: user-checks does not show custom attributes Action: opened PR body: """ "sssctl user-checks" also query the IFP interface of sssd. The problem is, that this command only displays POSIX us

[SSSD] [sssd PR#716][opened] CACHE: SSSD doesn't clear cache entries

2018-12-17 Thread thalman
URL: https://github.com/SSSD/sssd/pull/716 Author: thalman Title: #716: CACHE: SSSD doesn't clear cache entries Action: opened PR body: """ Once object is in cache it is refreshed when it is expired and requested by the system. Object ID is not checked before refresh, but c

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

2018-12-12 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

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

2018-12-12 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

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

2018-12-04 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories thalman commented: """ @simo5 I thought about it more and I get to the same conclusion, PR updated """ See the full comment at https://github.com/SSSD/sssd/p

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

2018-12-04 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: synchronized 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 From

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

2018-12-04 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: synchronized 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 From

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

2018-12-04 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories thalman commented: """ > > Or at least we should IMO add some backwards compatible handling when this > > patch makes it to fedora or RHEL otherwise admi

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

2018-12-05 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: synchronized 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 From

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

2018-12-04 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: synchronized 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 From

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

2018-12-04 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories thalman commented: """ there is now one extra commit with default for ad, just to be on the same page """ See the full comment at https://github.com/S

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

2018-12-05 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

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

2018-12-07 Thread thalman
URL: https://github.com/SSSD/sssd/pull/696 Author: thalman Title: #696: DYNDNS: SSSD does not batch DDNS update requests Action: synchronized 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 From

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

2018-12-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: opened PR body: """ For empty home directory in passwd file sssd returns "/". Sssd should respect system behaviour and return the same a

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

2018-12-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories thalman commented: """ > This should probably not be specific to the files provider. > An empty home directory is a valid value and should be returned as is if t

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

2018-12-03 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Title: #703: nss: sssd returns '/' for emtpy home directories thalman commented: """ > This should probably not be specific to the files provider. > An empty home directory is a valid value and should be returned as is if t

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

2018-12-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

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

2018-12-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/696 Author: thalman Title: #696: DYNDNS: SSSD does not batch DDNS update requests Action: synchronized 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 From

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

2018-12-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/696 Author: thalman Title: #696: DYNDNS: SSSD does not batch DDNS update requests Action: synchronized 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 From

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

2018-12-06 Thread thalman
URL: https://github.com/SSSD/sssd/pull/696 Author: thalman Title: #696: DYNDNS: SSSD does not batch DDNS update requests Action: synchronized 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 From

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

2018-12-10 Thread thalman
URL: https://github.com/SSSD/sssd/pull/703 Author: thalman Title: #703: nss: sssd returns '/' for emtpy home directories Action: synchronized 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 From

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

2018-11-29 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Author: thalman Title: #690: DYNDNS: Convert dyndns timer to be_ptask Action: synchronized 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 From

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

2018-11-29 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask thalman commented: """ > Where did you get this 60 seconds limit? I can not find it in original code: > > ```c > ad_dyndns_init: > if (period < 60) period =

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

2018-11-29 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask thalman commented: """ > Where did you get this 60 seconds limit? I can not find it in original code: > > ```c > ad_dyndns_init: > if (period < 60) period =

[SSSD] [sssd PR#690][-Changes requested] DYNDNS: Convert dyndns timer to be_ptask

2018-11-29 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

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

2018-11-30 Thread thalman
URL: https://github.com/SSSD/sssd/pull/677 Author: thalman Title: #677: pcre: port to pcre2 Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/677/head:pr677 git checkout pr677 From

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

2018-11-29 Thread thalman
URL: https://github.com/SSSD/sssd/pull/690 Title: #690: DYNDNS: Convert dyndns timer to be_ptask thalman commented: """ > Where did you get this 60 seconds limit? I can not find it in original code: > > ```c > ad_dyndns_init: > if (period < 60) period = 60; >

[SSSD] [sssd PR#696][-Changes requested] DYNDNS: SSSD does not batch DDNS update requests

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

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

2018-11-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/694 Author: thalman Title: #694: SSSCTL: user-show says that user is expired Action: synchronized 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 From

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

2018-11-22 Thread thalman
URL: https://github.com/SSSD/sssd/pull/694 Title: #694: SSSCTL: user-show says that user is expired thalman commented: """ Thanks for comments. PR updated. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#723][+Changes requested] MAN: Explicitly state that not all generic domain options are supported for the files provider

2019-01-07 Thread thalman
URL: https://github.com/SSSD/sssd/pull/723 Title: #723: MAN: Explicitly state that not all generic domain options are supported for the files provider Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To

[SSSD] [sssd PR#716][synchronized] CACHE: SSSD doesn't clear cache entries

2019-01-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/716 Author: thalman Title: #716: CACHE: SSSD doesn't clear cache entries Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/716/head:pr716 git checkout pr716 From

[SSSD] [sssd PR#716][comment] CACHE: SSSD doesn't clear cache entries

2019-01-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/716 Title: #716: CACHE: SSSD doesn't clear cache entries thalman commented: """ @jhrozek I fixed all minor comments. There is still question whether we move the check as you suggested. """ See the full comment at https:/

[SSSD] [sssd PR#723][+Accepted] MAN: Explicitly state that not all generic domain options are supported for the files provider

2019-01-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/723 Title: #723: MAN: Explicitly state that not all generic domain options are supported for the files provider Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe

[SSSD] [sssd PR#712][comment] SSSCTL: user-checks does not show custom attributes

2019-01-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/712 Title: #712: SSSCTL: user-checks does not show custom attributes thalman commented: """ > But this looks like I haven't gotten to your codepath yet. Did you see the > error locally as well? No, I did not see this error. It work

[SSSD] [sssd PR#722][comment] KCM: Deleting a non-existent ccache should not yield an error

2019-01-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/722 Title: #722: KCM: Deleting a non-existent ccache should not yield an error thalman commented: """ LGTM, ACK """ See the full comment at https://github.com/SSSD/sssd/pull/722#issuecomment-452259179 __

[SSSD] [sssd PR#722][+Accepted] KCM: Deleting a non-existent ccache should not yield an error

2019-01-08 Thread thalman
URL: https://github.com/SSSD/sssd/pull/722 Title: #722: KCM: Deleting a non-existent ccache should not yield an error Label: +Accepted ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#719][comment] ifp: extraAttributes is UnknownProperty

2019-01-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/719 Title: #719: ifp: extraAttributes is UnknownProperty thalman commented: """ Fixed test """ See the full comment at https://github.com/SSSD/sssd/pull/719#issuecomment-450834868 _

[SSSD] [sssd PR#719][-Changes requested] ifp: extraAttributes is UnknownProperty

2019-01-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/719 Title: #719: ifp: extraAttributes is UnknownProperty Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#790][-Changes requested] Lookahead resolving of host names

2019-04-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/790 Title: #790: Lookahead resolving of host names Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to

[SSSD] [sssd PR#790][comment] Lookahead resolving of host names

2019-04-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/790 Title: #790: Lookahead resolving of host names thalman commented: """ @jhrozek Code updated. Surprisingly test-linking was easy to fix. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#790][synchronized] Lookahead resolving of host names

2019-04-02 Thread thalman
URL: https://github.com/SSSD/sssd/pull/790 Author: thalman Title: #790: Lookahead resolving of host names Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/790/head:pr790 git checkout pr790 From

[SSSD] [sssd PR#775][synchronized] krb5: Write multiple dnsnames into kdc info file

2019-03-21 Thread thalman
URL: https://github.com/SSSD/sssd/pull/775 Author: thalman Title: #775: krb5: Write multiple dnsnames into kdc info file Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/775/head:pr775 git checkout pr775 From

[SSSD] [sssd PR#775][comment] krb5: Write multiple dnsnames into kdc info file

2019-03-21 Thread thalman
URL: https://github.com/SSSD/sssd/pull/775 Title: #775: krb5: Write multiple dnsnames into kdc info file thalman commented: """ Thanks @sumit-bose, Your fix is included now. """ See the full comment at https://github.com/SSSD/sssd/p

[SSSD] [sssd PR#790][synchronized] Lookahead resolving of host names

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/790 Author: thalman Title: #790: Lookahead resolving of host names Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/790/head:pr790 git checkout pr790 From

[SSSD] [sssd PR#785][comment] Providers: Delay online check on startup

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Title: #785: Providers: Delay online check on startup thalman commented: """ > btw reading the code, I don't understand why do we allow more than one > check_online request, do you understand the reason from the code? We don't.

[SSSD] [sssd PR#785][comment] Providers: Delay online check on startup

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Title: #785: Providers: Delay online check on startup thalman commented: """ > > > btw reading the code, I don't understand why do we allow more than one > > > check_online request, do you understand the reason from

[SSSD] [sssd PR#785][synchronized] Providers: Delay online check on startup

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Author: thalman Title: #785: Providers: Delay online check on startup Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/785/head:pr785 git checkout pr785 From

[SSSD] [sssd PR#785][synchronized] Providers: Delay online check on startup

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Author: thalman Title: #785: Providers: Delay online check on startup Action: synchronized To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/785/head:pr785 git checkout pr785 From

[SSSD] [sssd PR#785][comment] Providers: Delay online check on startup

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Title: #785: Providers: Delay online check on startup thalman commented: """ > I think this makes sense. There are some style issues (too long lines), but > that's trivial to fix. > > The most important question I have is:

[SSSD] [sssd PR#785][comment] Providers: Delay online check on startup

2019-03-26 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Title: #785: Providers: Delay online check on startup thalman commented: """ > The most important question I have is: Do you think it makes sense to > schedule multiple online checks? Maybe it could be done with delay > 1 only

[SSSD] [sssd PR#785][edited] WIP: Providers: Delay online check on startup

2019-03-21 Thread thalman
URL: https://github.com/SSSD/sssd/pull/785 Author: thalman Title: #785: WIP: Providers: Delay online check on startup Action: edited Changed field: title Original value: """ Providers: Delay online check on startup """ _

  1   2   3   4   5   >