celestia-node icon indicating copy to clipboard operation
celestia-node copied to clipboard

[EPIC] Telemetry tracking issue

Open liamsi opened this issue 3 years ago • 5 comments

Telemetry

We've started the telemetry integration process as outlined in #901, and this issue gathers all the sub-issues and/or PRs related to the integration/implementation process.

Base(d)

Scaffolding for further integration

  • [ ] #663
  • [ ] https://github.com/celestiaorg/celestia-node/pull/907
  • [x] https://github.com/celestiaorg/celestia-node/issues/934
  • [ ] #935
  • [ ] #936
  • [ ] #937
  • [ ] https://github.com/celestiaorg/celestia-node/issues/1003

Metrics Integration Spots

Actual code points for us to monitor/measure/observe/etc

  • [ ] https://github.com/celestiaorg/celestia-node/issues/639
  • [ ] https://github.com/celestiaorg/celestia-node/issues/98
  • [x] https://github.com/celestiaorg/celestia-node/issues/999
  • [x] https://github.com/celestiaorg/celestia-node/issues/1040
  • [ ] Uptime metric

Tracing Integration Spots

  • [ ] ipld: consider tracing for GetShares
  • [ ] share: simple trace for Light (besides Full) Availability

liamsi avatar Dec 02 '21 20:12 liamsi

this would be awesome!!

tac0turtle avatar Jan 07 '22 20:01 tac0turtle

Proposing https://github.com/VictoriaMetrics/VictoriaMetrics

Wondertan avatar Jan 07 '22 21:01 Wondertan

I'll try chew on this. It seems like geth's metrics package is a fork of the go-metrics package with prometheus support added. I'll try and integrate that to celestia-node.

LETS GET THAT MONITORING SETUP ONLINE

cemozerr avatar Feb 07 '22 18:02 cemozerr

Key metrics of the top off my head:

  • measure sampling time
  • measure BlockSync time (per block and in total from height h, where h could be 0/genesis) for full nodes
  • header sync time
  • ....

liamsi avatar Apr 13 '22 10:04 liamsi

@liamsi sync is different from DAS. sync is only header sync rather than the actual block reconstruction. That happens during DAS

renaynay avatar May 16 '22 18:05 renaynay