zuul/zuul
Code Issues Proposed changes

Don't log tracebacks on streaming errors

Browse Source 
A remote disconnection is not unexpected in the log streamer.  Log
the error but not the traceback.

Change-Id: Iafe953aa2f644a59557334f61e04460a2dcdffde
This commit is contained in:
James E. Blair 2024-07-30 11:12:51 -07:00
parent 8f75869b6a
commit dbd1db370c
1 changed files with 3 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
zuul/lib/log_streamer.py
View File

@ -79,6 +79,9 @@ class RequestHandler(streamer_utils.BaseFingerRequestHandler):
try:
self.stream_log(log_file)
except (ConnectionResetError, BrokenPipeError) as e:
self.log.error("Streaming failure for build UUID %s: %s",
build_uuid, str(e))
except Exception:
self.log.exception("Streaming failure for build UUID %s:",
build_uuid)