From 9242ab29e33d58b5910c5408a81dbe8b9b53dd84 Mon Sep 17 00:00:00 2001 From: Brian Koopman Date: Mon, 6 Jan 2025 13:44:20 -0600 Subject: [PATCH] ld-monitor: Remove `session.set_status()` calls and define protocol in compose example (#812) * Remove session.set_status calls, which are automatic now * Specify protocol for port mapping Docker otherwise assumes TCP, when we want UDP. --- docs/agents/ld_monitor.rst | 2 +- socs/agents/ld_monitor/agent.py | 4 ---- 2 files changed, 1 insertion(+), 5 deletions(-) diff --git a/docs/agents/ld_monitor.rst b/docs/agents/ld_monitor.rst index 61833b9cc..6072eec37 100644 --- a/docs/agents/ld_monitor.rst +++ b/docs/agents/ld_monitor.rst @@ -37,7 +37,7 @@ An example docker-compose configuration:: image: simonsobs/socs:latest hostname: ocs-docker ports: - - "1110:1110" + - "1110:1110/udp" environment: - INSTANCE_ID=ld-monitor - SITE_HUB=ws://127.0.0.1:8001/ws diff --git a/socs/agents/ld_monitor/agent.py b/socs/agents/ld_monitor/agent.py index f6d901c36..d331d86e4 100644 --- a/socs/agents/ld_monitor/agent.py +++ b/socs/agents/ld_monitor/agent.py @@ -270,8 +270,6 @@ def init_ld_monitor(self, session, params=None): "{} is already running".format(self.lock.job)) return False, "Could not acquire lock." - session.set_status('starting') - self._connect() if not self.initialized: return False, 'Could not connect to LD' @@ -315,8 +313,6 @@ def acq(self, session, params=None): .format(self.lock.job)) return False, "Could not acquire lock." - session.set_status('running') - self.take_data = True session.data = {"fields": {}}