Skip to content

Internal API changes to decouple redis from Shell

Alejandro Rodríguez 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 gitaly#467 (closed)

Does this MR meet the acceptance criteria?

What are the relevant issue numbers?

Closes gitaly#520 (closed)

Edited by Alejandro Rodríguez

Merge request reports