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

Port repos.yaml tests to GitHub Actions

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: lsstsw
    • Labels:
      None
    • Team:
      Architecture
    • Urgent?:
      No

      Description

      There are transient issues with the Travis tests for repos.yaml (including repo spec: HTTP Error 429: too many requests). These may go away if we port to GitHub Actions.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          It turns out that these do not go away if we move to GitHub Actions, but a change to urllib3 with auto-retry does resolve the 429 problem.  Move to GitHub Actions anyway.  Also remove the obsolete git scheme for URLs, as git.lsstcorp.org no longer exists.

          Show
          ktl Kian-Tat Lim added a comment - It turns out that these do not go away if we move to GitHub Actions, but a change to urllib3 with auto-retry does resolve the 429 problem.  Move to GitHub Actions anyway.  Also remove the obsolete git scheme for URLs, as git.lsstcorp.org no longer exists.
          Hide
          Permalink
          ktl Kian-Tat Lim added a comment -

          Actual changes are relatively minor.  Note the modifications to .yamllint.yaml versus what is in daf_butler.

          I have turned off the Travis branch check in anticipation of merging.

          A full check takes about 3.5 minutes.  We could speed it up by starting to remove obsolete repos.

          Show
          ktl Kian-Tat Lim added a comment - Actual changes are relatively minor.  Note the modifications to .yamllint.yaml versus what is in daf_butler . I have turned off the Travis branch check in anticipation of merging. A full check takes about 3.5 minutes.  We could speed it up by starting to remove obsolete repos.
          Hide
          Permalink
          tjenness Tim Jenness added a comment -

          Looks good.

          Show
          tjenness Tim Jenness added a comment - Looks good.

            People

            Assignee:
            ktl Kian-Tat Lim
            Reporter:
            ktl Kian-Tat Lim
            Reviewers:
            Tim Jenness
            Watchers:
            Kian-Tat Lim, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.