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
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
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
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
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.
>
>
>
>
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
___
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
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
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
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
10 matches
Mail list logo