Skip to content

Move usage of delayed_project_removal to namespace settings

What does this MR do?

Closes #322610 (closed) and unblocks a portion for !55678 (merged).

Ignores the namespaces delayed_project_removal column and moves usage of it to namespace_settings since the column and data were migrated there in the previous release.

This unblocks !55678 (merged) because the after update callback caused issues with the new cascading settings concern.

I also tried to make delegation work so we didn't have to update all usage en masse but it was way harder than it needed to be. I felt after all that it was better to just update usage and move on - and now we don't have technical debt, either.

Screenshots (strongly suggested)

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 Drew Blessing

Merge request reports