proposal: add info-metric label discovery APIs for info() autocomplete - #85
proposal: add info-metric label discovery APIs for info() autocomplete#85aknuds1 wants to merge 3 commits into
Conversation
ab1b9ec to
6bf4a84
Compare
|
|
||
| Constructing the same response with the endpoints that exist today requires either fetching far more than needed or multiple round-trips per autocomplete suggestion. | ||
|
|
||
| * **`/api/v1/labels` + per-name `/api/v1/label/{name}/values`** returns *all* labels matching a selector, including labels carried by the base metric, not just data labels on info metrics. Filtering client-side requires downloading the full universe of labels and then making one `/label/{name}/values` call per label of interest (N+1). |
There was a problem hiding this comment.
This works for the existing autocomplete implementations for PromQL, why doesn't it work for the info function?
There was a problem hiding this comment.
Existing label search API doesn't work for info function autocomplete because it can't derive info series scope. I.e. the info function autocomplete API has to implement info semantics, in order to return data labels matching the expression.
| * The response is NDJSON (`application/x-ndjson`) with the same batch + trailer contract as PROM-74. | ||
| * The endpoint is dual-gated: `--enable-feature=search-api` covers the NDJSON + parsing infrastructure it reuses; `--enable-feature=promql-experimental-functions` covers the only consumer (`info()`). Either missing flag returns the standard Prometheus JSON error with `errorType: unavailable` and a flag-specific message. | ||
|
|
||
| ### `GET|POST /api/v1/info_labels` |
There was a problem hiding this comment.
Given that this new endpoint leverages params and NDJSON from the search endpoint, perhaps this should be /api/v1/search/info_labels
There was a problem hiding this comment.
I thought about this. The (subjective) reason for not going that route was that info function autocomplete endpoint is not in the same logical family. I'm open to rethinking this, however.
There was a problem hiding this comment.
After revising the proposal in the meantime, I kept these as top-level info_* endpoints. The reason being that the endpoints (now split into two) are specific to info(). Please let me know if you still think nesting them under /api/v1/search/ would be preferable.
|
|
||
| ### 1. Extend `/api/v1/search/label_names` to optionally return values per name | ||
|
|
||
| Would collapse two endpoints into one. Rejected on cohesion grounds: PROM-74 keeps names and values in separate endpoints precisely so that each endpoint's response shape stays simple. The values payload is only useful when the names are scoped to info metrics, so the coupling does not belong on the general search endpoint. |
There was a problem hiding this comment.
I would not dismiss this option so quickly. Part of the reason for the search results format being objects/maps per record was so that additional data could be augmented with metric names, label names or label values.
The idea of extending the response format is noted https://github.com/prometheus/proposals/blob/main/proposals/0074-new-labels-values-api.md#extensibility-for-mimir-thanos-cortex.
This endpoint could be extended to include a "include_values=true&values_limit=10" and the values collections could be decorated into each label record.
We already have the "only useful in a certain context" problem on the metric_names endpoint. It supports the include_metadata=true to decorate in metric metadata records for each metric name. But this only can decorate metric names where we have metadata. It's acceptable that this part of the response may not be there.
There was a problem hiding this comment.
Thanks for the feedback @tcp13equals2, will consider.
There was a problem hiding this comment.
The proposal has been significantly revised in the meantime. Are you suggesting though to use /api/v1/search/label_names for info function autocomplete? I'm ruling that out for the reason that the autocomplete API needs info() specific semantics.
|
I've implemented a PoC in From PoC stand point it's working nicely. But we should challenge the return value data type (instead of returning all potential labels/values we might return labels and then values for the selected label. In simple terms I'd like to follow current |
1cb8122 to
8a7dc22
Compare
8a7dc22 to
ebb9e87
Compare
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
Document the request profiles and client requirements learned from the Grafana info() autocomplete proof of concept. Clarify expression interpolation, response bounds, and terminal NDJSON handling. Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
37ac590 to
ab6ea6e
Compare
ab6ea6e to
11a8be7
Compare
@itsmylife This dual-endpoint scheme has been implemented. Thanks for the feedback! |
Signed-off-by: Arve Knudsen <arve.knudsen@gmail.com>
11a8be7 to
477232b
Compare
Add a proposal for two top-level info-metric label discovery endpoints that support autocomplete for the second argument of the PromQL
info()function:GET|POST /api/v1/info_labelssearches info data-label names.GET|POST /api/v1/info_label_valuessearches values for one exact data-label name.Both endpoints share the same scope: an optional instant-vector
expr, repeated fullmetric_match[]matchers on__name__, and repeated fulldata_match[]matchers. Expression-derived storage selection follows the same lookback,offset, and@semantics as an instantinfo()evaluation.The proposal reuses the experimental search API storage and NDJSON contracts while keeping these function-specific operations at top-level paths. It specifies a per-endpoint result limit, pre-search bounds on expression-derived matcher construction, strict stream completion, and the dual
search-api,promql-experimental-functionsfeature gate. Composite storage search is fail-closed: if any participating non-noop backend lacksstorage.Searcher, setup returns a non-streamingerrorType: unavailableresponse rather than silently omitting that backend.The contract has been validated in both the Prometheus UI implementation and the Grafana Prometheus datasource integration.