workhorse: refactor access to Gitaly Sidechannel
What does this MR do and why?
Contributes to #472033 (closed)
Problem
NewConnectionWithSidechannel
returns Sidechannel even if it's not
needed.
Solution
Split NewConnectionWithSidechannel
function into:
-
NewConnection()
to get Gitaly connection -
Sidechannel()
to access Gitaly sidechannel