[autobackport: sssd-2-12] ci: add TMT plan for passkey testing in PRCI#8727
[autobackport: sssd-2-12] ci: add TMT plan for passkey testing in PRCI#8727sssd-bot wants to merge 2 commits into
Conversation
Add comprehensive TMT plan for testing SSSD passkey functionality across IPA, LDAP, and Samba identity providers using containerized environments. Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Co-authored-by: Claude Sonnet 4 <noreply@anthropic.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Justin Stephenson <jstephen@redhat.com> (cherry picked from commit d54cf52)
Enable automated passkey testing on pull requests after COPR builds complete. Tests run on fedora-all and centos-stream-10 targets using the TMT plan. Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com> Co-Authored-By: Claude Sonnet 4 <noreply@anthropic.com> Reviewed-by: Jakub Vávra <jvavra@redhat.com> Reviewed-by: Justin Stephenson <jstephen@redhat.com> (cherry picked from commit 210f50f)
There was a problem hiding this comment.
Code Review
This pull request introduces a new TMT test plan for SSSD passkey functionality and updates the Packit configuration to execute these tests on Fedora and CentOS Stream 10. The review feedback suggests several improvements to the test plan's robustness: adding -o UserKnownHostsFile=/dev/null to SSH commands to prevent host key management issues in CI, updating the fallback COPR repository to reflect the correct backport branch, and ensuring the preparation phase fails immediately if Samba connectivity is not established to avoid wasting CI resources.
| COPR_REPO="$PACKIT_COPR_PROJECT" | ||
| else | ||
| echo "WARNING: No PACKIT_COPR_PROJECT found, using latest master build" | ||
| COPR_REPO="packit/SSSD-sssd-master" |
There was a problem hiding this comment.
The fallback COPR repository is hardcoded to master. Since this is a backport to the sssd-2-12 branch, the fallback should reflect the correct branch to ensure that manual runs or runs where PACKIT_COPR_PROJECT is missing don't accidentally install incompatible packages from the master branch.
COPR_REPO="packit/SSSD-sssd-2-12"
There was a problem hiding this comment.
This is correct and I will implement it
This is an automatic backport of PR#8672 ci: add TMT plan for passkey testing in PRCI to branch sssd-2-12, created by @ikerexxe.
Please make sure this backport is correct.
Note
The commits were cherry-picked without conflicts.
You can push changes to this pull request
Original commits
d54cf52 - tests: add TMT plan for passkey testing
210f50f - ci: add TMT passkey tests to packit workflow
Backported commits
Original Pull Request Body
Summary
This PR introduces testing infrastructure for SSSD passkey functionality using TMT and testing farm. It is designed to automate verification across diverse environments and integrate directly into the existing Packit CI workflow.
Changes Made
Test Plan
The TMT plan follows an automated lifecycle for each PR:
Constraints & known limitations
fedora-all, containers inherit the distribution of the host VM (e.g., F44 VM hosts F44 containers). This approach links test stability to the target distribution’s stability. Rawhide environments may occasionally experience failures due to upstream shifts unrelated to SSSD.Observability & debugging
If a test fails, kKey data can be found at:
tmt-logand test artifacts directory:plans/passkey/data/artifacts/tests/Final notes