manager: remove preimport stage (#38381)

This commit is contained in:
Adeeb Shihadeh
2026-07-19 12:20:25 -07:00
committed by GitHub
parent 24893ebadb
commit caa9e770cc
3 changed files with 0 additions and 21 deletions
@@ -216,7 +216,6 @@ class ProcessContainer:
def _start_process(self):
if self.capture is not None:
self.process.launcher = LauncherWithCapture(self.capture, self.process.launcher) # ty: ignore[invalid-assignment] # intentional wrapper
self.process.prepare()
self.process.start()
def start(
-5
View File
@@ -87,11 +87,6 @@ def manager_init() -> None:
dirty=build_metadata.openpilot.is_dirty,
device=HARDWARE.get_device_type())
# preimport all processes
for p in managed_processes.values():
p.prepare()
def manager_cleanup() -> None:
# send signals to kill all procs
for p in managed_processes.values():
-15
View File
@@ -70,10 +70,6 @@ class ManagerProcess(ABC):
shutting_down = False
restart_if_crash = False
@abstractmethod
def prepare(self) -> None:
pass
@abstractmethod
def start(self) -> None:
pass
@@ -150,9 +146,6 @@ class NativeProcess(ManagerProcess):
self.sigkill = sigkill
self.launcher = nativelauncher
def prepare(self) -> None:
pass
def start(self) -> None:
# In case we only tried a non blocking stop we need to stop it before restarting
if self.shutting_down:
@@ -178,11 +171,6 @@ class PythonProcess(ManagerProcess):
self.launcher = launcher
self.restart_if_crash = restart_if_crash
def prepare(self) -> None:
if self.enabled:
cloudlog.info(f"preimporting {self.module}")
importlib.import_module(self.module)
def start(self) -> None:
# In case we only tried a non blocking stop we need to stop it before restarting
if self.shutting_down:
@@ -211,9 +199,6 @@ class DaemonProcess(ManagerProcess):
def should_run(started, params, CP):
return True
def prepare(self) -> None:
pass
def start(self) -> None:
if self.params is None:
self.params = Params()