Skip to content

Commit 9242ab2

Browse files
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.
1 parent 6c4c73e commit 9242ab2

2 files changed

Lines changed: 1 addition & 5 deletions

File tree

docs/agents/ld_monitor.rst

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -37,7 +37,7 @@ An example docker-compose configuration::
3737
image: simonsobs/socs:latest
3838
hostname: ocs-docker
3939
ports:
40-
- "1110:1110"
40+
- "1110:1110/udp"
4141
environment:
4242
- INSTANCE_ID=ld-monitor
4343
- SITE_HUB=ws://127.0.0.1:8001/ws

socs/agents/ld_monitor/agent.py

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -270,8 +270,6 @@ def init_ld_monitor(self, session, params=None):
270270
"{} is already running".format(self.lock.job))
271271
return False, "Could not acquire lock."
272272

273-
session.set_status('starting')
274-
275273
self._connect()
276274
if not self.initialized:
277275
return False, 'Could not connect to LD'
@@ -315,8 +313,6 @@ def acq(self, session, params=None):
315313
.format(self.lock.job))
316314
return False, "Could not acquire lock."
317315

318-
session.set_status('running')
319-
320316
self.take_data = True
321317

322318
session.data = {"fields": {}}

0 commit comments

Comments
 (0)