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

Mark remote read protocol and serving impl as stable. #14158

Open
bwplotka opened this issue May 28, 2024 · 1 comment
Open

Mark remote read protocol and serving impl as stable. #14158

bwplotka opened this issue May 28, 2024 · 1 comment

Comments

@bwplotka
Copy link
Member

Proposal

Our docs mentions remote read as not currently considered part of the stable API and is subject to change even between non-major version releases of Prometheus.. In practice this is not true, we never broke it, even when adding streaming mode. We are particularly careful when attempting any features.

I propose we mark the protocol and serving implementation stable. The motivation comes from DMs asking me if this protocol is really unstable and if they can use it in production. Obviously it's production ready, but some companies require this on paper. Perhaps do it on 3.x?

I am not sure about client implementation, I am personally not interested in maintaining/improving that use case--there are better tools to do aggregate from multiple sources than Prometheus (e.g. Thanos Querier, promxy), but maybe there are different opinions here. It feels we could achieve more deprecating this case for Prometheus (e.g. this blocked some PromQL engine optimizations in the past).

Acceptance Criteria

@bwplotka
Copy link
Member Author

As @GiedriusS mentioned it would be good to address #12605 first.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant