10000 MQE: make capitalisation of `cortex_mimir_query_engine_plan_stage_latency_seconds` label values consistent by charleskorn · Pull Request #11410 · grafana/mimir · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

MQE: make capitalisation of cortex_mimir_query_engine_plan_stage_latency_seconds label values consistent #11410

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

Merged
merged 1 commit into from
May 9, 2025

Conversation

charleskorn
Copy link
Contributor

What this PR does

This PR makes the capitalisation of the values for the stage_type label consistent.

Given stage has capitalised, human-readable names, I've chosen to make stage_type follow that pattern as well.

Which issue(s) this PR fixes or relates to

#11127

Checklist

  • Tests updated.
  • [n/a] Documentation added.
  • [covered by Mimir Query Engine #10067] CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX].
  • [n/a] about-versioning.md updated with experimental features.

@charleskorn charleskorn marked this pull request as ready for review May 7, 2025 06:46
@charleskorn charleskorn requested a review from a team as a code owner May 7, 2025 06:46
@charleskorn charleskorn enabled auto-merge (squash) May 8, 2025 23:33
@charleskorn charleskorn force-pushed the charleskorn/metric-label branch from 7193282 to 2bf5165 Compare May 9, 2025 00:27
@charleskorn charleskorn merged commit fb1e4b7 into main May 9, 2025
30 checks passed
@charleskorn charleskorn deleted the charleskorn/metric-label branch May 9, 2025 00:58
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
0