Skip to content

Conversation

jonasbardino
Copy link
Contributor

@jonasbardino jonasbardino commented Aug 25, 2025

Port the old paramiko quiter patches to the 4.0.0 version we use in Rocky9 with our default pip upgraded paramiko.

Can be applied with

cd /
patch -p 1 < /path/to/mig/src/paramiko/silence-scan-noise_python-paramiko-4.0.0.el9_transport.py.diff

to avoid the

Ignoring potentially dangerous file name /usr/lib/python3.9/site-packages/paramiko/transport.py

and resulting manual path to patch prompt.

Included a similar version of the old 2.12.0 patch just against rocky 9 python version so that we can also patch the default non-upgraded paramiko there and a similar sample for Ubuntu 24.04 (noble) for local testing.

Added a Makefile to support e.g. simple make patch PLATFORM=el9 and make unpatch PLATFORM=el9 to handle automatic patching and reverse patch of the installed paramiko in a sane way on RHEL/Rocky 9. Similar results just replacing with PLATFORM=noble on Ubuntu 24.04.

@jonasbardino jonasbardino self-assigned this Aug 25, 2025
@jonasbardino jonasbardino added the enhancement New feature or request label Aug 25, 2025
@jonasbardino jonasbardino requested a review from a team August 25, 2025 12:38
@jonasbardino jonasbardino linked an issue Aug 25, 2025 that may be closed by this pull request
Copy link
Contributor

@Martin-Rehr Martin-Rehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approved

Fix order of existing el9 patch for paramiko 2.12.0 and add a similar patch
for ubuntu noble to ease local testing.
@jonasbardino jonasbardino changed the title Port old paramiko scan noise log silencer to recent 4.0.0 version Port old paramiko scan noise log silencer to recent platforms and versions Aug 25, 2025
@jonasbardino jonasbardino merged commit 88022d1 into next Aug 25, 2025
3 checks passed
@jonasbardino jonasbardino deleted the add/paramiko-quieter-patch-to-silence-scan-log-noise branch August 25, 2025 17:32
@jonasbardino jonasbardino mentioned this pull request Aug 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SFTP log noise
2 participants