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

Make CSC.start more deterministic and easer to use

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Sprint:
      TSSW Sprint - Apr 11 - Apr 25
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      There is a sequencing issue in BaseCsc.start: it waits for the SalInfo to be started, then writes some topics and then transitions to the desired initial state. But it is common for subclasses to want to write topics before those state transitions occur.

      Fix this by breaking start into two pieces: start and start_phase2. Move the handling of initial state to phase2.

      Also improve SalLogHandler so messages can be logged from background threads.

        Attachments

          Activity

          Hide
          rowen Russell Owen added a comment -
          Show
          rowen Russell Owen added a comment - Pull request: https://github.com/lsst-ts/ts_salobj/pull/233
          Hide
          wvreeven Wouter van Reeven added a comment -

          Reviewed on GitHub. Please have a look at why the build failed and fix.

          Show
          wvreeven Wouter van Reeven added a comment - Reviewed on GitHub. Please have a look at why the build failed and fix.
          Hide
          rowen Russell Owen added a comment -

          Merged to develop. I'll postpone release until I know if it's to be included in cycle 25

          Show
          rowen Russell Owen added a comment - Merged to develop. I'll postpone release until I know if it's to be included in cycle 25

            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.