Skip to content

Internal API changes to decouple redis from Shell

username-removed-367626 requested to merge gitaly-shell-redis-2 into master

What does this MR do?

  • Add a new field to /internal/check to signal redis being available
  • Add a new endpoint, /internal/pre_receive to do all Shell pre-receive hook operations (for now, just increasing the reference counter).

Are there points in the code the reviewer needs to double check?

No

Why was this MR needed?

We want to decouple Shell from redis to be able to not have redis running on the NFS hosts. See https://gitlab.com/gitlab-org/gitaly/issues/467

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes https://gitlab.com/gitlab-org/gitaly/issues/520

Edited by username-removed-367626

Merge request reports

Loading