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

Add a CI Jekinsfile to the config packages, and update pre-commit config

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Sprint:
      TSSW Sprint - Jun 6 - Jun 20
    • Team:
      Telescope and Site
    • Urgent?:
      No

      Description

      Add a continuous integration Jenkinsfile to the ts_config packages and update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages.

      Make the Jenkinsfile work whether or not the CSC packages and dependencies already exist in the lsst-develop Docker image. Use a bash function for this, to cut down on repetition.

      Add webhooks to the repositories on github so the continuous integration Jenkins jobs run.

        Attachments

          Issue Links

            Activity

            No builds found.
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Status To Do [ 10001 ] In Progress [ 3 ]
            Hide
            rowen Russell Owen added a comment - - edited

            Pull requests:

            Notes:

            • ts_config_eas is not presently needed. It only contains configuration for AdamSensors, which no longer exists. ESS configuration is in ts_config_ocs. Talk to Tiago Ribeiro to decide what to do about the package. I did partially update the package using https://github.com/lsst-ts/ts_config_eas/pull/6 but do not plan to merge it unless we decide we want it. (I did not add a Jenkinsfile because it has no tests to run).
            • ts_config_mttcs is waiting until ts_mtmount DM-35193 is merged.
            Show
            rowen Russell Owen added a comment - - edited Pull requests: https://github.com/lsst-ts/ts_config_atcalsys/pull/9 https://github.com/lsst-ts/ts_config_attcs/pull/25 https://github.com/lsst-ts/ts_config_latiss/pull/22 https://github.com/lsst-ts/ts_config_mttcs/pull/36 https://github.com/lsst-ts/ts_config_ocs/pull/82 Notes: ts_config_eas is not presently needed. It only contains configuration for AdamSensors, which no longer exists. ESS configuration is in ts_config_ocs. Talk to Tiago Ribeiro to decide what to do about the package. I did partially update the package using https://github.com/lsst-ts/ts_config_eas/pull/6 but do not plan to merge it unless we decide we want it. (I did not add a Jenkinsfile because it has no tests to run). ts_config_mttcs is waiting until ts_mtmount DM-35193 is merged.
            rowen Russell Owen made changes -
            Summary Update .pre-commit-config.yaml for the config packages Add a CI Jekinsfile to the config packages, and update pre-commit config
            rowen Russell Owen made changes -
            Description Update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages. Add a continuous integration Jenkinsfile to the ts_config packages and update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages.
            rowen Russell Owen made changes -
            Story Points 0 2
            rowen Russell Owen made changes -
            Description Add a continuous integration Jenkinsfile to the ts_config packages and update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages. Add a continuous integration Jenkinsfile to the ts_config packages and update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages.

            Make the Jenkinsfile work whether or not the CSC packages and dependencies already exist in the lsst-develop Docker image. Use a bash function for this, to cut down on repetition,
            rowen Russell Owen made changes -
            Story Points 2 1
            rowen Russell Owen made changes -
            Story Points 1 2
            rowen Russell Owen made changes -
            Description Add a continuous integration Jenkinsfile to the ts_config packages and update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages.

            Make the Jenkinsfile work whether or not the CSC packages and dependencies already exist in the lsst-develop Docker image. Use a bash function for this, to cut down on repetition,
            Add a continuous integration Jenkinsfile to the ts_config packages and update the software versions in .pre-commit-config.yaml for the ts_config_x packages and a few other packages.

            Make the Jenkinsfile work whether or not the CSC packages and dependencies already exist in the lsst-develop Docker image. Use a bash function for this, to cut down on repetition.

            Add webhooks to the repositories on github so the continuous integration Jenkins jobs run.
            rowen Russell Owen made changes -
            Watchers Russell Owen [ Russell Owen ] Russell Owen, Wouter van Reeven [ Russell Owen, Wouter van Reeven ]
            rowen Russell Owen made changes -
            Reviewers Wouter van Reeven [ wvreeven ]
            Status In Progress [ 3 ] In Review [ 10004 ]
            Hide
            wvreeven Wouter van Reeven added a comment -

            All PRs approved.

            Show
            wvreeven Wouter van Reeven added a comment - All PRs approved.
            wvreeven Wouter van Reeven made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            rowen Russell Owen made changes -
            Link This issue relates to DM-35212 [ DM-35212 ]
            rowen Russell Owen made changes -
            Comment [ Pull requests:
            * https://github.com/lsst-ts/ts_config_atcalsys/pull/9
            * https://github.com/lsst-ts/ts_config_attcs/pull/25
            * https://github.com/lsst-ts/ts_config_latiss/pull/22
            * https://github.com/lsst-ts/ts_config_mttcs/pull/36 (requires an update to ts_mtmount for tests to pass; this is in review)
            * https://github.com/lsst-ts/ts_config_ocs/pull/82

            Also I started work on ts_config_eas (https://github.com/lsst-ts/ts_config_eas/pull/6) but it only had AdamSensors config in it, and that SAL component is gone, so the package is not useful. I've left the ticket branch but I suggest we just drop this package unless [~tribeiro] wants to retain it for something else in the future.
            ]
            rowen Russell Owen made changes -
            Link This issue mitigates DM-35249 [ DM-35249 ]
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Reviewers:
              Wouter van Reeven
              Watchers:
              Russell Owen, Wouter van Reeven
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.