Skip to content

Add param for GitalyClusterConfig initialize

James Nutt requested to merge jmd-allow-params-gitalyclusterconfig into master

What does this MR do and why?

https://gitlab.com/gitlab-org/gitlab-qa/-/merge_requests/1077+ introduced a mechanism to make Gitaly Cluster a component.
Part of this change was to manage GitalyClusterConfig in a class - however this class omitted including the ability to provide alternative config values to the default.
This MR covers this functionality.

How to set up and validate locally

  • This change introduces a signature and corresponding changes to init method, but no functional changes so output should remain the same

MR acceptance checklist

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

Merge request reports