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

Unified Storage: Adds metrics for database query counts #87781

Conversation

owensmallwood
Copy link
Contributor

This will allow unified storage to get the grafana_database_queries_duration_seconds_count metric.

@owensmallwood owensmallwood added no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes labels May 13, 2024
@owensmallwood owensmallwood added this to the 11.1.x milestone May 13, 2024
@owensmallwood owensmallwood requested a review from a team May 13, 2024 22:58
@owensmallwood owensmallwood self-assigned this May 13, 2024
@owensmallwood owensmallwood requested a review from a team as a code owner May 13, 2024 22:58
@owensmallwood owensmallwood requested review from zserge and removed request for a team May 13, 2024 22:58
@suntala
Copy link
Contributor

suntala commented May 14, 2024

Looks like it created some build issues in github.com/grafana/grafana/pkg/services/store/entity but otherwise LGTM 👍

@owensmallwood owensmallwood changed the title Unified Stroage: Adds metrics for database query counts Unified Storage: Adds metrics for database query counts May 14, 2024
@owensmallwood owensmallwood merged commit 557c3a9 into main May 14, 2024
13 checks passed
@owensmallwood owensmallwood deleted the owensmallwood/unified-storage-add-metrics-for-database-query-counts branch May 14, 2024 16:17
@ephemeral-instances-bot
Copy link

Error building instance: Contact #proj-ephemeral-hg-instances if it is not a compile error. Logs

Error message

handling pull request closed event: deleting instance by slug: unexpected response status: status=502 responseBody=

<title>502 Server Error</title>

Error: Server Error

The server encountered a temporary error and could not complete your request.

Please try again in 30 seconds.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/backend no-backport Skip backport of PR no-changelog Skip including change in changelog/release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants