Skip to content

Triage::ThankContribution: Fetch latest MR description

What does this MR do and why?

To minimize race conditions between merge request events which can lead to description overwrites by the processor we now fetch the latest merge request description before updating.

Event description is still used in case of an error.

Fixes #1562 (comment 1982959965)

Expected impact & dry-runs

These are strongly recommended to assist reviewers and reduce the time to merge your change.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/tree/master/doc/scheduled#testing-policies-with-a-dry-run on how to perform dry-runs for new policies.

See https://gitlab.com/gitlab-org/quality/triage-ops/-/blob/master/doc/reactive/best_practices.md#use-the-sandbox-to-test-new-processors on how to make sure a new processor can be tested.

Action items

Edited by Peter Leitzen

Merge request reports