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

Remove explicit buildbot dependency on datarel

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: buildbot, lsstDoxygen
    • Labels:
      None

      Description

      The buildbot scripts have an explicit dependency on the datarel package, which we'd like to remove from the stack. It uses datarel as the top-level product when building the cross-linked HTML documentation; lsstDoxygen's makeDocs script takes a single package, and generates the list of packages to include in the Doxygen build by finding all dependencies of that package.

      So, to remove the explicit dependency on datarel, we need to either:

      • find a new top-level product with a Doxygen build to pass to makeDocs (e.g. by adding a trivial Doxygen build to lsst_distrib)
      • modify the argument parsing in lsstDoxygen to take a list of multiple products (it looks like the limitation to one package is only in the argument parsing), and pass it a list of top-level products in the buildbot scripts.

      This is currently a blocker for DM-2928, which itself a blocker for DM-1766, which has now been lingering for a few weeks now. I'm going to look for other ways to remove the block on the latter, but I don't have a solution yet.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Done. DM-7298 now closed.

            Show
            tjenness Tim Jenness added a comment - Done. DM-7298 now closed.
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            Gabriele Comoretto [X] +1

             

            Tim Jenness I think DM-7298 can be invalidated now.

            Show
            jhoblitt Joshua Hoblitt added a comment - Gabriele Comoretto [X] +1   Tim Jenness I think DM-7298 can be invalidated now.
            Hide
            gcomoretto Gabriele Comoretto [X] (Inactive) added a comment -

            repos.yaml has been updated.

            Show
            gcomoretto Gabriele Comoretto [X] (Inactive) added a comment - repos.yaml has been updated.
            Hide
            tjenness Tim Jenness added a comment -

            Joshua Hoblitt I have removed datarel from lsst_distrib and your change to makedocs has worked. Feel free to close this ticket (that was opened nearly 3 years ago)

            Show
            tjenness Tim Jenness added a comment - Joshua Hoblitt I have removed datarel from lsst_distrib and your change to makedocs has worked. Feel free to close this ticket (that was opened nearly 3 years ago)
            Hide
            jhoblitt Joshua Hoblitt added a comment -

            OK - I have merged that PR. I'm currently re-run last nights release, so it will probably be several hours before this can be tested via release/run-rebuild.

            Show
            jhoblitt Joshua Hoblitt added a comment - OK - I have merged that PR. I'm currently re-run last nights release, so it will probably be several hours before this can be tested via release/run-rebuild .

              People

              Assignee:
              jhoblitt Joshua Hoblitt
              Reporter:
              jbosch Jim Bosch
              Watchers:
              Frossie Economou, Gabriele Comoretto [X] (Inactive), Jim Bosch, John Swinbank, Jonathan Sick, Joshua Hoblitt, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              7 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.