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

Run ap_verify end-to-end in Gen 3

    Details

    • Story Points:
      6
    • Sprint:
      AP S20-6 (May), AP F20-1 (June), AP F20-2 (July), AP F20-3 (August)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      Once all the individual components support Gen 3, add a command-line option to ap_verify.py that ingests into a Gen 3 repository, then runs the AP pipeline and any metric tasks as Gen 3 products. Design the UI so that Gen 3 can naturally become the default behavior as Gen 2 is phased out.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment - - edited

            This is another ticket that sprawled out of control. I'd like to divide up the review three ways:

            Show
            krzys Krzysztof Findeisen added a comment - - edited This is another ticket that sprawled out of control. I'd like to divide up the review three ways: Nate Lust , could you review the changes to ctrl_mpexec , pipe_base , verify , ip_diffim , ap_association , and ap_pipe ? It's a total of 75 lines. Simon Krughoff , could you review the first three ap_verify commits ("Make Workspace...", "Create verification pipelines", "Rename runApPipe..."), for a total of 200 lines? Christopher Waters , could you review the last three ap_verify commits ("Add Gen 3 pipeline...", "Implement --gen3...", "Add pipeline override..."), for a total of 300 lines?
            Hide
            czw Christopher Waters added a comment -

            I've made a couple of minor comments on my changeset on github.  I'll remove myself from the reviewer list, but feel free to re-add me if other changes make what I reviewed out-of-date.

            Show
            czw Christopher Waters added a comment - I've made a couple of minor comments on my changeset on github.  I'll remove myself from the reviewer list, but feel free to re-add me if other changes make what I reviewed out-of-date.
            Hide
            nlust Nate Lust added a comment -

            I think I marked all my stuff reviewed and will take myself off as a reviewer. If I missed a pull request let me know and I will look at it. All looks good with one minor comment.

            Show
            nlust Nate Lust added a comment - I think I marked all my stuff reviewed and will take myself off as a reviewer. If I missed a pull request let me know and I will look at it. All looks good with one minor comment.
            Hide
            krughoff Simon Krughoff added a comment -

            Krzysztof Findeisen sorry for the delay. I have just a couple of comments. They are essentially clarifying comments. In general it looks good to me.

            Show
            krughoff Simon Krughoff added a comment - Krzysztof Findeisen sorry for the delay. I have just a couple of comments. They are essentially clarifying comments. In general it looks good to me.
            Hide
            krughoff Simon Krughoff added a comment -

            Since I'm the last reviewer, I'm marking as complete. Let me know if that's not what I should do.

            Show
            krughoff Simon Krughoff added a comment - Since I'm the last reviewer, I'm marking as complete. Let me know if that's not what I should do.

              People

              • Assignee:
                krzys Krzysztof Findeisen
                Reporter:
                krzys Krzysztof Findeisen
                Reviewers:
                Simon Krughoff
                Watchers:
                Christopher Waters, Krzysztof Findeisen, Nate Lust, Simon Krughoff
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel