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

Add Configurable attribute to SALSubsystems XML

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ts_middleware
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Sprint:
      TSSW Sprint - Nov 25 - Dec 6, TSSW Sprint - Dec 9 - Dec 20
    • Team:
      Telescope and Site

      Description

      A new Configuration attribute should be added to the SALSubsystems.xml file. The values of the attribute should be a single character of Y if a CSC is configurable and N if not.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          aheyer Andrew Heyer [X] (Inactive) added a comment -

          Changes can be seen on this PR, I also threw in the changes for the RuntimeLanguages which was adding the definition for it.

          Show
          aheyer Andrew Heyer [X] (Inactive) added a comment - Changes can be seen on this PR, I also threw in the changes for the RuntimeLanguages which was adding the definition for it.
          Hide
          Permalink
          aheyer Andrew Heyer [X] (Inactive) added a comment -

          https://github.com/lsst-ts/ts_xml/pull/189

           

          Show
          aheyer Andrew Heyer [X] (Inactive) added a comment - https://github.com/lsst-ts/ts_xml/pull/189  
          Hide
          Permalink
          ecoughlin Eric Coughlin added a comment -

          Reviewed in GitHub.

          Show
          ecoughlin Eric Coughlin added a comment - Reviewed in GitHub.
          Hide
          Permalink
          rbovill Rob Bovill added a comment -

          I don't see any tests for the new <Configurable> tag.  Please add those before finalizing this work.

          Show
          rbovill Rob Bovill added a comment - I don't see any tests for the new <Configurable> tag.  Please add those before finalizing this work.
          Hide
          Permalink
          aheyer Andrew Heyer [X] (Inactive) added a comment -

          Branch merged here https://github.com/lsst-ts/ts_xml/commit/032d6bb4956083f9e7967b9aa788979bf05eafd0

          Show
          aheyer Andrew Heyer [X] (Inactive) added a comment - Branch merged here  https://github.com/lsst-ts/ts_xml/commit/032d6bb4956083f9e7967b9aa788979bf05eafd0

            People

            Assignee:
            aheyer Andrew Heyer [X] (Inactive)
            Reporter:
            mareuter Michael Reuter
            Reviewers:
            Eric Coughlin
            Watchers:
            Andrew Heyer [X] (Inactive), Eric Coughlin, Michael Reuter, Rob Bovill
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.