Merge pull request #148 from lsst-ts/tickets/DM-46458

Tickets/dm 46458: Support Summit Observing Weeks 39-40 of 2024
lsst-ts · Oct 7, 2024 · e5d199b · e5d199b
2 parents 0e38143 + 63dcd96
commit e5d199b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/doc/news/DM-46458.feature.rst b/doc/news/DM-46458.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
@@ -774,7 +774,6 @@ async def call_pipetask(self, image_type):
             f"Received acknowledgement of ocps command for {image_type} pipetask."
         )
 
-        ack.print_vars()
         job_id = json.loads(ack.result)["job_id"]
 
         # Wait for the command completion acknowledgement.
@@ -783,7 +782,9 @@ async def call_pipetask(self, image_type):
             f"Received command completion acknowledgement from ocps for {image_type}."
         )
         if ack.ack != salobj.SalRetCode.CMD_COMPLETE:
-            ack.print_vars()
+            self.log.debug(
+                f"OCPS job not complete, received {ack}. Continuing to wait for job result."
+            )
         # Wait for the job result message that matches the job id we're
         # interested in ignoring any others (from other remotes).
         # This needs to follow the first acknowledgement
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1 @@
		Update BaseMakeCalibrations.callpipetask to remove a call to ack.print_vars.