Skip to content

WIP: Dynamically sets the database connection for Geo::BaseRegistry model

username-removed-283999 requested to merge fix/2181 into master

What does this MR do?

Dynamically sets the database connection for Geo::BaseRegistry model.

Are there points in the code the reviewer needs to double check?

Is there a better approach to dynamically set the database connection?

Why was this MR needed?

If Gitlab::Geo.secondary? changes on primary, we need to reinitialize the connection properly in the model to avoid requiring a full unicorn restart on secondary.

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes #2181 (closed)

/cc @stanhu @MrChrisW @rspeicher @brodock

Merge request reports