Skip to content

Add logging for bulk update recipients

username-removed-443319 requested to merge add-recipients-logging into 9-0-stable

What does this MR do?

Add logging for the params the BulkUpdateService received, the params it sent to the child update service, and the recipients at various stages of build_recipients.

Are there points in the code the reviewer needs to double check?

Do I need to rescue around these logging calls? I tested locally, but I don't want to make an email fail to send because I made a dumb mistake.

Why was this MR needed?

I can't figure out what's up with https://gitlab.com/gitlab-org/gitlab-ce/issues/28836.

Merge request reports