search

runar-rkmedia / gotally

0 stars 1 forks source link

add opentelemetry #5

Closed runar-rkmedia closed 1 year ago

runar-rkmedia commented 1 year ago

Could we also add https://pyroscope.io/docs/golang/ , but hide it under a config-flag? It would be useful in the beginning, to get some better insights.

runar-rkmedia commented 1 year ago

Some opentelemetry has been added for tracing.

I tried adding metrics, but there were some package-issues, most likely because metrics for otel is not stable.

I added simple prometheus-metrics, and exposed pprof-traces. These should be exposed on a different port, so not everybody can snoop around in it (and cause possible slowdowns).

I added Pyroscope. It is mostly like pprof, but constantly monitoring and a bit more shiny. It seems nice for tracking possible bottlenecks for the server under load over time.