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

lsstsw should provide newer git-lfs

    XMLWordPrintable

    Details

    • Team:
      Architecture

      Description

      Apparently, git-lfs as provisioned by lsstsw does not behave as documented in our developer guide because it is too old. Specifically, heather999 reports on GitHub:

      Currently, bin/deploy installs git-lfs 1.0.2. This is rather old and does not include the more recent feature modification which is referenced in the DM Doc, namely "git lfs install":
      http://developer.lsst.io/en/latest/tools/git_lfs.html
      See: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3779 
      Further it seems the current DM configuration instructions do not work at all for git-lfs 1.0.2. Once I upgraded to 1.2.0 things were working in accordance to the DM documentation.

      Would it be possible to upgrade the version of git-lfs that ./bin/deploy installs so others do not run into similar problems?

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-6057 v12.0 [Winter 2016 / Extra 2016] release

          • Done
          is triggered by

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-4888 Update git-lfs documentation to work with git-lfs 1.1.0+

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            I think this was an unfortunate disconnect between updating the docs in DM-4469 and not updating the $LFS_VERSION entry in bin/deploy.

            Show
            tjenness Tim Jenness added a comment - I think this was an unfortunate disconnect between updating the docs in DM-4469 and not updating the $LFS_VERSION entry in bin/deploy .
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Frossie Economou I think this ticket should be a blocker for v12.0 release.

            Show
            tjenness Tim Jenness added a comment - Frossie Economou I think this ticket should be a blocker for v12.0 release.
            Hide
            Permalink
            tjenness Tim Jenness added a comment -

            Trivial change to bin/deploy to move us from 1.0.2 to v1.2.0 so that the documentation matches a version of LFS we are installing for people.

            Show
            tjenness Tim Jenness added a comment - Trivial change to bin/deploy to move us from 1.0.2 to v1.2.0 so that the documentation matches a version of LFS we are installing for people.
            Hide
            Permalink
            jhoblitt Joshua Hoblitt added a comment -

            Ack.

            Show
            jhoblitt Joshua Hoblitt added a comment - Ack.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              swinbank John Swinbank
              Reviewers:
              J Matt Peterson [X] (Inactive), Joshua Hoblitt
              Watchers:
              Heather Kelly, J Matt Peterson [X] (Inactive), John Swinbank, Jonathan Sick, Joshua Hoblitt, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.