Skip to content

Enforce instance-level MR approval settings

What does this MR do?

The original MR has been reverted as a precautionary measure of excessive Gitaly load observed on staging. We later confirmed that the underlining issues were not caused by this MR but rather existing performance issues with project-level merge request approval settings (see note).

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

Related to #273765 (closed)

Edited by Tan Le

Merge request reports