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

Fix bug in dataset fetching due to DM-26244

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: qa_explorer
    • Labels:
      None
    • Story Points:
      1
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      The butler removal of DM-22255 required some minor hacking to reinstate the ability to write the output from pipe_analysis to a subdirectory such that the data ID now requires a "subdir" key.  Add this key to the matchVisits.py task in qa_explorer so that the tables it needs can be found.

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          4 older comments
          Hide
          Permalink
          emorganson Eric Morganson [X] (Inactive) added a comment -

          I thought I was on the ticket branch but was not. Then I tied myself in knots looking for a thing that didn't exist.

          Show
          emorganson Eric Morganson [X] (Inactive) added a comment - I thought I was on the ticket branch but was not. Then I tied myself in knots looking for a thing that didn't exist.
          Hide
          Permalink
          lauren Lauren MacArthur added a comment -

          Great (that it works, not the tying yourself in the knots part...!)  If/when you are satisfied with the changes, please hit the "Review Complete" button on this ticket and I'll merge to master.

          Show
          lauren Lauren MacArthur added a comment - Great (that it works, not the tying yourself in the knots part...!)  If/when you are satisfied with the changes, please hit the "Review Complete" button on this ticket and I'll merge to master .
          Hide
          Permalink
          emorganson Eric Morganson [X] (Inactive) added a comment -

          Yup. Waited for it to complete.

          Show
          emorganson Eric Morganson [X] (Inactive) added a comment - Yup. Waited for it to complete.
          Hide
          Permalink
          emorganson Eric Morganson [X] (Inactive) added a comment -

          matchVisits now works

          Show
          emorganson Eric Morganson [X] (Inactive) added a comment - matchVisits now works
          Hide
          Permalink
          lauren Lauren MacArthur added a comment -

          Thanks!  Merged to master.

          Show
          lauren Lauren MacArthur added a comment - Thanks!  Merged to master.

            People

            Assignee:
            lauren Lauren MacArthur
            Reporter:
            lauren Lauren MacArthur
            Reviewers:
            Eric Morganson [X] (Inactive)
            Watchers:
            Eric Morganson [X] (Inactive), Lauren MacArthur, Tim Morton [X] (Inactive), Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.