github-actions[bot] opened a new pull request, #66684: URL: https://github.com/apache/airflow/pull/66684
When a trigger finishes, the supervisor uploads its log to the remote log store and then closes the local file descriptor. If `upload_to_remote()` raised (e.g., S3/GCS throttling, transient network error), `close()` was never called and the underlying BufferedWriter — plus its 8 KiB buffer and the open fd — leaked for every failed upload. Wrap the cleanup in try/except/finally so the fd is always closed, and log the upload failure instead of letting it propagate into `handle_requests`. Surfaced in discussion #65985. (cherry picked from commit a6798ab8d8311e98b1c29cad54bd51cffd5ccac4) Co-authored-by: Jarek Potiuk <[email protected]> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
