Skip to content

Remove security processor slack output

Mayra Cabrera requested to merge remove-security-slack-output into master

What does this MR do and why?

This merge request removes the Slack output every time security issues and merge requests are processed.

Example Example
Screenshot_2023-03-30_at_13.07.50 Screenshot_2023-03-30_at_13.08.17

Why?

  • We have a better and (more reliable) way to visualize what security issues were processed and which are pending. Example https://gitlab.com/gitlab-org/gitlab/-/issues/393774#note_1326214053.
  • The Slack output is unreliable, it doesn't display what issues are deployed to master nor if the backports are processed.
  • The Slack output is also ephemeral, after 90 days the messages will be deleted.
  • Printing Slack output involves a lot of code, this is code the Delivery team has to continuously maintain.

In general, we should focus our efforts on improving the table that is posted in the tracking issue, deleting the code associated with the old way is the first step for that.

Author Check-list

  • [-] Has documentation been updated?
Edited by Mayra Cabrera

Merge request reports