Skip to content

Conversation

@algobarb
Copy link
Contributor

@algobarb algobarb commented Dec 22, 2022

GitHub Logo

Overview

Updates to telemetry and reporting.

What's New

  • Better telemetry logging
  • Report metrics on telemetry errors

Changelog

Enhancement

  • telemetry: don't report let asyncTelemetryHook wrap and log its own errors(#4932)
  • metrics: count telemetry server errors (#4933)
  • log: stderr offending large log messages(#4930)

Protocol Upgrade

This release does not contain a protocol upgrade.

Additional Resources

@codecov
Copy link

codecov bot commented Dec 22, 2022

Codecov Report

Merging #4936 (425443a) into rel/beta (e9e1851) will decrease coverage by 0.54%.
The diff coverage is 20.00%.

@@             Coverage Diff              @@
##           rel/beta    #4936      +/-   ##
============================================
- Coverage     54.62%   54.07%   -0.55%     
============================================
  Files           431      427       -4     
  Lines         53568    53537      -31     
============================================
- Hits          29260    28952     -308     
- Misses        22021    22319     +298     
+ Partials       2287     2266      -21     
Impacted Files Coverage Δ
logging/cyclicWriter.go 46.46% <0.00%> (-3.00%) ⬇️
logging/telemetryhook.go 50.70% <50.00%> (+0.34%) ⬆️
node/netprio.go 0.00% <0.00%> (-65.52%) ⬇️
node/impls.go 0.00% <0.00%> (-59.26%) ⬇️
node/node.go 4.29% <0.00%> (-27.46%) ⬇️
util/db/dbutil.go 42.26% <0.00%> (-6.55%) ⬇️
util/metrics/reporter.go 56.31% <0.00%> (-4.86%) ⬇️
ledger/tracker.go 75.10% <0.00%> (-3.80%) ⬇️
logging/telemetryFilteredHook.go 86.66% <0.00%> (-3.34%) ⬇️
cmd/algoh/blockWatcher.go 76.11% <0.00%> (-2.99%) ⬇️
... and 23 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@algojohnlee algojohnlee merged commit 4b2d297 into algorand:rel/beta Dec 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants