ci: Workaround pylibssh Failed to open session#8071
ci: Workaround pylibssh Failed to open session#8071justin-stephenson wants to merge 2 commits intoSSSD:masterfrom
Conversation
There was a problem hiding this comment.
Code Review
This pull request introduces a temporary workaround for an issue with pylibssh by installing a version from a personal fork. This is a reasonable short-term solution to unblock tests. My review includes a suggestion to pin the dependency to a specific commit hash instead of a branch. This will improve the security and reproducibility of your builds by ensuring you always use a specific, known version of the code.
7694aeb to
e88e28f
Compare
|
I guess this will not work with the |
|
It's just a warning. Not sure why pip is emitting it. Perhaps, it doesn't have access to the metadata in the place where the check is happening.. |
2b86a73 to
8f9e21a
Compare
|
PRCI is hanging and timing out at From https://github.com/SSSD/sssd/actions/runs/16894356542/job/47907649607 |
8f9e21a to
e5c6f6d
Compare
|
|
200a1d8 to
f11bd7f
Compare
|
I see the below error also in #8073 attempting to update CI to use python 3.13 (3.12 works however) Some issue with fetching a compatible tag for ruamel-yaml-clib?? https://github.com/SSSD/sssd/actions/runs/16917468308/job/47936151614?pr=8071 |
|
Obviously, there's no compatible wheels for ruamel.yaml.clib-0.1.0 (https://pypi.org/project/ruamel.yaml.clib/0.1.0/#files), but there's some for newer versions: https://pypi.org/project/ruamel.yaml.clib/0.2.12/#files |
064ff02 to
1b097a0
Compare
1b097a0 to
200118f
Compare
|
PR is now ready, moving out of draft status. |
200118f to
6fd2089
Compare
There was a problem hiding this comment.
I usually put a comment with guidelines for such currated constraint files. Example: https://github.com/ansible/awx-plugins/blob/12091e4/dependencies/direct/py-constraints.in#L1-L15
There was a problem hiding this comment.
I added the comment, thank you.
c004a95 to
d479ce4
Compare
12c1b6e to
f804a11
Compare
|
@justin-stephenson still not working |
|
You might need to pull in the tags into your pytest-mh fork. |
Add constraints.txt file to specify version requirements separate from installed packages
f804a11 to
32d07b6
Compare
Good to go now. CI failures are unrelated. |
|
Pushed PR: #8071
|
Workaround ansible pylibssh issue which causes test failures
pylibsshext.errors.LibsshChannelException: Failed to open_session: [-2]PR ansible/pylibssh#756 is under review but workaround it in the meantime.