Skip to content

jsonnet: add common and per component settings for annotations - #17143

Merged
56quarters merged 2 commits into
mainfrom
56quarters/annotations
Nov 3, 2026
Merged

56quarters merged 2 commits into
mainfrom
56quarters/annotations

Conversation

@56quarters

@56quarters 56quarters commented Nov 2, 2026 •

Copy link
Copy Markdown
Contributor

What this PR does

Add common and per-component configuration to allow key-value pairs to be set as annotations on each Mimir component. The goal is to allow us to easily turn on various types of profiling at a per-component granularity via annotations.

Which issue(s) this PR fixes or relates to

N/A

Note to reviewers

There are places in our existing jsonnet that could use this. I'll update those in a follow-up PR.

Checklist

  • Tests updated.
  • Documentation added.
  • CHANGELOG.md updated - the order of entries should be [CHANGE], [FEATURE], [ENHANCEMENT], [BUGFIX]. If changelog entry is not needed, please add the changelog-not-needed label to the PR.
  • about-versioning.md updated with experimental features.

@56quarters
56quarters force-pushed the 56quarters/annotations branch 3 times, most recently from 51c23e2 to 910ca9d Compare November 2, 2026 20:33
@56quarters
56quarters marked this pull request as ready for review November 2, 2026 21:07
@56quarters
56quarters requested a review from a team as a code owner November 2, 2026 21:07

@cursor cursor Bot left a comment •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Stale Bugbot comment from a previous run.

Comment thread operations/mimir/querier.libsonnet Resolved
@56quarters
56quarters marked this pull request as draft November 2, 2026 21:17
@56quarters
56quarters force-pushed the 56quarters/annotations branch from 910ca9d to def1744 Compare November 3, 2026 00:48
Add common and per-component configuration to allow key-value pairs to
be set as annotations on each Mimir component. The goal is to allow us
to easily turn on various types of profiling at a per-component
granularity via annotations.
@56quarters
56quarters force-pushed the 56quarters/annotations branch from def1744 to 9693ce1 Compare November 3, 2026 00:50
@56quarters

Copy link
Copy Markdown
Contributor Author

@cursor review

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 9693ce1. Configure here.

@56quarters
56quarters marked this pull request as ready for review November 3, 2026 14:06
Signed-off-by: Nick Pillitteri <nick.pillitteri@grafana.com>
@56quarters
56quarters merged commit 88ff247 into main Nov 3, 2026
90 checks passed
@56quarters
56quarters deleted the 56quarters/annotations branch November 3, 2026 17:41
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