You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It would be cool to have tracing on the initial dial request, but I think this would require a protocol-level enhancement to DialResponse (grpc).
The initial flow (happy path) looks like:
client -> server -> agent -[ endpoint ]- agent -> server -> client
It would be great to have latency information for each hop. In other words, something like:
Server records dial_req received timestamp (and server ID?)
Agent records dial_req received timestamp (and agent ID?)
Agent records endpoint dial complete timestamp
Agent includes traces in the DialResponse
Server records dial_resp received timestamp
Server adds request & response received timestamps to DialResponse
Client constructs the full trace, records latency metrics for each hop, logs full trace at high verbosity
It looks like there's OpenCensus gRPC integration that we should investigate, but I'm not sure if it would be work with our multiplexed streams. At the very least, we should make sure our design fits the OpenCensus tracing spec
The text was updated successfully, but these errors were encountered:
k8s-ci-robot
added
lifecycle/frozen
Indicates that an issue or PR should not be auto-closed due to staleness.
and removed
lifecycle/stale
Denotes an issue or PR has remained open with no activity and has become stale.
labels
Jan 4, 2023
It would be cool to have tracing on the initial dial request, but I think this would require a protocol-level enhancement to
DialResponse
(grpc).The initial flow (happy path) looks like:
It would be great to have latency information for each hop. In other words, something like:
dial_req received
timestamp (and server ID?)dial_req received
timestamp (and agent ID?)endpoint dial complete
timestampdial_resp received
timestampIt looks like there's OpenCensus gRPC integration that we should investigate, but I'm not sure if it would be work with our multiplexed streams. At the very least, we should make sure our design fits the OpenCensus tracing spec
The text was updated successfully, but these errors were encountered: