-
Notifications
You must be signed in to change notification settings - Fork 233
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tests: Add a fast, reliable, accurate ssh module #6760
Conversation
cfaa593
to
a8747d3
Compare
c424fc6
to
33040d4
Compare
e32d085
to
1a98fc1
Compare
c511c1a
to
67707f3
Compare
c09d37d
to
6097393
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Couple suggestions for docstrings and changing bash code to python calls.
db762dc
to
47e02d9
Compare
For Alltest: For IPA: For AD: |
f72e8fc
to
1971669
Compare
1971669
to
bb0f0f5
Compare
Sssd tests seems to be failing with current ssh module without any reason.
bb0f0f5
to
ff55fd0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use requirements.txt for installing ssh2-python otherwise looks good.
Added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Pushed PR: #6760 |
No description provided.