Skip to content

Remove vuln slo reminder

Jennifer Li requested to merge jennli-remove-vuln-nudge into master

What does this MR do and why?

This MR removes the vulnerability SLO reminder, as discussed in #1242 (comment 1650713244).

In short, we believe that the similar comment posted by @gitlab-securitybot fullfills the purpose of nudging for a vulnerability fix before it breaches SLA, and we should just remove the one posted by @gitlab-bot.

This was also discussed with @ankelly today in Slack, and we think that having just 1 bot being responsible for 1 type of nudge is a good step forward to reduce complexity.

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

Merge request reports