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

Add conda compiler support to jenkins, lsstsw, and newinstall

    XMLWordPrintable

    Details

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

      Description

      This ticket will add a new compiler, simply called "conda-system", which denotes a conda (specifically conda-forge) provided compiler which is based around the GCC 7.2 toolchain.

      For tarballs, there should not be compiler conflicts as tarballs are stored under the conda environment ref, and fundamentally are compatible with that environment.

      Other names that were considered, but it doesn't seem important to distinguish between these.

      • conda-forge-gcc7 (denotes the gcc7 toolchain, which, for mac, means clang-9.0.1)
      • compilers-1.0.4 - this is the compilers metapackage version number, which is actually what’s installed, but that version number is arbitrary and changes every few months

        Attachments

          Issue Links

            Activity

            Hide
            bvan Brian Van Klaveren added a comment -

            Tim Jenness could you double check newinstall.sh changes - I asked Gabriele Comoretto [X] for a review there already
            Kian-Tat Lim Could you review jenkins-dm-jobs, ci-scripts, and the new docker-scipipe repo for centos 8. There is a change to docker-newinstall too, but that is effectively just removing devtoolset-8, so it could be a bit later.

            Show
            bvan Brian Van Klaveren added a comment - Tim Jenness could you double check newinstall.sh changes - I asked Gabriele Comoretto [X] for a review there already Kian-Tat Lim Could you review jenkins-dm-jobs, ci-scripts, and the new docker-scipipe repo for centos 8. There is a change to docker-newinstall too, but that is effectively just removing devtoolset-8, so it could be a bit later.
            Hide
            ktl Kian-Tat Lim added a comment -

            I've reviewed what was requested. It generally looks good, except for the Travis failures that need to be corrected.

            Show
            ktl Kian-Tat Lim added a comment - I've reviewed what was requested. It generally looks good, except for the Travis failures that need to be corrected.
            Hide
            bvan Brian Van Klaveren added a comment -

            Holding off on the merge to docker-newinstall for now, but all other things are merged.

            I've kicked off a CI job and mac has finished lsst_distrib - waiting for CentOS7/8 to finish.

            Show
            bvan Brian Van Klaveren added a comment - Holding off on the merge to docker-newinstall for now, but all other things are merged. I've kicked off a CI job and mac has finished lsst_distrib - waiting for CentOS7/8 to finish.
            Hide
            tjenness Tim Jenness added a comment -

            I just noticed this is still in review with my name against it. Is there still anything required of me?

            Show
            tjenness Tim Jenness added a comment - I just noticed this is still in review with my name against it. Is there still anything required of me?
            Hide
            bvan Brian Van Klaveren added a comment -

            nope - I realized there is one uncommitted merge I will double check before merging this though, so I think this can be put as reviewed.

            Show
            bvan Brian Van Klaveren added a comment - nope - I realized there is one uncommitted merge I will double check before merging this though, so I think this can be put as reviewed.
            Hide
            ktl Kian-Tat Lim added a comment -

            The base container for the newinstall container was never switched in the lsst-sqre Dockerfile, but it has been switched in the Jenkins build job, effectively the same thing.

            Show
            ktl Kian-Tat Lim added a comment - The base container for the newinstall container was never switched in the lsst-sqre Dockerfile, but it has been switched in the Jenkins build job, effectively the same thing.

              People

              Assignee:
              bvan Brian Van Klaveren
              Reporter:
              bvan Brian Van Klaveren
              Reviewers:
              Tim Jenness
              Watchers:
              Brian Van Klaveren, Kian-Tat Lim, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.