search

openziti / ziti

The parent project for OpenZiti. Here you will find the executables for a fully zero trust, application embedded, programmable network @OpenZiti
https://openziti.io
Apache License 2.0
2.69k stars 153 forks source link

Additional context required for latency timeout messages #473

Closed mikegorman-nf closed 2 years ago

mikegorman-nf commented 3 years ago

The following message indicates a probe failure, but there is no context to know what probe failed, significantly impairing its usefulness. If there was some context as to what the target was, it could assist in building a timeline.

Aug 23 02:14:21 ziti-edge-router ziti-router[448542]: {"file":"/home/runner/go/pkg/mod/github.com/openziti/foundation@v0.15.60/metrics/latency.go:41","func":"github.com/openziti/foundation/metrics.ProbeLatency.func2","level":"error","msg":"latency timeout after [10s]","time":"2021-08-23T02:14:21Z"}

plorenz commented 2 years ago

Need to make sure this works for the heartbeat code as well. Want the context of what the heartbeat is to/from.

plorenz commented 2 years ago

Control channel and links both use heartbeats now. Control channels now close on heartbeat timeout. Links will log a warning which includes the link id, so this should be resolved. Please re-open if it needs further refinement.