Disabled signups in ci for new deployments (!1577) · Merge requests · GitLab.org / charts / GitLab Chart · GitLab

DJ Mountney requested to merge fix-disable-signups into master Sep 25, 2020

What does this MR do?

Reverts !1527 (merged) which did now work well for us
Makes use of the fact that https://gitlab.com/gitlab-org/gitlab/-/blob/c580b306e974539cf9fca71334c9d8d9adacbc53/config/initializers/1_settings.rb#L183 was never removed from the gitlab settings when we deprecated it, and is still used to populate the default initial application settings.

Related issues

Fixes https://gitlab.com/gitlab-org/charts/gitlab/-/issues/2315

Checklist

See Definition of done.

For anything in this list which will not be completed, please provide a reason in the MR discussion.

Required

Merge Request Title and Description are up to date, accurate, and descriptive
MR targeting the appropriate branch
MR has a green pipeline on GitLab.com

Expected (please provide an explanation if not completing)

Test plan indicating conditions for success has been posted and passes
Documentation created/updated
Tests added
Integration tests added to GitLab QA
Equivalent MR/issue for omnibus-gitlab opened