Skip to content

Fail batch-aborted pipelines with reason

Matthias Käppler requested to merge 324646-bulk-fail-with-reason into master

What does this MR do?

We recently rolled out a performance improvement to blocking users, which would cancel any of their running pipelines in bulk. This was part of rapid actionpipeline abuse and we did the simplest thing we could think of then: just cancelling them all in batch.

However, the semantics of the cancel status are actually that this is a user initiated action, which is not true in this case.

As suggested in #324646 (closed) we should use the failed status instead and provide a failure_reason so that it is clear why these pipelines have been canceled. This is implemented in this MR.

Screenshots (strongly suggested)

Screenshot_from_2021-04-01_11-39-43

Screenshot_from_2021-04-01_11-40-02

Screenshot_from_2021-04-01_11-40-08

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Related to #324646 (closed)

Edited by Matthias Käppler

Merge request reports

Loading