Closed Bug 1788533 Opened 2 years ago Closed 2 years ago

fix validate_processed_crash.py to handle pattern_properties better

Categories

(Socorro :: Processor, defect, P2)

Product:
Socorro
Component:
Processor
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: willkg, Assigned: willkg)

Details

Attachments

(2 files)

pr 6163: bug 1788533: support pattern_properties better
53 bytes, text/x-github-pull-request
Details | Review
pr 6164: bug 1788533: support pattern_properties for schema keys in doc
53 bytes, text/x-github-pull-request
Details | Review

validate_processed_crash.py handles pattern_properties poorly. This manifests itself in two ways:

  1. The list of schema keys not in documents lists a bunch of pattern properties that totally are in documents. For example, registers.
  2. The list of document keys not in the schema lists a bunch of keys that are totally in the schema, but as pattern_properties. For example, registers. The output I have to ignore here is quite large and it's hard to see the real issues because of it.

This covers reworking that code to eliminate both problems.

Grabbing this because I started working on it already on account of it making it a lot harder to update rust-minidump.

Assignee: nobody → willkg
Status: NEW → ASSIGNED

This went to prod just now in bug #1788726. Marking as FIXED.

Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: