The GitHub Actions job "Tests (AMD)" on 
airflow.git/fix-triggerer-logger-fd-leak-on-upload-failure has succeeded.
Run started by GitHub user potiuk (triggered by potiuk).

Head commit for run:
9094b1da190870d1b437754076f3d8ce163f4477 / Jarek Potiuk <[email protected]>
Fix triggerer file handle leak when remote log upload fails

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.

Report URL: https://github.com/apache/airflow/actions/runs/25641835957

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to