diff options
| author | Toon Claes <toon@iotcl.com> | 2017-09-19 09:44:58 +0200 |
|---|---|---|
| committer | Toon Claes <toon@iotcl.com> | 2017-10-06 22:37:40 +0200 |
| commit | d13669716ab0c31ce9039ae9f7f073e33a4dc40f (patch) | |
| tree | 001bb2e6aa76ea7531c93e469c396f7fdcc408a7 /lib/system_check | |
| parent | 2cf5dca8f80cdefeb8932bf80417f52f289668c8 (diff) | |
| download | gitlab-ce-d13669716ab0c31ce9039ae9f7f073e33a4dc40f.tar.gz | |
Create idea of read-only databasetc-geo-read-only-idea
In GitLab EE, a GitLab instance can be read-only (e.g. when it's a Geo
secondary node). But in GitLab CE it also might be useful to have the
"read-only" idea around. So port it back to GitLab CE.
Also having the principle of read-only in GitLab CE would hopefully
lead to less errors introduced, doing write operations when there
aren't allowed for read-only calls.
Closes gitlab-org/gitlab-ce#37534.
Diffstat (limited to 'lib/system_check')
| -rw-r--r-- | lib/system_check/app/git_user_default_ssh_config_check.rb | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/lib/system_check/app/git_user_default_ssh_config_check.rb b/lib/system_check/app/git_user_default_ssh_config_check.rb index dfa8b8b3f5b..9af21078403 100644 --- a/lib/system_check/app/git_user_default_ssh_config_check.rb +++ b/lib/system_check/app/git_user_default_ssh_config_check.rb @@ -11,10 +11,10 @@ module SystemCheck ].freeze set_name 'Git user has default SSH configuration?' - set_skip_reason 'skipped (git user is not present or configured)' + set_skip_reason 'skipped (GitLab read-only, or git user is not present / configured)' def skip? - !home_dir || !File.directory?(home_dir) + Gitlab::Database.read_only? || !home_dir || !File.directory?(home_dir) end def check? |