Merge pull request #168 from lsst-ts/tickets/DM-47890

lsst-ts · Dec 17, 2024 · 7c92106 · 7c92106
2 parents dd41a64 + aada3d2
commit 7c92106
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 4 deletions.
diff --git a/doc/news/DM-47890.feature.1.rst b/doc/news/DM-47890.feature.1.rst
@@ -0,0 +1 @@
+Move make_love_stress_tests received messages log from debug to info level.
diff --git a/doc/news/DM-47890.feature.rst b/doc/news/DM-47890.feature.rst
@@ -0,0 +1 @@
+Update BaseMakeCalibrations.callpipetask to remove a call to ack.print_vars.
diff --git a/python/lsst/ts/externalscripts/base_make_calibrations.py b/python/lsst/ts/externalscripts/base_make_calibrations.py
@@ -1002,15 +1002,12 @@ async def verify_calib(self, image_type, job_id_calib):
             f"Received acknowledgement of ocps command for {image_type} verification."
         )
 
-        ack.print_vars()
         job_id_verify = json.loads(ack.result)["job_id"]
 
         ack = await self.ocps.cmd_execute.next_ackcmd(ack)
         self.log.debug(
             f"Received command completion acknowledgement from ocps ({image_type})"
         )
-        if ack.ack != salobj.SalRetCode.CMD_COMPLETE:
-            ack.print_vars()
 
         while True:
             msg = await self.ocps.evt_job_result.next(

diff --git a/python/lsst/ts/externalscripts/make_love_stress_tests.py b/python/lsst/ts/externalscripts/make_love_stress_tests.py
@@ -224,7 +224,7 @@ async def run(self):
             await asyncio.sleep(self.loop_time_message_collection)
             for client in self.clients:
                 msg_count += len(client.msg_traces)
-            self.log.debug(
+            self.log.info(
                 f"Received {msg_count}/{self.config.number_of_messages} messages"
             )
         self.log.info(
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Move make_love_stress_tests received messages log from debug to info level.
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Update BaseMakeCalibrations.callpipetask to remove a call to ack.print_vars.