Skip to content
This repository has been archived by the owner on Apr 4, 2024. It is now read-only.

analytics(app): update telemetry to Ethermint modules #1106

Merged
merged 8 commits into from
Jun 2, 2022

Conversation

danburck
Copy link
Contributor

@danburck danburck commented Jun 2, 2022

Description

This PR adds telemetry to Ethermint modules and is an advancement of the recent telemetry implementation on Evmos.

Metrics

I implemented the following metrics to observe. They still need to be added to the respective specs that use Ethermint:

Metric Description Unit Type
feemarket_base_fee Amount of base fee per EIP-1559 block token gauge
feemarket_block_gas Amount of gas used in an EIP-1559 block token gauge
tx_msg_ethereum_tx_gas_limit_per_gas_used Ratio of gas limit to gas used for a etheruem tx ratio gauge

@linear
Copy link

linear bot commented Jun 2, 2022

ENG-422 Add Observability (telemetry) on Ethermint

Add Gauges and counters specially on the fee market module to track changes in the BaseFee, block gas, tx refunds, etc

@codecov
Copy link

codecov bot commented Jun 2, 2022

Codecov Report

Merging #1106 (d0a95e3) into main (046cd00) will increase coverage by 0.10%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1106      +/-   ##
==========================================
+ Coverage   61.25%   61.36%   +0.10%     
==========================================
  Files          88       88              
  Lines        7194     7213      +19     
==========================================
+ Hits         4407     4426      +19     
  Misses       2563     2563              
  Partials      224      224              
Impacted Files Coverage Δ
x/evm/keeper/msg_server.go 85.10% <100.00%> (+2.60%) ⬆️
x/feemarket/keeper/abci.go 91.42% <100.00%> (+1.42%) ⬆️

@danburck danburck changed the title ENG 422 add telemetry analytics(app): add telemetry to Ethermint modules Jun 2, 2022
@danburck danburck changed the title analytics(app): add telemetry to Ethermint modules analytics(app): update telemetry to Ethermint modules Jun 2, 2022
@danburck danburck marked this pull request as ready for review June 2, 2022 10:19
Copy link

@Prosp3r Prosp3r left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

x/evm/keeper/msg_server.go Outdated Show resolved Hide resolved
x/evm/keeper/msg_server.go Outdated Show resolved Hide resolved
Copy link
Contributor

@crypto-facs crypto-facs left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM besides @fedekunze comments

x/evm/keeper/msg_server.go Outdated Show resolved Hide resolved
x/feemarket/keeper/abci.go Show resolved Hide resolved
@danburck danburck enabled auto-merge (squash) June 2, 2022 12:58
@danburck danburck merged commit cad7ff6 into main Jun 2, 2022
@danburck danburck deleted the ENG-422-add-telemetry branch June 2, 2022 13:04
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants