Skip to content

Expose BulkImports::Failure#exception_message in the API

What does this MR do and why?

This MR updates BulkImports API in order to expose exception_message and provide a bit of clarity in what went wrong with the migration. Exposed attribute gets:

  • Any files paths looking like /foo/bar get filtered and replaced with [FILTERED]
  • Truncated to 72 characters (even though original db value is 255 characters). This is mainly to cover a situation if a message contains sensitive information we truncate it even further. 72 characters is a guess based on standard commit title length, can be adjusted if needed

Mentions #363339 (closed)

Screenshots or screen recordings

image

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by George Koltsov

Merge request reports

Loading