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

Support SQuaSH upload of arbitrary Job .json files

    Details

    • Type: Improvement
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: squash, verify
    • Labels:
      None

      Description

      Currently, the ap_verify CI job generates a hard-coded list of Job files. However, ap_verify is moving toward a system that will generate and persist Jobs semi-automatically, and there will be a transition period where both the hardcoded files and the new ones are in use.

      To avoid breaking SQuaSH uploads, it would be useful if the upload script or the CI system could search for Job files ("*.json" or "*.verify.json" come to mind) and upload them all.

        Attachments

          Issue Links

            Activity

            Hide
            afausti Angelo Fausti added a comment -

            Krzysztof Findeisen good news, looks like the `ap_verify` pipeline in CI already handles multiple verification job JSON files that way, as shown in the attached screenshot.

            So if you add more files to the same directory they should be uploaded as well.

            Show
            afausti Angelo Fausti added a comment - Krzysztof Findeisen good news, looks like the `ap_verify` pipeline in CI already handles multiple verification job JSON files that way, as shown in the attached screenshot. So if you add more files to the same directory they should be uploaded as well.
            Hide
            krzys Krzysztof Findeisen added a comment -

            Great, thanks!

            So they need to match ap_verify.*.verify.json, correct? (Double-checking because I really don't want DM-16017 to break the build.)

            Show
            krzys Krzysztof Findeisen added a comment - Great, thanks! So they need to match ap_verify.*.verify.json , correct? (Double-checking because I really don't want DM-16017 to break the build.)
            Hide
            afausti Angelo Fausti added a comment -

            That's correct.

            Show
            afausti Angelo Fausti added a comment - That's correct.
            Hide
            afausti Angelo Fausti added a comment -

            The `ap_verify` pipeline in CI already uploads multiple Job files. No fix needed.

            Show
            afausti Angelo Fausti added a comment - The `ap_verify` pipeline in CI already uploads multiple Job files. No fix needed.

              People

              • Assignee:
                afausti Angelo Fausti
                Reporter:
                krzys Krzysztof Findeisen
                Watchers:
                Angelo Fausti, Krzysztof Findeisen
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel