1930128 - Perfherder ingestion error: duplicate key values for performance_alert_summary table insertion

Status: NEW

(Tree Management :: Perfherder, defect, P2)

Description

[Greg Mierzwinski [:sparky]]

The alert summary table seems to be having issues with ingestion again. It's unclear why this is happening now since the primary key is not out-of-sync with last_value.

I noticed that all the failing alert summary insertions were inserted at one point though, and that all the failures seem to be duplicate insertions: https://sql.telemetry.mozilla.org/queries/103711/source

It's odd that this is happening though because the code that creates these alert summaries is wrapped in a transaction: https://github.com/mozilla/treeherder/blob/cfe5c6cea5244a0d8fc9d03f8bc3c62bec419af2/treeherder/perf/alerts.py#L90

I do wonder if this is preventing the insertion creation of alerts though since we don't catch an error from that creation call, and the alerts get created after it.

Comment 1

[Greg Mierzwinski [:sparky]]

I don't think we need to be too concerned about this issue because if a performance alert summary already exists, in future runs for the alert creation, we should be able to create the alerts if any were missed because of a failure.