Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-25905

Add script heartbeat monitoring to ScriptQueueCommander

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      TSSW Sprint - Jul 20 - Aug 3, TSSW Sprint - Aug 3 - Aug 17, TSSW Sprint - Aug 17 - Aug 31, TSSW Sprint - Aug 31 - Sep 14, TSSW Sprint - Sep 14 - Sep 28
    • Team:
      Telescope and Site

      Description

      In DM-25639 I replaced the old command-line script launcher with one based on lsst.ts.salobj.CscCommander. However, this lost two important features: the script heartbeat monitor and the ability to retrieve old commands with up-arrow.

      Add heartbeat monitoring. Also continue to think about ways to add command-line history.
      This looks like it might do the job: https://github.com/vxgmichel/aioconsole. The input function call can be configured to support command history and editing and if that technique works for ainput then we're good.

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment -

            I chose to only monitor the currently running script. I felt that would result in the least noise and also be simpler to code than having one timer per queued script, in addition to the currently running script.

            I tested it by running the attached script.

            Pull request: https://github.com/lsst-ts/ts_scriptqueue/pull/55

            Show
            rowen Russell Owen added a comment - I chose to only monitor the currently running script. I felt that would result in the least noise and also be simpler to code than having one timer per queued script, in addition to the currently running script. I tested it by running the attached script. Pull request: https://github.com/lsst-ts/ts_scriptqueue/pull/55
            Hide
            rowen Russell Owen added a comment -

            Reviewed on github

            Show
            rowen Russell Owen added a comment - Reviewed on github
            Hide
            rowen Russell Owen added a comment -

            Released as v2.7.1

            Show
            rowen Russell Owen added a comment - Released as v2.7.1

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Tiago Ribeiro
              Watchers:
              Russell Owen, Tiago Ribeiro
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.