Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DAS RPC Client Metrics #2549

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

DAS RPC Client Metrics #2549

wants to merge 4 commits into from

Conversation

Tristan-Wilson
Copy link
Member

@Tristan-Wilson Tristan-Wilson commented Aug 2, 2024

This PR adds metrics to the client side of DAS batch posting (eg on the Nitro batch poster). The metrics are designed to mirror the metrics on the daserver side.

There are combined metrics for both legacy and chunked Store methods:

arb_das_rpcclient_store_success: Count of successful batch stores
arb_das_rpcclient_store_failure: Count of failed batch stores
arb_das_rpcclient_store_bytes: Bytes stored
arb_das_rpcclient_store_duration: Histogram of total duration of batch stores (ns)

There are metrics specific for chunked Store requests:

arb_das_rpcclient_sendchunk_success: Count of chunks sent successfully arb_das_rpcclieny_sendchunk_failure: Count of chunks that failed

NIT-1317

This PR adds metrics to the client side of DAS batch posting (eg on the
Nitro batch poster). The metrics are designed to mirror the metrics on
the daserver side.

There are combined metrics for both legacy and chunked Store methods:
arb_das_rpcclient_store_requests: Count of initiated batch stores
arb_das_rpcclient_store_success: Count of successful batch stores
arb_das_rpcclient_store_failure: Count of failed batch stores
arb_das_rpcclient_store_bytes: Bytes stored
arb_das_rpcclient_store_duration: Histogram of total duration of batch stores (ns)

There are metrics specific for chunked Store requests:
arb_das_rpcclient_sendchunk_success: Count of chunks sent successfully
arb_das_rpcclieny_sendchunk_failure: Count of chunks that failed
@cla-bot cla-bot bot added the s Automatically added by the CLA bot if the creator of a PR is registered as having signed the CLA. label Aug 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
s Automatically added by the CLA bot if the creator of a PR is registered as having signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants