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

Ignore the "metadata" field in config files in salobj

    XMLWordPrintable

    Details

    • Story Points:
      0
    • Epic Link:
    • Sprint:
      TSSW Sprint - Apr 12 - Apr 26, TSSW Sprint - Apr 26 - May 10
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Update BaseScript and ConfigurableCsc to strip the metadata dict, if present, from config files when reading configuration.

      This is in preparation to adding metadata which will be checked by unit tests but not used by salobj itself (at least not initially).

        Attachments

          Activity

          No builds found.
          rowen Russell Owen created issue -
          rowen Russell Owen made changes -
          Field Original Value New Value
          Sprint TSSW Sprint - Apr 12 - Apr 26 [ 1089 ]
          wvreeven Wouter van Reeven made changes -
          Sprint TSSW Sprint - Apr 12 - Apr 26 [ 1089 ] TSSW Sprint - Apr 12 - Apr 26, TSSW Sprint - Apr 26 - May 10 [ 1089, 1091 ]
          rowen Russell Owen made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          Hide
          rowen Russell Owen added a comment -
          Show
          rowen Russell Owen added a comment - Pull request: https://github.com/lsst-ts/ts_salobj/pull/191
          rowen Russell Owen made changes -
          Description Update ConfigurableCsc to strip the "metadata" field from configuration files when reading config as part of the {{start}} command.

          This is in preparation to adding metadata which will be checked by unit tests but not used by salobj itself (at least not initially).
          Update {{BaseScript}} and {{ConfigurableCsc}} to strip the {{metadata}} dict, if present, from config files when reading configuration.

          This is in preparation to adding metadata which will be checked by unit tests but not used by salobj itself (at least not initially).
          rowen Russell Owen made changes -
          Reviewers Eric Coughlin [ ecoughlin ]
          Status In Progress [ 3 ] In Review [ 10004 ]
          Hide
          ecoughlin Eric Coughlin added a comment -

          Reviewed in GitHub.

          Show
          ecoughlin Eric Coughlin added a comment - Reviewed in GitHub.
          ecoughlin Eric Coughlin made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Hide
          rowen Russell Owen added a comment -

          Released ts_salobj v6.3.6

          Show
          rowen Russell Owen added a comment - Released ts_salobj v6.3.6
          rowen Russell Owen made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]

            People

            Assignee:
            rowen Russell Owen
            Reporter:
            rowen Russell Owen
            Reviewers:
            Eric Coughlin
            Watchers:
            Eric Coughlin, Patrick Ingraham, Russell Owen, Tiago Ribeiro
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.