-
Notifications
You must be signed in to change notification settings - Fork 23
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
Misc fixes #181
base: master
Are you sure you want to change the base?
Misc fixes #181
Conversation
Certain platforms like Android do not have permission to list all child processes. In this instance, just continue
DSA has been removed from openssh for some time now, and this causes running tests on newer machines to fail
@dwoz How do you feel about removing dsa keys from salt-factories ? |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #181 +/- ##
==========================================
- Coverage 85.27% 83.98% -1.28%
==========================================
Files 97 97
Lines 5977 5705 -272
Branches 636 632 -4
==========================================
- Hits 5096 4791 -305
- Misses 705 721 +16
- Partials 176 193 +17
Flags with carried forward coverage won't be shown. Click here to find out more.
|
@dead10ck Even though the failures may not be due to your changes, can you look into why the tests are failing, wondering about that 3006.0 and 3007.0 given those are not the most recent versions of those branches |
@dead10ck Plan tp merge in your changes, but have to fix up the GitHub Actions , for example: Python 3.7 and 3.8, need to go, both EOL. Once that is fixed, will update / refresh this PR and it should build / test fine. |
Certain platforms like Android do not have permission to list all child processes. In this instance, just continue
DSA has been removed from openssh for some time now, and this causes running tests on newer machines to fail