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

[WIP] PromQL: Add support for quoted metric and label names #4672

Closed
wants to merge 34 commits into from

Conversation

ywwg
Copy link
Member

@ywwg ywwg commented Apr 5, 2023

DO NOT REVIEW / MERGE

First draft of adding support for quoted metric names and labels for promql.

Adds support for
{"my.quoted.metric", "my.dotted.label"="whatever"}

Metric name must be the first item in the bracketed list.

Adds a single correctness test. All other tests still pass.

@CLAassistant
Copy link

CLAassistant commented Apr 5, 2023

CLA assistant check
All committers have signed the CLA.

@lamida
Copy link
Contributor

lamida commented Apr 17, 2023

The CHANGELOG has just been cut to prepare for the next Mimir release. Please rebase main and eventually move the CHANGELOG entry added / updated in this PR to the top of the CHANGELOG document. Thanks!

@ywwg
Copy link
Member Author

ywwg commented Aug 16, 2023

Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
@ywwg ywwg force-pushed the owilliams/quoted-metric-name branch from 4492ca3 to 0a0adc5 Compare January 5, 2024 19:16
@ywwg ywwg force-pushed the owilliams/quoted-metric-name branch from 0a0adc5 to 890d7d0 Compare January 22, 2024 15:31
@ywwg
Copy link
Member Author

ywwg commented Jan 22, 2024

This PR will be open when the changes to Prometheus are made

@ywwg ywwg closed this Jul 23, 2024
@ywwg ywwg deleted the owilliams/quoted-metric-name branch July 23, 2024 20:21
@ywwg ywwg restored the owilliams/quoted-metric-name branch July 23, 2024 20:21
@ywwg ywwg reopened this Jul 23, 2024
@ywwg
Copy link
Member Author

ywwg commented Jul 23, 2024

accidentally deleted

@ywwg ywwg closed this Sep 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants