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

DM-36743 broke ap_verify

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_verify
    • Labels:
      None

      Description

      The dc2 dataset failed with:

      lsst.daf.butler.registry._exceptions.MissingDatasetTypeError: "Registry is missing an input dataset type DatasetType('fakes_goodSeeingRealBogusSources', {band, instrument, detector, physical_filter, visit}, Catalog)"
      

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            ap_verify always uses --register-dataset-types, so it's a bit strange that it would get this error. Possibly the pipeline graph isn't connected properly (i.e., it's an input but not an output)?

            Show
            krzys Krzysztof Findeisen added a comment - ap_verify always uses --register-dataset-types , so it's a bit strange that it would get this error. Possibly the pipeline graph isn't connected properly (i.e., it's an input but not an output)?
            Hide
            nima Nima Sedaghat added a comment -

            Pushed a fix.

            Show
            nima Nima Sedaghat added a comment - Pushed a fix.
            Show
            ebellm Eric Bellm added a comment - Jenkins run: https://ci.lsst.codes/blue/organizations/jenkins/stack-os-matrix/detail/stack-os-matrix/38737/pipeline
            Hide
            sullivan Ian Sullivan added a comment -

            Looks good. GitHub seems to be stuck on the linting checks, which might be due to GitHub problems today, but could also need to be unstuck with a new push. You can try to unstick the check by amending the previous commit message, and pushing again.

            Show
            sullivan Ian Sullivan added a comment - Looks good. GitHub seems to be stuck on the linting checks, which might be due to GitHub problems today, but could also need to be unstuck with a new push. You can try to unstick the check by amending the previous commit message, and pushing again.
            Hide
            nima Nima Sedaghat added a comment -

            New push didn't help the situation

            Show
            nima Nima Sedaghat added a comment - New push didn't help the situation
            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            Ian Sullivan, "new push" was needed back in the Travis days. With GitHub Actions, you can always force a check to rerun from its Details page.

            Show
            krzys Krzysztof Findeisen added a comment - - edited Ian Sullivan , "new push" was needed back in the Travis days. With GitHub Actions, you can always force a check to rerun from its Details page.
            Hide
            krzys Krzysztof Findeisen added a comment -

            I take it back – some of these checks are so corrupted, they can't be rerun.

            Show
            krzys Krzysztof Findeisen added a comment - I take it back – some of these checks are so corrupted, they can't be rerun.

              People

              Assignee:
              nima Nima Sedaghat
              Reporter:
              ktl Kian-Tat Lim
              Reviewers:
              Ian Sullivan
              Watchers:
              Eric Bellm, Ian Sullivan, Kian-Tat Lim, Krzysztof Findeisen, Nima Sedaghat
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.