Skip to content

Add gitlab shell config values to gitaly config.toml

John Cai requested to merge jc-write-gitlab-shell-config-in-gitaly into master

What does this Merge Request do?

gitaly!2187 (merged) changed gitaly's config.toml to include gitlab shell hook configuration values. This MR aligns GDK with that change.

gitaly!2168 (merged) will remove support for reading from the gitlab shell yml, so this change is necessary otherwise GDK will break since we will be exclusively reading from the gitaly config toml so we need it to be in the new format that gitaly understands.

Currently, gitaly still supports reading from the gitlab-shell yaml, so these values won't actually be relied upon yet until gitaly!2168 (merged) is merged

Merge Request checklist

  • Tests added for new functionality. If not, please raise Issue to follow-up.
  • This change is backward compatible. If not, please include steps to communicate to our users.
  • Documentation added/updated, if needed.
  • gdk doctor test added, if needed.
Edited by John Cai

Merge request reports