neon
neon copied to clipboard
CI(ingest_regress_test_result): badly broken test breaks the script indefinitely for the PR (#8004)
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