lslebodn commented on a pull request
"""
On (09/09/16 05:50), mzidek-rh wrote:
>See the updated tests. It uses export and import and checks if override works
>after import.
>
master:
* 1c72723cde8bea0d390b928c7cd29e48e7a7deab
* 07e7683f5a86991feaa764e2055116554ada1b93
LS
"""
See the full comme
lslebodn commented on a pull request
"""
On (09/09/16 05:50), mzidek-rh wrote:
>See the updated tests. It uses export and import and checks if override works
>after import.
>
ACK
http://sssd-ci.duckdns.org/logs/job/53/24/summary.html
LS
"""
See the full comment at
https://github.com/SSSD/sss
lslebodn commented on a pull request
"""
On (09/09/16 05:50), mzidek-rh wrote:
>See the updated tests. It uses export and import and checks if override works
>after import.
>
ACK for test; just waiting for CI
LS
"""
See the full comment at
https://github.com/SSSD/sssd/pull/20#issuecomment-246
mzidek-rh commented on a pull request
"""
See the updated tests. It uses export and import and checks if override works
after import.
"""
See the full comment at
https://github.com/SSSD/sssd/pull/20#issuecomment-245903519
___
sssd-devel mailing list
s
lslebodn commented on a pull request
"""
On (09/09/16 03:27), mzidek-rh wrote:
>Btw. I am currently working on tests for some features we do not have covered
>in upstream CI. I can add test for the sss_overide after I am finished with
>that. But I would really prefer not to block this fix (pbrezin
mzidek-rh commented on a pull request
"""
Btw. I am currently working on tests for some features we do not have covered
in upstream CI. I can add test for the sss_overide after I am finished with
that. But I would really prefer not to block this fix (pbrezina confirmed it is
a regression, we ne
mzidek-rh commented on a pull request
"""
I agree that we need better tests for sss_override, but the purpose of this
test was to catch the error in the ticket so that we do not regress. We can
create a ticket to add more comprehensive tests for sss_overide export/import
as a separate effort.
"
lslebodn commented on a pull request
"""
On (07/09/16 10:26), mzidek-rh wrote:
>Here is a fix + CI test for https://fedorahosted.org/sssd/ticket/3179.
>
>Michal
>You can view, comment on, or merge this pull request online at:
>
> https://github.com/SSSD/sssd/pull/20
>
I have just few comments to