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

Allow UTF-8 labels in label_replace #15974

Merged
merged 2 commits into from
Feb 5, 2025

Conversation

jhesketh
Copy link
Contributor

@jhesketh jhesketh commented Feb 5, 2025

This makes it consistent with label_join.

This makes it consistent with label_join.

Signed-off-by: Joshua Hesketh <[email protected]>
"invalid-label-name" is valid in utf-8 mode

Signed-off-by: Joshua Hesketh <[email protected]>
Copy link
Member

@beorn7 beorn7 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 at first glance, but I would prefer @ywwg to have a look.

@beorn7 beorn7 requested a review from ywwg February 5, 2025 14:59
Copy link
Member

@ywwg ywwg left a comment

Choose a reason for hiding this comment

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

lgtm, thank you!

@beorn7 beorn7 merged commit fa1bd02 into prometheus:main Feb 5, 2025
26 checks passed
charleskorn added a commit to grafana/mimir that referenced this pull request Mar 7, 2025
charleskorn added a commit to grafana/mimir that referenced this pull request Mar 11, 2025
charleskorn added a commit to grafana/mimir that referenced this pull request Mar 12, 2025
* Upgrade mimir-prometheus

* Bring in upstream test changes

* Apply change to match prometheus/prometheus#15974

* Remove test case added in upstream in prometheus/prometheus#15975

* Add tests for case where range vector selectors and subqueries have 0 range

Brings in change from #10586

* Update `sort` and `sort_desc` behaviour to match prometheus/prometheus#15964

* Add support for native histograms to `irate` and `idelta` to match prometheus/prometheus#15853

* Adjust test cases to match support for native histograms in `irate` and `idelta`

* Change binop annotations to match behaviour in prometheus/prometheus#15895

* Ignore incompatible schemas between the first and second point in `rate` and `increase` if the second point is a reset (prometheus/prometheus#15902)

* Adjust annotation test cases to match new behaviour introduced in previous commit

* Disable tests failing due to prometheus/prometheus#16199

* Add changelog entry

* Update `TestQuerySharding_FunctionCorrectness` to reflect changes in prometheus/prometheus#15964
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.

3 participants