[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread jhrozek
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures jhrozek commented: """ * master: .44bc6e8f49eec9e7ab9a952845bffcc0fd3b3a44 """ See the full comment at https://github.com/SSSD/sssd/pull/451#issuecomment-345060225

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures fidencio commented: """ Unfortunately, as PR #454 is not passing our CI and also has some comments to be addressed (which most likely won't happen till next week), let's have this PR merged ASAP

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures mzidek-rh commented: """ Here is the PR: https://github.com/SSSD/sssd/pull/454 """ See the full comment at https://github.com/SSSD/sssd/pull/451#issuecomment-344992710

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures mzidek-rh commented: """ @lslebodn yeah I see (just tried the patch)... the tests need to be modified and that part in the branch may not be even necessary... so forget the branch, I will push new

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread lslebodn
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures lslebodn commented: """ On (16/11/17 12:40), mzidek-rh wrote: >Here is the branch if interested to look at the scope, but as I said I need to >run it in the environment where it was failing before

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures mzidek-rh commented: """ Here is the branch if interested to look at the scope, but as I said I need to run it in the environment where it was failing before submitting PR

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread mzidek-rh
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures mzidek-rh commented: """ Sorry I forgot about this. I do have a patch for https://pagure.io/SSSD/sssd/issue/3563 in ordererd-string-arrays (I just pushed the branch to gh). But I only tested it on

[SSSD] [sssd PR#451][comment] BUILD: Disable tests with know failures

2017-11-16 Thread fidencio
URL: https://github.com/SSSD/sssd/pull/451 Title: #451: BUILD: Disable tests with know failures fidencio commented: """ Although the patch looks good and there's a strong motivation behind this (having our internal CI tests passing), I'd like to hear from @mzidek-rh about what's the status of