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

ap_verify CI command line broken

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_verify
    • Labels:
      None
    • Story Points:
      1
    • Sprint:
      AP F20-2 (July)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      After merging DM-21915, ap_verify CI runs fail with the following error:

      ap_verify.py: error: the following arguments are required: --output
      

      We believe this is from the addition of an extra argument (--gen2), with a shell comment that got parsed as an argument instead. Moving the comment to its own line should fix the problem.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            John Swinbank, would you be willing to review this?

            Show
            krzys Krzysztof Findeisen added a comment - John Swinbank , would you be willing to review this?
            Hide
            swinbank John Swinbank added a comment -

            Looks good; thanks for the prompt fix.

            Show
            swinbank John Swinbank added a comment - Looks good; thanks for the prompt fix.

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              John Swinbank
              Watchers:
              John Swinbank, Kian-Tat Lim, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.