Skip to content

Commit

Permalink
Add logs around usage tracker shutdown (#3391)
Browse files Browse the repository at this point in the history
Parsl shutdown logs are deliberately rich at shutdown time because this is an
incredibly hang prone part of the code-base.
See https://github.com/Parsl/parsl/labels/safe-exit

This PR adds logs bracketing the shutdown of the usage tracker, in line
with later shutdowns of the job status poller, executors and monitoring.
  • Loading branch information
benclifford committed Apr 23, 2024
1 parent 1a2d924 commit 7324e29
Showing 1 changed file with 2 additions and 0 deletions.
2 changes: 2 additions & 0 deletions parsl/dataflow/dflow.py
Original file line number Diff line number Diff line change
Expand Up @@ -1245,8 +1245,10 @@ def cleanup(self) -> None:
self._checkpoint_timer.close()

# Send final stats
logger.info("Sending end message for usage tracking")
self.usage_tracker.send_end_message()
self.usage_tracker.close()
logger.info("Closed usage tracking")

logger.info("Closing job status poller")
self.job_status_poller.close()
Expand Down

0 comments on commit 7324e29

Please sign in to comment.