Skip to content

[RUN AS-IF-FOSS] Move read_only/read_write back to Gitlab::Database

Yorick Peterse requested to merge move-gitlab-database-readonly into master

What does this MR do?

These methods were scoped to a connection in !65262 (merged). A discussion after the merge request was merged revealed these methods aren't tied to a specific connection, and as such better belong in Gitlab::Database.

I briefly considered moving them to the Gitlab module instead, as the methods don't actually run any database queries. Unfortunately, this isn't possible as this module is loaded too early to prepend it with an EE module.

Does this MR meet the acceptance criteria?

Conformity

Edited by Yorick Peterse

Merge request reports