Skip to content

Only react to approvals for tiers transitions

David Dieulivol requested to merge ddieulivol-do_not_react_on_all_events into master

Context

We noticed we're triggering several pipelines when an MR is approved. This is not wanted.

What does this MR do and why?

React to less events to do pipeline transitions.

This will partially revert !2875 (merged), but we can focus on fixing it differently later on (I created #1551 to track).

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 David Dieulivol

Merge request reports