Move snippet repository storage classes to namespace
What does this MR do?
According to our guidelines and cops, all classes should be namespaced. Related to the repository storage update, we introduced the new classes for groups in a namespace, but there are others, for projects and snippets, that still follow the old naming. So now, we have mixed namespaces in this functionality.
With this MR we start cleaning that up, and move everything to a namespace.
Since some of the classes are workers or called inside existing workers, we cannot get rid entirely of these classes, and we need to wait for 14.0 to do that.
In specific we do:
- Move
SnippetRepositoryStorageMoves
entity toSnippets::RepositoryStorageMoves
- Copy
SnippetRepositoryStorageMoves
model toSnippetsRepositoryStorageMoves
- Remove all the logic from
SnippetRepositoryStorageMoves
- Make
SnippetRepositoryStorageMoves
a subclass ofSnippets::RepositoryStorageMoves
- Remove all the logic from
- Copy
SnippetUpdateRepositoryStorageWorker
toSnippets::UpdateRepositoryStorageWorker
- Remove all the logic from
SnippetUpdateRepositoryStorageWorker
- Make
SnippetUpdateRepositoryStorageWorker
a subclass ofSnippets::UpdateRepositoryStorageWorker
- Remove all the logic from
- Copy
SnippetScheduleBulkRepositoryShardMovesWorker
toSnippets::ScheduleBulkRepositoryShardMovesWorker
- Remove all the logic from
SnippetScheduleBulkRepositoryShardMovesWorker
- Make
SnippetScheduleBulkRepositoryShardMovesWorker
a subclass ofSnippets::ScheduleBulkRepositoryShardMovesWorker
- Remove all the logic from
- Replace all references to the old classes with the new ones.
Does this MR meet the acceptance criteria?
Conformity
- [X Code review guidelines
-
Merge request performance guidelines -
Style guides -
Separation of EE specific content
Related to #299853 (closed)
Edited by Francisco Javier López (ex-Gitlab)