Skip to content

Only move repos for legacy project storage

John Cai requested to merge jc-dont-try-to-movedirs-unless-legacy into master

What does this MR do?

fixes: #35456 (closed)

After removing most add namespace calls, we noticed there are still quite a few that are happening. It turns out we are calling namespace service for every user rename, even if all of their projects are on hashed storage.

Here is an example: https://log.gitlab.net/goto/6078622d952009761a1eadcef9e1ec43

This MR changes the legacy namespace so that it only cares about moving the namespace directory for projects with legacy namespace.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team
Edited by Michael Kozono

Merge request reports