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

lsstsw should symlink afwdata or allow an envvar

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: lsstsw
    • Labels:
      None
    • Story Points:
      5
    • Team:
      SQuaRE

      Description

      To reduce disk usage, it is very handy to be able to make build/afwdata and stack/afwdata/BLAH be symlinks. build/ is easy: just make the symlink and then never have touch it unless you rm your whole stack. stack/afwdata/BLAH is harder: each time you rebuild something that depends on afwdata, it will install a new copy of afwdata, which you'll have to manually remove and declare your symlink with a new tag.

      A couple of ways to make this more automatic:

      • lsstsw checks whether build/afwdata is a symlink, and if so just makes the new stack "install" a symlink to the same directory.
      • Check for some environment variable (e.g. AFWDATA_BASE_DIR or something) and if that exists, just make a symlink to it, or make a dummy eups table that points to that directory and don't put anything in stack at all.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            I've attached a short bash script that does the cleaning up. Modify the two directories at the top as appropriate, and run it after you rebuild lsstsw, and it will remove any new testdata directories and replace them with symlinks, in a way that I believe to be fairly safe.

            Show
            Parejkoj John Parejko added a comment - I've attached a short bash script that does the cleaning up. Modify the two directories at the top as appropriate, and run it after you rebuild lsstsw, and it will remove any new testdata directories and replace them with symlinks, in a way that I believe to be fairly safe.
            Hide
            rowen Russell Owen added a comment -

            Thanks John Parejko; this is wonderful. I have added a Python version which requires defining two env variables

            Show
            rowen Russell Owen added a comment - Thanks John Parejko ; this is wonderful. I have added a Python version which requires defining two env variables
            Hide
            tjenness Tim Jenness added a comment -

            Given that the big problem with afwdata installing multiple times has been fixed by DM-22305 (it no longer depends on sconsUtils so will only be reinstalled when it changes) I suggest we close this ticket as Won't Fix.

            Show
            tjenness Tim Jenness added a comment - Given that the big problem with afwdata installing multiple times has been fixed by DM-22305 (it no longer depends on sconsUtils so will only be reinstalled when it changes) I suggest we close this ticket as Won't Fix.
            Hide
            Parejkoj John Parejko added a comment -

            I've tweaked my bash script version so that it doesn't muck up the new conda environment and does the symlinking in the right place.

            Show
            Parejkoj John Parejko added a comment - I've tweaked my bash script version so that it doesn't muck up the new conda environment and does the symlinking in the right place.
            Hide
            Parejkoj John Parejko added a comment -

            Marking this won't fix, as there are a couple of scripts that take care of the symlinking for those that need it, and dependencies on big packages have been trimmed down.

            Show
            Parejkoj John Parejko added a comment - Marking this won't fix, as there are a couple of scripts that take care of the symlinking for those that need it, and dependencies on big packages have been trimmed down.

              People

              Assignee:
              frossie Frossie Economou
              Reporter:
              Parejkoj John Parejko
              Watchers:
              John Parejko, Meredith Rawls, Russell Owen, Scott Daniel, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.