[SSSD] [sssd PR#591][comment] sss_obuscate: Fix traceback while reading space in sssd.conf

2018-06-11 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: sss_obuscate: Fix traceback while reading space in sssd.conf amitkumar50 commented: """ @jhrozek I corrected the Title. Issue is same as mentioned in upstream. If we place space before any option in sssd.conf. And run sss_obfuscate, it pro

[SSSD] [sssd PR#591][edited] sss_obuscate: Fix traceback while reading space in sssd.conf

2018-06-11 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/591 Author: amitkumar50 Title: #591: sss_obuscate: Fix traceback while reading space in sssd.conf Action: edited Changed field: title Original value: """ Remove spaces after comment delimiters in the example config file """ _

[SSSD] [sssd PR#591][+Changes requested] Remove spaces after comment delimiters in the example config file

2018-06-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: Remove spaces after comment delimiters in the example config file Label: +Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-de

[SSSD] [sssd PR#591][comment] Remove spaces after comment delimiters in the example config file

2018-06-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/591 Title: #591: Remove spaces after comment delimiters in the example config file jhrozek commented: """ I don't think the patch is correct, at the very least the title is confusing. I know the upstream ticket is also confusing and doesn't say what should

[SSSD] [sssd PR#579][comment] krb5 locator: add support for multiple addresses and sub-domains

2018-06-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/579 Title: #579: krb5 locator: add support for multiple addresses and sub-domains jhrozek commented: """ I think the commit that runs the offline callbacks is correct. The kdcinfo files are now removed. The code looks OK, Coverity is not complaining so I w

[SSSD] [sssd PR#579][-Changes requested] krb5 locator: add support for multiple addresses and sub-domains

2018-06-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/579 Title: #579: krb5 locator: add support for multiple addresses and sub-domains Label: -Changes requested ___ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-dev

[SSSD] Re: all tests fail in intgcheck

2018-06-11 Thread Chris Kowalczyk
On 06/11/2018 01:43 PM, Fabiano Fidêncio wrote: Chris, On Mon, Jun 11, 2018 at 1:10 PM, Chris Kowalczyk wrote: Hello, Do you have any idea what could be wrong with my environment? Please, join #sssd on freenode. I have an opensuse system here, I'll give it a try and get back to you. Thank

[SSSD] [sssd PR#596][comment] [CONFDB]:[RFE] Add "enabled" option to domain section

2018-06-11 Thread sumit-bose
URL: https://github.com/SSSD/sssd/pull/596 Title: #596: [CONFDB]:[RFE] Add "enabled" option to domain section sumit-bose commented: """ ok to test """ See the full comment at https://github.com/SSSD/sssd/pull/596#issuecomment-396220212 ___ sssd-devel

[SSSD] [sssd PR#596][comment] [CONFDB]:[RFE] Add "enabled" option to domain section

2018-06-11 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/596 Title: #596: [CONFDB]:[RFE] Add "enabled" option to domain section centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/596#issuecomment-396217822 _

[SSSD] [sssd PR#596][comment] [CONFDB]:[RFE] Add "enabled" option to domain section

2018-06-11 Thread centos-ci
URL: https://github.com/SSSD/sssd/pull/596 Title: #596: [CONFDB]:[RFE] Add "enabled" option to domain section centos-ci commented: """ Can one of the admins verify this patch? """ See the full comment at https://github.com/SSSD/sssd/pull/596#issuecomment-396217823 _

[SSSD] [sssd PR#596][opened] [CONFDB]:[RFE] Add "enabled" option to domain section

2018-06-11 Thread amitkumar50
URL: https://github.com/SSSD/sssd/pull/596 Author: amitkumar50 Title: #596: [CONFDB]:[RFE] Add "enabled" option to domain section Action: opened PR body: """ Upstream Request: Instead of enabling domains using the "domains" option in [sssd] section we could have [domain/*] option "enabled". Th

[SSSD] Re: all tests fail in intgcheck

2018-06-11 Thread Fabiano Fidêncio
Chris, On Mon, Jun 11, 2018 at 1:10 PM, Chris Kowalczyk wrote: > Hello, > > Do you have any idea what could be wrong with my environment? > Please, join #sssd on freenode. I have an opensuse system here, I'll give it a try and get back to you. > Sorry for bothering, but this blocks my PR .. No

[SSSD] Re: all tests fail in intgcheck

2018-06-11 Thread Chris Kowalczyk
Hello, Do you have any idea what could be wrong with my environment? Sorry for bothering, but this blocks my PR .. Regard, Chris On 06/07/2018 02:18 PM, Chris Kowalczyk wrote: Hello All, I have been trying to run sssd intgcheck for some time. Thanks to your help, there is some progress bu

[SSSD] [sssd PR#595][edited] sudo: allow use of default domain suffix (sssd-1-13)

2018-06-11 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/595 Author: pbrezina Title: #595: sudo: allow use of default domain suffix (sssd-1-13) Action: edited Changed field: title Original value: """ sudo: allow use of default domain suffix """ ___ sssd-devel mailin

[SSSD] [sssd PR#595][opened] sudo: allow use of default domain suffix

2018-06-11 Thread pbrezina
URL: https://github.com/SSSD/sssd/pull/595 Author: pbrezina Title: #595: sudo: allow use of default domain suffix Action: opened PR body: """ This patch is based on post 1.13 work. The purpose is to allow combination of fully qualified names and default domain suffix when user log in with a