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

Support SQuaSH upload of arbitrary Job .json files

    XMLWordPrintable

    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:

                  CI Builds

                  No builds found.