Change database check in geo:check task

Chloe Fonsrequested to merge
554690-better-fix-for-geo-check-rake-task into master

What does this MR do and why?

Change database check in geo:check task by replacing database.connected? in favour of doing an actual query, SELECT 1. It does not hit any table but will check the database connection. Using select_value instead of execute to avoid any potential memory management issues.

References

#545533 (closed) #554690 (closed)

How to set up and validate locally

  1. Set the GITLAB_LICENSE_KEY environment variable in your shell, with the text of a GitLab Premium or Ultimate license key.
  2. Run the installation script from this MR:
curl "https://gitlab.com/gitlab-org/gitlab-development-kit/-/raw/554690-better-fix-for-geo-check-rake-task/support/geo-install" | bash

At the very end of the process, when the gitlab:geo:check command runs, you should see:

GitLab Geo tracking database is correctly configured ... yes

while previous to this MR it would have been no.

MR acceptance checklist

Evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports