[autobackport: sssd-2-12] adding sss_ssh_knownhosts test case#8493
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a system test for sss_ssh_knownhosts. The new test contains a syntax error that will prevent it from running. Additionally, it introduces dependencies on external network services, which can cause test flakiness. I've provided a suggestion to fix the syntax and remove the external dependencies to improve test reliability.
Reviewed-by: Anuj Borah <aborah@redhat.com> Reviewed-by: Alejandro López <allopez@redhat.com> (cherry picked from commit b4e88e8)
|
The pull request was accepted by @danlavu with the following PR CI status: 🟢 rpm-build:centos-stream-10-x86_64:upstream (success) There are unsuccessful or unfinished checks. Make sure that the failures are not related to this pull request before merging. |
sssd-bot
force-pushed
the
SSSD-sssd-backport-pr8448-to-sssd-2-12
branch
from
618ff6a to
2a23b93
Compare
|
@aborah-sudo ping |
4 participants
This is an automatic backport of PR#8448 adding sss_ssh_knownhosts test case to branch sssd-2-12, created by @danlavu.
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
b4e88e8 - adding sss_ssh_knownhosts test case
Backported commits
Original Pull Request Body