neon icon indicating copy to clipboard operation
neon copied to clipboard

CI(ingest_regress_test_result): badly broken test breaks the script indefinitely for the PR (#8004)

Open sdesalas opened this issue 5 months ago • 0 comments

Problem

If a test fails badly enough, the JSON file for it might have some important fields missing, like labels, which we use; this makes scripts/ingest_regress_test_result-new-format.py fail. See #8004. Currently with @bayandin

Summary of changes

This change ignores parsing errors for corrupted JSON files in scripts/ingest_regress_test_result-new-format.py.

Checklist before requesting a review

  • [x] I have performed a self-review of my code.
  • [ ] If it is a core feature, I have added thorough tests.
  • [ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
  • [ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

  • [ ] Do not forget to reformat commit message to not include the above checklist

sdesalas avatar Aug 29 '24 15:08 sdesalas