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

Add Gen3 conversion scripting and tests to ci_hsc

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ci_hsc
    • Labels:
    • Story Points:
      2
    • Sprint:
      BG3_S18_05
    • Team:
      Data Release Production

      Description

      Add gen3 conversion to the ci_hsc SCons build and test that we can use a Gen3 Butler to get Datasets put with a Gen2 Butler.

        Attachments

          Activity

          Hide
          jbosch Jim Bosch added a comment -

          Pim Schellart [X], lots of fixes here for the ci_hsc Gen3 conversion, including some you discovered first; figure it makes sense for you to review so you'll know what you're getting when you rebase on this.  Running "scons" in ci_hsc will now run the Gen3 conversion scripts and repeat the vast majority of the output validation tests that were already in ci_hsc, now using a Gen3 Butler (with no shim).

           

          Show
          jbosch Jim Bosch added a comment - Pim Schellart [X] , lots of fixes here for the ci_hsc Gen3 conversion, including some you discovered first; figure it makes sense for you to review so you'll know what you're getting when you rebase on this.  Running "scons" in ci_hsc will now run the Gen3 conversion scripts and repeat the vast majority of the output validation tests that were already in ci_hsc, now using a Gen3 Butler (with no shim).  
          Hide
          jbosch Jim Bosch added a comment -

          I expect this to land after DM-14191, and I imagine there will be some additional commits on this ticket to both be compatible with and take advantage of those changes.  I don't think that would negate the utility of reviewing what's here, though.

          Show
          jbosch Jim Bosch added a comment - I expect this to land after DM-14191 , and I imagine there will be some additional commits on this ticket to both be compatible with and take advantage of those changes.  I don't think that would negate the utility of reviewing what's here, though.
          Hide
          pschella Pim Schellart [X] (Inactive) added a comment -

          New updates on PR.

          Show
          pschella Pim Schellart [X] (Inactive) added a comment - New updates on PR.
          Hide
          jbosch Jim Bosch added a comment -

          Merged to master (or gen3-middleware, for obs_subaru and ci_hsc).

          Show
          jbosch Jim Bosch added a comment - Merged to master (or gen3-middleware, for obs_subaru and ci_hsc).

            People

            • Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              Pim Schellart [X] (Inactive)
              Watchers:
              Jim Bosch, Pim Schellart [X] (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel