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

Metric for any das.Aggregator Store errors, fixes: NIT-2550 #2410

Merged
merged 5 commits into from
Jun 27, 2024

Conversation

Tristan-Wilson
Copy link
Member

If there are errors sending to any backends in das.Aggregator.Store then the new metric arb_das_rpc_aggregator_store_error_gauge will be 1 until there is a call to Store with no errors. The metric will be 1 even if the Store succeeds, as long as at least one backend failed.

If there are errors sending to any backends in das.Aggregator.Store then
the new metric arb_das_rpc_aggregator_store_error_gauge will be 1 until
there is a call to Store with no errors. The metric will be 1 even if
the Store succeeds, as long as at least one backend 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 Jun 18, 2024
Copy link
Contributor

@ganeshvanahalli ganeshvanahalli left a comment

Choose a reason for hiding this comment

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

LGTM

amsanghi
amsanghi previously approved these changes Jun 21, 2024
@Tristan-Wilson Tristan-Wilson changed the title Metric for any das.Aggregator Store errors Metric for any das.Aggregator Store errors, fixes: NIT-2550 Jun 26, 2024
@Tristan-Wilson Tristan-Wilson merged commit b87ce0d into master Jun 27, 2024
11 checks passed
@Tristan-Wilson Tristan-Wilson deleted the das-store-failure-metric branch June 27, 2024 22:53
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
design-approved 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.

4 participants