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

Make StorageClass configurable in Butler Datastore

    Details

      Description

      Currently StorageClass subclasses (e.g. Exposure) are defined in code. Move these to YAML configuration (e.g. as done for Formatter).

        Attachments

          Issue Links

            Activity

            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Note that for the redesign of Registry I'm assuming StorageClass will appear as a simple string field. Although for Butler it may have to be a class to support assemble.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Note that for the redesign of Registry I'm assuming StorageClass will appear as a simple string field. Although for Butler it may have to be a class to support assemble .
            Hide
            tjenness Tim Jenness added a comment -

            Pim Schellart [X] I think this does what we want. StorageClasses are now read from YAML and become real classes.

            Show
            tjenness Tim Jenness added a comment - Pim Schellart [X] I think this does what we want. StorageClasses are now read from YAML and become real classes.
            Hide
            pschella Pim Schellart [X] (Inactive) added a comment -

            Looks good! See comments on PR.

            Show
            pschella Pim Schellart [X] (Inactive) added a comment - Looks good! See comments on PR.
            Hide
            tjenness Tim Jenness added a comment -

            I made all the requested changes and have now merged.

            Show
            tjenness Tim Jenness added a comment - I made all the requested changes and have now merged.

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                pschella Pim Schellart [X] (Inactive)
                Reviewers:
                Pim Schellart [X] (Inactive)
                Watchers:
                Jim Bosch, Pim Schellart [X] (Inactive), Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel