Skip to content
This repository was archived by the owner on Sep 23, 2024. It is now read-only.

Add some metrics to agglayer #94

Merged
merged 12 commits into from
Mar 4, 2024
Merged

Conversation

vcastellm
Copy link
Contributor

@vcastellm vcastellm commented Mar 1, 2024

Introduce the following metrics:

Interop

  • check_tx_total
  • execute_total
  • get_tx_status_{status}_total
  • settle_total
  • verify_signature_total
  • verify_zkp_total

RPC

  • send_tx_total
  • get_tx_status_total

@vcastellm vcastellm requested a review from a team March 1, 2024 11:47
Co-authored-by: Stefan Negovanović <[email protected]>
@vcastellm vcastellm requested a review from zuiris March 1, 2024 14:31
vcastellm and others added 3 commits March 1, 2024 15:32
Co-authored-by: Stefan Negovanović <[email protected]>
Co-authored-by: Stefan Negovanović <[email protected]>
Co-authored-by: Stefan Negovanović <[email protected]>
@Sharonbc01
Copy link

Will the CDK chain ID related to transactions be captured in the verified signatures data so that when we have more chains beyond Astar attached we can identify transaction activity by chain? Thanks

@vcastellm
Copy link
Contributor Author

Will the CDK chain ID related to transactions be captured in the verified signatures data so that when we have more chains beyond Astar attached we can identify transaction activity by chain? Thanks

Really good question @Sharonbc01, and answer is no, going to add that metric attributes

Copy link

sonarqubecloud bot commented Mar 4, 2024

@vcastellm vcastellm merged commit 0fed799 into main Mar 4, 2024
5 of 6 checks passed
@vcastellm vcastellm deleted the CDK-104-add-some-metrics-to-agg-layer branch March 4, 2024 10:14
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