URL: https://github.com/SSSD/sssd/pull/708 Author: mzidek-rh Title: #708: TESTS: ldb-tools are required for multihost tests Action: opened
PR body: """ Some of the test do use ldbsearch.
Resolves: https://pagure.io/SSSD/sssd/issue/3894 """
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/708/head:pr708 git checkout pr708
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
mzidek-rh commented: """ The OSCI tests are running, will give update here when they finish. """
See the full comment at https://github.com/SSSD/sssd/pull/708#issuecomment-444619543
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
mzidek-rh commented: """ Hi, sorry for sending the PR too early. The patch actually does not work for RHEL (I think it could work for Fedora, but I fixed this in the yaml config file for the OSCI for now). The reason why the patch did not work is probably because I only added the ldb-tools to the branch that gets executed only for Fedora (and not to the elif branch with RHEL). So it could be probably fixed, but I do not see that much motivation for this PR as it is fixed in the yaml file already.
So, jhrozek, feel free to close this PR if you agree withe the above :) """
See the full comment at https://github.com/SSSD/sssd/pull/708#issuecomment-444808219
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
jhrozek commented: """ Well I do test on Fedora, so I care about the patch. The elif RHEL branch is just a bug, we need to fix the dependency detection better, but for now I would prefer to upstream the patch BUT please also add sssd-tools to the list (see a recent in rhpkg git..) """
See the full comment at https://github.com/SSSD/sssd/pull/708#issuecomment-445774981
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
Label: +Changes requested
URL: https://github.com/SSSD/sssd/pull/708 Author: mzidek-rh Title: #708: TESTS: ldb-tools are required for multihost tests Action: synchronized
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/708/head:pr708 git checkout pr708
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
mzidek-rh commented: """ I rebased this and added the sssd-tools as well.
I did not touch the RHEL 8 elif branch for now. """
See the full comment at https://github.com/SSSD/sssd/pull/708#issuecomment-504012086
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
Label: -Changes requested
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
Label: +Accepted
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
jhrozek commented: """ * master: 39686a584e1286366e0e34910074d45f88c8f4e6 """
See the full comment at https://github.com/SSSD/sssd/pull/708#issuecomment-508712538
URL: https://github.com/SSSD/sssd/pull/708 Author: mzidek-rh Title: #708: TESTS: ldb-tools are required for multihost tests Action: closed
To pull the PR as Git branch: git remote add ghsssd https://github.com/SSSD/sssd git fetch ghsssd pull/708/head:pr708 git checkout pr708
URL: https://github.com/SSSD/sssd/pull/708 Title: #708: TESTS: ldb-tools are required for multihost tests
Label: +Pushed
sssd-devel@lists.fedorahosted.org