Phase out configuration to split out database-related metrics
With separate_database_metrics
, we have an option which will move all metrics which scrape the Praefect database to a separate endpoint. This is done to not hit the database with expensive queries every time metrics are scraped, which is important especially in the context of large-scale Gitaly Cluster deployments.
This configuration is currently disabled by default because we had to backport it into bugfix releases, and we didn't want to introduce backwards-incompatible changes back then. We should default-enable this key now though with an announcement that it's about to change and then eventually remove the configuration altogether such that database-scraping metrics are always on a separate endpoint.