[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-15 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing lslebodn commented: """ On (15/03/17 04:05), mzidek-rh wrote: >I know this is a corner case, but I would like sssctl config-check to behave >more user friendly in the case when config file

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-15 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing mzidek-rh commented: """ I know this is a corner case, but I would like sssctl config-check to behave more user friendly in the case when config file is missing for whatever reason, it is

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing jhrozek commented: """ if this patch is already written and you'd ack it otherwise then yes. """ See the full comment at https://github.com/SSSD/sssd/pull/194#issuecomment-286547773

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing lslebodn commented: """ BTW; I've just realized that missing configuration file is not a problem since commit a4837791f62283079e7be4b17efb769be8b2dfd1. @jhrozek Do we want this patch just

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing mzidek-rh commented: """ On 03/14/2017 06:08 PM, lslebodn wrote: > *@lslebodn* commented on this pull request. > > > >

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing mzidek-rh commented: """ New version attached. """ See the full comment at https://github.com/SSSD/sssd/pull/194#issuecomment-286493411 ___

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing mzidek-rh commented: """ I agree with the comments. See new version pushed. """ See the full comment at https://github.com/SSSD/sssd/pull/194#issuecomment-286488429

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing lslebodn commented: """ There are few inline comments """ See the full comment at https://github.com/SSSD/sssd/pull/194#issuecomment-286485973

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread celestian
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing celestian commented: """ LGTM and I pushed it to CI test. """ See the full comment at https://github.com/SSSD/sssd/pull/194#issuecomment-286474277

[SSSD] [sssd PR#194][comment] config-check: Message when sssd.conf is missing

2017-03-14 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/194 Title: #194: config-check: Message when sssd.conf is missing mzidek-rh commented: """ To test just remove the sssd.conf file and do sssctl config-check. """ See the full comment at https://github.com/SSSD/sssd/pull/194#issuecomment-286467249