Skip to content

Pass correlation_id over to gitaly-ssh

Pavlo Strokov requested to merge ps-gitaly-ssh-correlation-id into master

When gitaly-ssh is called it doesn't receive correlation_id. That is why it is hard to understand what to what chain of calls this call is related.
correlation_id passed as env var CORRELATION_ID to gitaly-ssh process. So it would be picked by it and used in outgoing requests.
In order to cover other missing parts where correlation_id should be passed the client.DialContext method includes interceptors for it by default. If correlation_id is present in the context.Context used to invoke the method it will be passed to the remote.

In order to get more consistent setup for function that establishes gRPC connections to remotes TracingInterceptor moved into client.DialContext to unify method of opening connection. The included interceptor covered with tests, so we are sure it is included into the chain.

Part of: #3047 (closed)

Merge request reports