The following discussion from !162678 should be addressed: - [ ] @brodock started a [discussion](https://gitlab.com/gitlab-org/gitlab/-/merge_requests/162678#note_2062268366): (+1 comment) > Should we consider refactoring to use a ServiceResponse instead of using exceptions? > > We have this new pattern as the preferred way to report failure or success: https://docs.gitlab.com/ee/development/reusing_abstractions.html#serviceresponse