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

Make lsst_ci run obs_cfht, obs_decam example in its scons test.

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: Continuous Integration
    • Labels:
      None

      Description

      Make lsst_ci run obs_cfht, obs_decam examples in its scons test.

      The test step of lsst_ci should verify that the simple test example runs of obs_ packages succeed.

      Monitoring the results using validate_drp will be delayed for a later ticket. Just making sure things run successfully through processCcd.py or the equivalent is the goal here.

      1. [x] Add validation_data_cfht, validation_data_decam to required dependencies.
      2. [x] Run command-line based test scripts.
      3. [x] Wrap these scripts in scons test to report pass/fail based on just successfully completing.

        Attachments

          Issue Links

            Activity

            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -
            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Build on python3 submitted: https://ci.lsst.codes/job/stack-os-matrix/22273/
            Hide
            tjenness Tim Jenness added a comment -

            These changes look fine to me. Took me a while to realize that these are the first steps to be added to lsst_ci (which are the first tests in a meta package as far as I can tell). I don't think that's a problem though.

            Show
            tjenness Tim Jenness added a comment - These changes look fine to me. Took me a while to realize that these are the first steps to be added to lsst_ci (which are the first tests in a meta package as far as I can tell). I don't think that's a problem though.
            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -

            Tests such as these are largely the purpose of lsst_ci, in my conception of the package.

            It's also of course been useful as a pure meta package, just to make sure we have a good place to put some additional obs_* packages.

            But it will also remain a meta package in the sense that it doesn't actually install itself anywhere.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Tests such as these are largely the purpose of lsst_ci , in my conception of the package. It's also of course been useful as a pure meta package, just to make sure we have a good place to put some additional obs_* packages. But it will also remain a meta package in the sense that it doesn't actually install itself anywhere.
            Hide
            wmwood-vasey Michael Wood-Vasey added a comment -

            Merged to master.

            Show
            wmwood-vasey Michael Wood-Vasey added a comment - Merged to master.
            Hide
            tjenness Tim Jenness added a comment -

            Obs packages now have lsst_obs metapackage.

            Show
            tjenness Tim Jenness added a comment - Obs packages now have lsst_obs metapackage.

              People

              • Assignee:
                wmwood-vasey Michael Wood-Vasey
                Reporter:
                wmwood-vasey Michael Wood-Vasey
                Reviewers:
                Tim Jenness
                Watchers:
                Michael Wood-Vasey, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel