1787931 - use processed crash schema permissions for super search

Status: RESOLVED FIXED

(Socorro :: Processor, task, P2)

Description

[Will Kahn-Greene [:willkg] ET needinfo? me]

super_search_fields.py encodes field permissions. However, given that all fields now derive from corresponding processed crash fields, we should have super_search_fields.py pull the permissions (and descriptions) from the processed crash schema.

(Alternatively, maybe we can build super_search_fields.py from the processed crash schema and then we don't need super_search_fields.py at all.)

Comment 1

[Will Kahn-Greene [:willkg] ET needinfo? me]

Also, maybe we should have a test that raises an error when super_search_fields.py has a source field that doesn't exist in the processed crash schema.

Comment 2

[Will Kahn-Greene [:willkg] ET needinfo? me]

Grabbing this to do soon.

Comment 3

[Will Kahn-Greene [:willkg] ET needinfo? me]

Attachment: pr 6207: bug 1787931: remove SuperSearchMissingFields and related bits

Comment 4

[Will Kahn-Greene [:willkg] ET needinfo? me]

willkg merged PR #6207: "bug 1787931: remove SuperSearchMissingFields and related bits" in d57e4e8.

This removes a bunch of code we don't need anymore.

Comment 5

[Will Kahn-Greene [:willkg] ET needinfo? me]

Attachment: pr 6208: bug 1787931: fix super search fields to pull permissions from processed crash schema

Comment 6

[Will Kahn-Greene [:willkg] ET needinfo? me]

willkg merged PR #6208: "bug 1787931: fix super search fields to pull permissions from processed crash schema" in 6bd47e4.

Comment 7

[Will Kahn-Greene [:willkg] ET needinfo? me]

This was deployed yesterday in bug #1794650. Marking as FIXED.