Skip to content

Remove use_model_load_balancing feature flag and GITLAB_USE_MODEL_LOAD_BALANCING env var

Dylan Griffith requested to merge cleanup-use-model-load-balancing-feature-flag into master

What does this MR do and why?

Previously these feature flags and environment variables were introduced as a way to control the rollout of a new CI database. The new CI database needs to be configured in config/database.yml but since changing this required a full redeployment of all hosts it was not going to be as easy to switch on/off as a feature flag so this feature flag was setup such that we just ignored the values in config/database.yml unless the feature flag was set.

Now that we've deployed "Phase 3" in production gitlab-com/gl-infra/production#6513 (closed) we no longer need these feature flags or environment variables to enable this behaviour. Now we just use the values as set in the config/database.yml.

Screenshots or screen recordings

These are strongly recommended to assist reviewers and reduce the time to merge your change.

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Dylan Griffith

Merge request reports