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

Update mixin to use new backend metric #1151

Merged
merged 3 commits into from
Dec 2, 2021
Merged

Conversation

zalegrala
Copy link
Contributor

What this PR does:

  • Update examples and operations to use the newer backend-agnostic metric name.

Checklist

  • Tests updated
  • Documentation added
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]

@zalegrala zalegrala changed the title Backend metric Update examples and operations dashboards to use new backend metric Dec 2, 2021
Copy link
Member

@yvrhdn yvrhdn left a comment

Choose a reason for hiding this comment

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

Looks good to me, just a small comment. Btw, you've only changed the tempo-mixin right? The PR mentions "examples and dashboards", are you also planning to update the examples?
Do they even use this metric?

@@ -67,11 +67,11 @@ dashboard_utils {
g.row('GCS')
Copy link
Member

Choose a reason for hiding this comment

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

Can you also change the name of these rows? There are a couple references to GCS in these dashboards.

@zalegrala
Copy link
Contributor Author

zalegrala commented Dec 2, 2021

You're right, I'll update the commit. I think before I'd rebased there were changes elsewhere.

@zalegrala zalegrala changed the title Update examples and operations dashboards to use new backend metric Update operations dashboards to use new backend metric Dec 2, 2021
@zalegrala zalegrala force-pushed the backendMetric branch 2 times, most recently from 975c847 to ab80def Compare December 2, 2021 17:56
@zalegrala zalegrala changed the title Update operations dashboards to use new backend metric Update mixin to use new backend metric Dec 2, 2021
Copy link
Member

@yvrhdn yvrhdn left a comment

Choose a reason for hiding this comment

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

Nice 👍

@yvrhdn yvrhdn merged commit 35574f7 into grafana:main Dec 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants