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

Support Block Adapter stats blockstoreTag #8680

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

Conversation

itaigilo
Copy link
Contributor

@itaigilo itaigilo commented Feb 17, 2025

Closes #8672.

Change Description

Currently, the Block Adapters that use reportMetrics only report with operation and isErrStr labels -
Adding another blockstoreTag label to it, to allow better tracing.

@itaigilo itaigilo added exclude-changelog PR description should not be included in next release changelog msb labels Feb 17, 2025
Copy link

E2E Test Results - DynamoDB Local - Local Block Adapter

14 passed

Copy link

E2E Test Results - Quickstart

11 passed

@itaigilo itaigilo marked this pull request as ready for review February 17, 2025 20:27
@itaigilo itaigilo changed the title Support Block Adapter stats tag Support Block Adapter stats blockstoreTag Feb 18, 2025
@itaigilo itaigilo requested review from arielshaqed and a team February 20, 2025 10:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
exclude-changelog PR description should not be included in next release changelog msb
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow label in Metrics Adapter
1 participant