Skip to content

Geo: Prevent RegistryFinder calls on the primary

Michael Kozono requested to merge mk/fix-registry-finders-on-primary-geo into master

What does this MR do?

Registry tables only exist on Geo secondaries, so at the very least, it's semantically wrong to use RegistryFinders on a Geo primary. I think it used to be fine since the secondary didn't used to take into account the registry tables.

Fixes https://gitlab.com/gitlab-org/gitlab-ee/issues/11451

Does this MR meet the acceptance criteria?

Conformity

Performance 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