sss pac responder client#150
Conversation
|
Hi @pbrezina, to be able to the multi-threaded clients of the PAC responder I took related code form the SSSD tree and currently build ti as part of the Additionally, I wonder, since Thanks. bye, |
|
Hi, since this is part of the test and performs testing, I think it should be part of SSSD repository so we can change it if needed. The random.so is not used directly by tests, it is something that makes umockdev work. But in our previous discussions @jakub-vavra-cz did not want to install build dependencies when running in IdM CI to have minimal environment, so unless there is a move in this opinion, we need to put it here. Maybe we can just remove the sssd-ci-random package and rename it to sssd-ci[-test-deps]?
Looking at 9486575 it looks to be used in Ubuntu and Debian, these distros were now removed so it can be removed as well. |
48102cb to
87014e6
Compare
Hi, do I understand correctly that it would be better to rename the rpm in a separate PR and wait until it is committed and a new package is build in copr before it can be used here? bye, |
Yes, if we want to make CI green. But the RPM is tested in "Build RPMS" job, so I do not think it is necessary. |
Hi, so how to move forward, are you fine if this PR is reviewed as is or shall I split out the name change of the rpm into a different PR? bye, |
|
I am perfectly fine, I just waited to be ready for review. Ack. |
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150.
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150.
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150.
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]>
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]>
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c) (cherry picked from commit 47361fd)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c) (cherry picked from commit 47361fd)
This new system test is a replacment of the integration test with the same name. It depends on a test binary which was added to the SSSD CI containers in SSSD/sssd-ci-containers#150. Reviewed-by: Alexey Tikhonov <[email protected]> Reviewed-by: Jakub Vávra <[email protected]> (cherry picked from commit c78855c) (cherry picked from commit 47361fd)
2 participants
No description provided.