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

bin/deploy should fail when curl fails to find appropriate certificate

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: To Do
    • Resolution: Unresolved
    • Fix Version/s: None
    • Component/s: lsstsw
    • Labels:
      None
    • Team:
      Architecture

      Description

      heather999 reports on GitHub:

      When using lsstsw/bin/deploy at NERSC and at Harvard, I typically lack administrative access and have trouble with the curl commands for git and eups due to the missing certificate, as mentioned here:
      https://confluence.lsstcorp.org/display/LSWUG/Building+the+LSST+Stack+from+Source
      It's easy enough to fix by adding a --cacert parameter to the curl command. However, it is very difficult to spot the problem given the extensive output from ./bin/deploy. It would be much better if the script could report a hard failure due to the unsuccessful curl command(s). Otherwise, it may appear to a user that everything was set up, but then problems ensue when trying to use eups or git-lfs, for example.

        Attachments

          Activity

          Hide
          frossie Frossie Economou added a comment -

          Tim - is bin/deploy still something we recommend people do?

          Show
          frossie Frossie Economou added a comment - Tim - is bin/deploy still something we recommend people do?
          Hide
          tjenness Tim Jenness added a comment -

          We use bin/deploy all the time – it's how lsstsw works.

          Show
          tjenness Tim Jenness added a comment - We use bin/deploy all the time – it's how lsstsw works.
          Hide
          ktl Kian-Tat Lim added a comment -

          My in-development replacement for it will have better curl error handling.

          Show
          ktl Kian-Tat Lim added a comment - My in-development replacement for it will have better curl error handling.

            People

            Assignee:
            Unassigned Unassigned
            Reporter:
            swinbank John Swinbank
            Watchers:
            Frossie Economou, Heather Kelly, Kian-Tat Lim, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:

                Jenkins

                No builds found.