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

[receiver/sqlserver] Add database status metric #33064

Merged

Conversation

crobert-1
Copy link
Member

Description:

This adds one more metric, sqlserver.properties.db.status, which shows the total number of databases in each state.

Link to tracking Issue:
#29865

Testing:
Testing added, existing tests are passing.

@crobert-1 crobert-1 requested a review from a team May 14, 2024 21:57
@github-actions github-actions bot requested a review from StefanKurek May 14, 2024 21:57
@crobert-1 crobert-1 added the Run Windows Enable running windows test on a PR label May 14, 2024
@crobert-1 crobert-1 force-pushed the sqlserver_add_db_status_metrics branch from 1c28108 to 20d536b Compare May 16, 2024 22:41
@fatsheep9146 fatsheep9146 added the ready to merge Code review completed; ready to merge by maintainers label May 20, 2024
@djaglowski djaglowski merged commit f876d49 into open-telemetry:main May 20, 2024
201 checks passed
@github-actions github-actions bot added this to the next release milestone May 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge Code review completed; ready to merge by maintainers receiver/sqlserver Run Windows Enable running windows test on a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants