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

Update MTMount to report connected and to monitor the telemetry connection

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      TSSW Sprint - Sep 26 - Oct 10
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      ts_mtmount used to report a connected event, which had separate fields for commands and replies. When we switched the API to use a single connection the CSC stopped publishing that outdated event.

      Update the connected event to have a single field "connected" and write it, but only if the new version is available (skip it if ts_xml has the old version).

      Also add a new telemetryConnected event and update the telemetry client to write it, if available. (We need two separate events because they are written by separate processes.)

      Also monitor the telemetry process and go to fault if telemetry is not being received in a timely fashion. This catches the telemetry server halting (which has been an issue).

        Attachments

          Issue Links

            Activity

            Hide
            rowen Russell Owen added a comment - - edited

            I have just updated the fall22 branch of ts_mtmount for now. I hope that I can wait to update the develop branch until we have released ts_xml 12.1, so I can omit the extra backwards-compatibility code (i.e. implement DM-36445 before back-porting).

            Pull requests:

            Show
            rowen Russell Owen added a comment - - edited I have just updated the fall22 branch of ts_mtmount for now. I hope that I can wait to update the develop branch until we have released ts_xml 12.1, so I can omit the extra backwards-compatibility code (i.e. implement DM-36445 before back-porting). Pull requests: https://github.com/lsst-ts/ts_mtmount/pull/84 https://github.com/lsst-ts/ts_xml/pull/622
            Hide
            wvreeven Wouter van Reeven added a comment -

            I am tentatively marking this ticket as Reviewed, because one test case is failing in Jenkins.

            Show
            wvreeven Wouter van Reeven added a comment - I am tentatively marking this ticket as Reviewed, because one test case is failing in Jenkins.
            Hide
            rowen Russell Owen added a comment -

            Thanks. I found and fixed the test issue.

            Merged to fall22 and tagged v0.22.3.alpha.101

            Show
            rowen Russell Owen added a comment - Thanks. I found and fixed the test issue. Merged to fall22 and tagged v0.22.3.alpha.101

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Wouter van Reeven
              Watchers:
              Russell Owen, Wouter van Reeven
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.