[Bug]: AttributeError: 'EventStreamRuntime' object has no attribute 'sid' if runtime_extra_deps is not None

[grizxlyzx]

Is there an existing issue for the same bug?

• I have checked the existing issues.

Describe the bug and reproduction steps

• in config.toml, add field runtime_extra_deps = <any dockerfile command> under [sandbox]
• set log level to debug
• make run and start a task

backend will raise AttributeError: 'EventStreamRuntime' object has no attribute 'sid' from EventStreamRuntime.__init__

OpenHands Installation

Development workflow

OpenHands Version

main

Operating System

Linux

Logs, Errors, Screenshots, and Additional Context

In EventStreamRuntime's __init__:

......
      if self.config.sandbox.runtime_extra_deps:
          self.log(
              'debug',
              f'Installing extra user-provided dependencies in the runtime image: {self.config.sandbox.runtime_extra_deps}',
          )

      self.init_base_runtime(
          config,
          event_stream,
          sid,
          plugins,
          env_vars,
          status_callback,
          attach_to_existing,
          headless_mode,
      )
......

check runtime_extra_deps and log happens before initializing base class Runtime, but self.log() reads self.sid, which at the time is not initialized yet.

[github-actions]

OpenHands started fixing the issue! You can monitor the progress here.

[github-actions]

A potential fix has been generated and a draft PR #5330 has been created. Please review the changes.