Skip to content

Do not search by username

Reuben Pereira requested to merge rp/do-not-search-by-user into master

What does this MR do and why?

Describe in detail what your merge request does and why.

The Chatops bot username used when posting messages was changed in !1767 (merged).

This caused the Slack Search API called in Slack::CoordinatedPipelineNotification to stop finding the correct results.

This has been fixed by !1769 (merged), but we can simply remove the username search filter. No other user posts similar messages in #announcements channel, so the accuracy of the search should not be affected.

gitlab-com/gl-infra/delivery#2225 (closed)

Author Check-list

  • Has documentation been updated?
Edited by Reuben Pereira

Merge request reports