Adding Metrics to the node

[otherview]

Description

• Adds OpenTelemetry Metrics / Prometheus Metrics to thor
• The API can now serve the /node/metrics endpoint with metrics for the node
• Metrics can be enabled with the enable-telemetry flag set to true (defaults to false)
• All API endpoints now have a histogram and a count of requests
• Swagger updated

Type of change

• New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Unit tests in the code.

Checklist:

• My code follows the style guidelines of this project
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• My changes generate no new warnings
• I have added tests that prove my fix is effective or that my feature works
• New and existing unit tests pass locally with my changes
• New and existing E2E tests pass locally with my changes
• Any dependent changes have been merged and published in downstream modules
• I have not added any vulnerable dependencies to my code

[claytonneal]

Hiya, can the swagger docs be updated and the README updated about that also? cheers
What data is collected? thanks

[otherview]

Hiya, can the swagger docs be updated and the README updated about that also? cheers What data is collected? thanks

Updated ! Also, upon review of the swagger I think it becomes clear that this endpoint should live under the node endpoint 🙏

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 61.20092% with 168 lines in your changes are missing coverage. Please review.

Project coverage is 61.89%. Comparing base (e39091e) to head (8ca08ff).

Files	Patch %	Lines
cmd/thor/utils.go	0.00%	68 Missing ⚠️
api/utils/http.go	0.00%	23 Missing ⚠️
cmd/thor/main.go	0.00%	22 Missing ⚠️
metrics/prometheus.go	86.41%	11 Missing and 11 partials ⚠️
cmd/thor/node/node.go	0.00%	10 Missing ⚠️
cmd/thor/node/packer_loop.go	0.00%	10 Missing ⚠️
p2psrv/server.go	0.00%	5 Missing ⚠️
block/block.go	25.00%	3 Missing ⚠️
comm/handle_rpc.go	0.00%	3 Missing ⚠️
metrics/noop.go	94.44%	1 Missing ⚠️
... and 1 more

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #671      +/-   ##
==========================================
+ Coverage   61.87%   61.89%   +0.01%     
==========================================
  Files         199      202       +3     
  Lines       18318    18696     +378     
==========================================
+ Hits        11335    11572     +237     
- Misses       5880     6013     +133     
- Partials     1103     1111       +8     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[nwbrettski]

@darrenvechain does anything need to be done in order to support this via stoplight?

[darrenvechain]

@darrenvechain does anything need to be done in order to support this via stoplight?

Nope, it's all configured via Open API spec, which is compatible with swagger and stoplight

Implemented already