Skip to content

Refactor snippet repo path error handling

Vijay Hawoldar requested to merge vij-snippet-repository-exceptions into master

What does this MR do?

Refs #216273 (closed)

When performing the snippet backfill migration, we check for a specific invalid path error from Gitaly, so that we can handle it appropriately and continue the migration.

This MR refactors that check to be handled within SnippetRepository which feels like a more logical place to handle it (along with the other errors). This also means that when we then come to re-check for that error in other areas, such as the Create/Update services, we won't need to do the same string comparison in multiple places - we can just check for a PathError.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by 🤖 GitLab Bot 🤖

Merge request reports