Skip to content

Conversation

jonasbardino
Copy link
Contributor

In order to prevent the rather verbose python tracebacks in the logs from common ssh/sftp scans to end up in our log monitoring.

@jonasbardino jonasbardino self-assigned this Aug 25, 2025
@jonasbardino
Copy link
Contributor Author

jonasbardino commented Aug 28, 2025

CI probably fails on rocky8 because the new paramiko Makefile used for patching is not included in edge or because we don't have the corresponding patch for RHEL/Rocky 8 and selected python there.

UPDATE: searching the web it sounds like its actually a docker limit of 127 steps in each layer we happen to hit on rocky8 builds:
https://support.domino.ai/support/s/article/Docker-Layers-failed-to-register-layer-max-depth-exceeded-1718868032014
It appears to build fine with podman on our rocky8 test deployment and we're going to ditch rocky8 soon anyway, so I don't think it's worth pursuing further.

@jonasbardino jonasbardino marked this pull request as ready for review August 28, 2025 13:44
@jonasbardino jonasbardino requested a review from a team August 28, 2025 13:44
@jonasbardino jonasbardino added the enhancement New feature or request label Aug 28, 2025
@jonasbardino jonasbardino added this to the Rocky9 Deployments milestone Aug 28, 2025
@jonasbardino jonasbardino force-pushed the adjust/paramiko-to-silence-scan-log-noise-if-patch-available branch from c299d20 to 565f47d Compare August 30, 2025 05:34
@jonasbardino
Copy link
Contributor Author

It looks like rebasing after merging PR #126 solved the CI issue.

@jonasbardino jonasbardino force-pushed the adjust/paramiko-to-silence-scan-log-noise-if-patch-available branch from 565f47d to 92f043e Compare September 24, 2025 15:24
@jonasbardino jonasbardino merged commit eb71554 into master Sep 26, 2025
4 checks passed
@jonasbardino jonasbardino deleted the adjust/paramiko-to-silence-scan-log-noise-if-patch-available branch September 26, 2025 10:35
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.

1 participant