search

inspirehep / inspire-next

The INSPIRE repo.
https://inspirehep.net
GNU General Public License v3.0
59 stars 69 forks source link

migrator: shorten the validation errors for Sentry #2262

Closed jacquerie closed 7 years ago

jacquerie commented 7 years ago

As you might have noticed from the 100+ emails you got about validation errors on properties.authors.items.properties.ids.items.anyOf, Sentry apparently has trouble clustering errors that are too long.

Now that the schema contains descriptions we hit this unknown limit, so we need to shorten the string we log in https://github.com/inspirehep/inspire-next/blob/12239630a763a8db96ecb6c78ebc7abec1a60c5d/inspirehep/modules/migrator/tasks/records.py#L333-L336 to avoid spamming everyone with one email per validation error.

jacquerie commented 7 years ago

I will do this soon-ish because I need a new Nightly build for the paper.

jacquerie commented 7 years ago

(Also not Type: Bug, because no user is affected by this)