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

Write a technote describing our findings using Kafka for telescope control

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_middleware
    • Labels:
      None
    • Story Points:
      5
    • Epic Link:
    • Sprint:
      TSSW Sprint - Apr 11 - Apr 25, TSSW Sprint - Apr 25 - May 09
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Write a T&S tech note (TSTN) describing the Kafka version of ts_salobj and lessons learned, with a focus on performance and compatibility

        Attachments

          Issue Links

          relates to

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-33192 Create prototype salobj using Kafka

          • Done

            Activity

            Ascending order - Click to sort in descending order
            No builds found.
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            Permalink
            rowen Russell Owen added a comment - - edited

            Pull request: https://github.com/lsst-tstn/tstn-033/pull/1

            Rendered: https://tstn-033.lsst.io/v/DM-34488/index.html

            Show
            rowen Russell Owen added a comment - - edited Pull request: https://github.com/lsst-tstn/tstn-033/pull/1 Rendered: https://tstn-033.lsst.io/v/DM-34488/index.html
            rowen Russell Owen made changes -
            Link This issue relates to DM-33192 [ DM-33192 ]
            rowen Russell Owen made changes -
            Reviewers Dave Mills, Tiago Ribeiro [ dmills, tribeiro ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            Permalink
            tribeiro Tiago Ribeiro added a comment -

            Review done on GH, PR approved. Thank you very much for the detailed documentation page.

            I left some comments in the PR I hope you will take into account but, other than that, I think it is good to go.

            Show
            tribeiro Tiago Ribeiro added a comment - Review done on GH, PR approved. Thank you very much for the detailed documentation page. I left some comments in the PR I hope you will take into account but, other than that, I think it is good to go.
            tribeiro Tiago Ribeiro made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            aclements Andy Clements made changes -
            Epic Link DM-27719 [ 442124 ]
            Hide
            Permalink
            rowen Russell Owen added a comment -

            Thank you for a very helpful review. I made the following updates:

            Rework the Performance as follows:

            • Include our requirements.
            • Add the MTMount trackTarget command and updated the latency table to show latency when sending messages at 20 Hz, since our main latency requirement is for tracking commands at 20 Hz. (Latency when writing flat out is not very interesting, since it will go up if the reader is struggling to keep up. In particular, that made the DDS latency look artificially bad.)

            Remove all use of "I", often by replacing with "we" (since I dislike passive voice) and some rewriting to make it closer to that you suggested.

            Show
            rowen Russell Owen added a comment - Thank you for a very helpful review. I made the following updates: Rework the Performance as follows: Include our requirements. Add the MTMount trackTarget command and updated the latency table to show latency when sending messages at 20 Hz, since our main latency requirement is for tracking commands at 20 Hz. (Latency when writing flat out is not very interesting, since it will go up if the reader is struggling to keep up. In particular, that made the DDS latency look artificially bad.) Remove all use of "I", often by replacing with "we" (since I dislike passive voice) and some rewriting to make it closer to that you suggested.
            wvreeven Wouter van Reeven made changes -
            Sprint TSSW Sprint - Apr 11 - Apr 25 [ 1160 ] TSSW Sprint - Apr 11 - Apr 25, TSSW Sprint - Apr 25 - May 09 [ 1160, 1163 ]
            rowen Russell Owen made changes -
            Story Points 3 4
            rowen Russell Owen made changes -
            Story Points 4 5
            rowen Russell Owen made changes -
            Watchers Dave Mills, Russell Owen, Tiago Ribeiro [ Dave Mills, Russell Owen, Tiago Ribeiro ] Dave Mills, Michael Reuter, Russell Owen, Tiago Ribeiro [ Dave Mills, Michael Reuter, Russell Owen, Tiago Ribeiro ]
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            Hide
            Permalink
            rowen Russell Owen added a comment -

            I don't want to keep dragging this ticket out so I'll file a new ticket to put finishing touches on the document.

            Show
            rowen Russell Owen added a comment - I don't want to keep dragging this ticket out so I'll file a new ticket to put finishing touches on the document.

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Dave Mills, Tiago Ribeiro
              Watchers:
              Dave Mills, Michael Reuter, Russell Owen, Tiago Ribeiro
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.