From 984b46098bd39180dcba1bd11345527702abe38f Mon Sep 17 00:00:00 2001
From: openhands <openhands@all-hands.dev>
Date: Thu, 14 Nov 2024 14:40:47 +0000
Subject: [PATCH] Fix file descriptor leak in LogBuffer when closing runtime

---
 openhands/runtime/impl/eventstream/eventstream_runtime.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/openhands/runtime/impl/eventstream/eventstream_runtime.py b/openhands/runtime/impl/eventstream/eventstream_runtime.py
index fab0cf0244..26cdb60f1d 100644
--- a/openhands/runtime/impl/eventstream/eventstream_runtime.py
+++ b/openhands/runtime/impl/eventstream/eventstream_runtime.py
@@ -111,6 +111,9 @@ class LogBuffer:
     def close(self, timeout: float = 5.0):
         self._stop_event.set()
         self.log_stream_thread.join(timeout)
+        # Close the log generator to release the file descriptor
+        if hasattr(self.log_generator, 'close'):
+            self.log_generator.close()
 
 
 class EventStreamRuntime(Runtime):