Skip to content

Commit

Permalink
Update BaseMakeCalibrations.callpipetask to remove a call to ack.prin…
Browse files Browse the repository at this point in the history
…t_vars.

This call is not really doing anything and the method is no longer available in kafka version of ts-salobj.
  • Loading branch information
tribeiro authored and edennihy committed Oct 7, 2024
1 parent 0e38143 commit 348d632
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions python/lsst/ts/externalscripts/base_make_calibrations.py
Original file line number Diff line number Diff line change
Expand Up @@ -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.
Expand All @@ -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
Expand Down

0 comments on commit 348d632

Please sign in to comment.